[tracker] evolution: Fixed includes ... yet again
- From: Martyn James Russell <mr src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [tracker] evolution: Fixed includes ... yet again
- Date: Mon, 2 Apr 2012 17:28:36 +0000 (UTC)
commit 5fc7616ebb89386b7575e4f831899d341aa34a57
Author: Martyn Russell <martyn lanedo com>
Date: Mon Apr 2 18:25:16 2012 +0100
evolution: Fixed includes ... yet again
So I have this time put each include in separate sections. The logic was
getting too complicated and it was hard to maintain. This is why single
include files make so much sense.
I've checked each release of the libraries we import from and the files do
exist for each of the versions we support.
I removed the camel include because all the Evolution includes include that
already.
I also removed the include for older than 2.91 versions which didn't exist
there and was reported under the bug below.
Fixes GB#672415
src/plugins/evolution/tracker-evolution-plugin.c | 39 ++++++++++++----------
1 files changed, 21 insertions(+), 18 deletions(-)
---
diff --git a/src/plugins/evolution/tracker-evolution-plugin.c b/src/plugins/evolution/tracker-evolution-plugin.c
index 4c4692f..9e31753 100644
--- a/src/plugins/evolution/tracker-evolution-plugin.c
+++ b/src/plugins/evolution/tracker-evolution-plugin.c
@@ -41,41 +41,44 @@
#include <sqlite3.h>
-#include <libedataserver/eds-version.h>
-
-#include <camel/camel.h>
-
-#ifdef EVOLUTION_SHELL_3_2
-#include <mail/em-utils.h>
-#include <mail/e-mail.h>
-#endif /* EVOLUTION_SHELL_3_2 */
-
-#ifdef EVOLUTION_SHELL_3_3_5
+/* This is why you should seal your headers with one include for
+ * client sanity. This is the horendous shit we've had to put up with
+ * for years from the Evo shell API :(
+ */
+#if defined(EVOLUTION_SHELL_3_3_5)
#include <libemail-engine/e-mail-folder-utils.h>
#include <libemail-engine/e-mail-session.h>
#include <libemail-engine/mail-config.h>
#include <libemail-engine/mail-ops.h>
#include <libemail-utils/e-account-utils.h>
-#else /* EVOLUTION_SHELL_3_3_5 */
-#include <mail/e-mail-folder-utils.h>
+#elif defined(EVOLUTION_SHELL_3_2)
+#include <mail/em-utils.h>
+#include <mail/e-mail.h>
#include <mail/mail-config.h>
#include <mail/mail-ops.h>
#include <e-util/e-account-utils.h>
-#ifdef EVOLUTION_SHELL_2_91
-#include <mail/e-mail-session.h>
-#else /* EVOLUTION_SHELL_2_91 */
+#elif defined(EVOLUTION_SHELL_2_91)
+#include <mail/e-mail.h>
+#include <mail/mail-config.h>
+#include <mail/mail-ops.h>
+#include <e-util/e-account-utils.h>
+#else /* OLDER, mostly 2.32 or so */
+#include <mail/mail-config.h>
+#include <mail/mail-ops.h>
#include <mail/mail-session.h>
-#endif /* EVOLUTION_SHELL_2_91 */
-#endif /* EVOLUTION_SHELL_3_3_5 */
+#include <e-util/e-account-utils.h>
+#endif
-#include <mail/e-mail-backend.h>
+/* Remaining includes which are in ALL versions: */
#include <shell/e-shell.h>
#include <e-util/e-config.h>
#include <libedataserver/e-account.h>
#include <libedataserver/e-account-list.h>
+#include <libedataserver/eds-version.h>
+/* Back to sanity */
#include <libtracker-sparql/tracker-sparql.h>
#include <libtracker-common/tracker-date-time.h>
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]