[geary] Don't include internal/private symbols in Engine VAPI



commit 13665f77839a763199f27b68630bce9f85a770a1
Author: Jim Nelson <jim yorba org>
Date:   Thu Feb 12 18:39:33 2015 -0800

    Don't include internal/private symbols in Engine VAPI
    
    Removing internal and private symbols from geary-static.vapi results
    in removing 1,000 lines from the file.  Although testing doesn't show
    an appreciable increase in compile time (w/ make -j8 on my 8-core
    machine), by reducing symbol leakage partial builds should improve.
    
    This patch also fixes a couple of symbols marked internal but being
    used publicly (and necessarily so) and a couple of errordomains in
    the Engine not placed in the Geary namespace.

 cmake/ValaPrecompile.cmake                         |    2 +-
 src/engine/api/geary-account-information.vala      |    7 ++++++-
 src/engine/api/geary-search-folder.vala            |    4 ++--
 .../gmail/imap-engine-gmail-search-folder.vala     |    2 +-
 src/engine/mime/mime-error.vala                    |    2 +-
 src/engine/nonblocking/nonblocking-error.vala      |    2 +-
 src/engine/rfc822/rfc822.vala                      |    2 +-
 7 files changed, 13 insertions(+), 8 deletions(-)
---
diff --git a/cmake/ValaPrecompile.cmake b/cmake/ValaPrecompile.cmake
index 89dae62..42ad3e3 100644
--- a/cmake/ValaPrecompile.cmake
+++ b/cmake/ValaPrecompile.cmake
@@ -170,7 +170,7 @@ macro(vala_precompile output source_bundle_name)
     set(vapi_arguments "")
     if(ARGS_GENERATE_VAPI)
         list(APPEND out_files "${DIRECTORY}/${ARGS_GENERATE_VAPI}.vapi")
-        set(vapi_arguments "--internal-vapi=${ARGS_GENERATE_VAPI}.vapi")
+        set(vapi_arguments "--vapi=${ARGS_GENERATE_VAPI}.vapi")
         
         # Header and internal header is needed to generate internal vapi
         if(NOT ARGS_GENERATE_HEADER)
diff --git a/src/engine/api/geary-account-information.vala b/src/engine/api/geary-account-information.vala
index 1c9ad5d..7f9f49a 100644
--- a/src/engine/api/geary-account-information.vala
+++ b/src/engine/api/geary-account-information.vala
@@ -52,7 +52,12 @@ public class Geary.AccountInformation : BaseObject {
     
     private static Gee.HashMap<string, Geary.Endpoint>? known_endpoints = null;
     
-    internal File? settings_dir = null;
+    /**
+     * Location account information is stored (as well as other data, including database and
+     * attachment files.
+     */
+    public File? settings_dir { get; private set; default = null; }
+    
     internal File? file = null;
     
     // IMPORTANT: When adding new properties, be sure to add them to the copy method.
diff --git a/src/engine/api/geary-search-folder.vala b/src/engine/api/geary-search-folder.vala
index faee7d9..92363ae 100644
--- a/src/engine/api/geary-search-folder.vala
+++ b/src/engine/api/geary-search-folder.vala
@@ -4,7 +4,7 @@
  * (version 2.1 or later).  See the COPYING file in this distribution.
  */
 
-public class Geary.SearchFolderRoot : Geary.FolderRoot {
+private class Geary.SearchFolderRoot : Geary.FolderRoot {
     public const string MAGIC_BASENAME = "$GearySearchFolder$";
     
     public SearchFolderRoot() {
@@ -12,7 +12,7 @@ public class Geary.SearchFolderRoot : Geary.FolderRoot {
     }
 }
 
-public class Geary.SearchFolderProperties : Geary.FolderProperties {
+private class Geary.SearchFolderProperties : Geary.FolderProperties {
     public SearchFolderProperties(int total, int unread) {
         base(total, unread, Trillian.FALSE, Trillian.FALSE, Trillian.TRUE, true, true, false);
     }
diff --git a/src/engine/imap-engine/gmail/imap-engine-gmail-search-folder.vala 
b/src/engine/imap-engine/gmail/imap-engine-gmail-search-folder.vala
index 66fefd4..c11f0c9 100644
--- a/src/engine/imap-engine/gmail/imap-engine-gmail-search-folder.vala
+++ b/src/engine/imap-engine/gmail/imap-engine-gmail-search-folder.vala
@@ -7,7 +7,7 @@
 /**
  * Gmail-specific SearchFolder implementation.
  */
-public class Geary.ImapEngine.GmailSearchFolder : Geary.SearchFolder {
+private class Geary.ImapEngine.GmailSearchFolder : Geary.SearchFolder {
     private Geary.App.EmailStore email_store;
     
     public GmailSearchFolder(Geary.Account account) {
diff --git a/src/engine/mime/mime-error.vala b/src/engine/mime/mime-error.vala
index 58520b9..639f17b 100644
--- a/src/engine/mime/mime-error.vala
+++ b/src/engine/mime/mime-error.vala
@@ -8,6 +8,6 @@
  * Errors related to { link Geary.Mime}.
  */
 
-public errordomain MimeError {
+public errordomain Geary.MimeError {
     PARSE
 }
diff --git a/src/engine/nonblocking/nonblocking-error.vala b/src/engine/nonblocking/nonblocking-error.vala
index ac5c0f2..8972baa 100644
--- a/src/engine/nonblocking/nonblocking-error.vala
+++ b/src/engine/nonblocking/nonblocking-error.vala
@@ -4,7 +4,7 @@
  * (version 2.1 or later).  See the COPYING file in this distribution.
  */
 
-public errordomain NonblockingError {
+public errordomain Geary.NonblockingError {
     /**
      * Indicates a call was made when it shouldn't have been; that the primitive was in such a
      * state that it cannot properly respond or account for the requested change.
diff --git a/src/engine/rfc822/rfc822.vala b/src/engine/rfc822/rfc822.vala
index 170d5c5..4acfa04 100644
--- a/src/engine/rfc822/rfc822.vala
+++ b/src/engine/rfc822/rfc822.vala
@@ -10,7 +10,7 @@ private int init_count = 0;
 
 internal Regex? invalid_filename_character_re = null;
 
-internal void init() {
+public void init() {
     if (init_count++ != 0)
         return;
     


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]