[geary] Don't include internal/private symbols in Engine VAPI
- From: Jim Nelson <jnelson src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [geary] Don't include internal/private symbols in Engine VAPI
- Date: Fri, 13 Feb 2015 02:41:40 +0000 (UTC)
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]