sqlite addressbook: fix memory corruption in get_revision

Instead of setting a pointer, the pointer pointing to the pointer was
overwritten. As a result, e_book_backend_file_load_revision() always saw a
NULL pointer when opening the address book (regardless whether in
evolution-addressbook-factory or a DRA client) and used
e_book_backend_file_bump_revision() to write a new revision.

It is not entirely clear whether overwriting the wrong pointer (which was on
the stack and thus should not have been that important) or these concurrent
writes were to blame, but testing shows that this fix solves a mysterious
sqlite query failure ("corrupt database") in the DRA client when running
SyncEvolution's testpim.py TestContacts.testFilterStartupRefineSmart.

Fixes BGO #697173.

Remove EBookBackendSync.

Useless subclass, added no value other than a slightly less awkward API.

EBookBackendFile: Implement the new backend API.

EBookBackendFile cleanups.

EBookBackend: Add the new backend API alongside the old one.

The new and old EBookBackend class methods will co-exist temporarily,
so I can start converting the simpler EBookBackendSync-based backends
and make final adjustments to the new API.

Remove e_data_cal_is_opened().

With the new public API in place, ECalBackend can own its "opened"
state again.

Remove e_data_book_is_opened().

With the new public API in place, EBookBackend can own its "opened"
state again.

EBookBackend: Fix a GError typo.

Remove e_cal_backend_sync_set_lock().

ECalBackendSync purports to offer serialized method dispatching but
implements it completely wrong.  Backends which rely on this should
instead set use_serial_dispatch_queue in their class_init() method.

ECalBackendClass: Add a 'use_serial_dispatch_queue' option.

ECalBackend subclasses can set this to TRUE to use a serial dispatch
queue, instead of a concurrent dispatch queue.  A serial dispatch queue
executes one method at a time in the order in which methods were called.
This is generally slower than a concurrent dispatch queue, but can help
ECalBackend subclasses avoid thread-safety issues.

EBookBackendClass: Add a 'use_serial_dispatch_queue' option.

EBookBackend subclasses can set this to TRUE to use a serial dispatch
queue, instead of a concurrent dispatch queue.  A serial dispatch queue
executes one method at a time in the order in which methods were called.
This is generally slower than a concurrent dispatch queue, but can help
EBookBackend subclasses avoid thread-safety issues.

ECalClient: Use G_DBUS_MESSAGE_FLAGS_NO_REPLY_EXPECTED for close().

Invoke the close() method without a callback function so the server
knows not to reply.  ECalClient tests that use GTestDBus generally
close their GDBusConnection immediately after disposing ECalClient,
and because we call close() asynchronously the server sometimes has
no chance to reply before the GDBusConnection closes, resulting in
a spurious console warning during the test.

EBookClient: Use G_DBUS_MESSAGE_FLAGS_NO_REPLY_EXPECTED for close().

Invoke the close() method without a callback function so the server
knows not to reply.  EBookClient tests that use GTestDBus generally
close their GDBusConnection immediately after disposing EBookClient,
and because we call close() asynchronously the server sometimes has
no chance to reply before the GDBusConnection closes, resulting in
a spurious console warning during the test.

Updated Basque language

EBookBackendFile: Prefer EClientError domain.

The E_DATA_BOOK_ERROR domain will die when I'm finished with the backend
API rewrite, but I'm not quite ready to go all out with it yet.  This is
just to make the direct access mode tests happy again.

Remove e_return_data_cal_error_val_if_fail() macro.

No longer used, and backends had been mainly misusing it anyway.

Remove e_return_data_cal_error_if_fail() macro.

No longer used, and backends had been mainly misusing it anyway.

Remove e_return_data_book_error_val_if_fail() macro.

No longer used, and backends had been mainly misusing it anyway.

Remove e_return_data_book_error_if_fail() macro.

No longer used, and backends had been mainly misusing it anyway.

ECalBackendSync: Prefer EClientError domain.

Also, don't return GErrors for argument checks.

EBookBackendSync: Prefer EClientError domain.

Also, don't return GErrors for argument checks.

ECalBackend: Prefer EClientError domain.

EBookBackend: Prefer EClientError domain.

EDataCal: Move data_cal_convert_to_client_error() calls.

Move these calls out of the "respond" functions so we don't lose them as
we remove the "respond" functions.  Instead, place the calls immediately
before g_dbus_method-invocation_take_error().

EDataBook: Move data_book_convert_to_client_error() calls.

Move these calls out of the "respond" functions so we don't lose them as
we remove the "respond" functions.  Instead, place the calls immediately
before g_dbus_method_invocation_take_error().

ECalBackend: Convert get_backend_property() method.

This method does not block, so simplify its signature and ditch the
asynchronous wrappers.  Also, ECalBackendSync does not need its own
get_backend_property_sync() method.

Removed functions:

    e_cal_backend_get_backend_property_sync()
    e_cal_backend_get_backend_property_finish()
    e_cal_backend_sync_get_backend_property()
    e_data_cal_respond_get_backend_property()

EBookBackend: Convert get_backend_property() method.

This method does not block, so simplify its signature and ditch the
asynchronous wrappers.  Also, EBookBackendSync does not need its own
get_backend_property_sync() method.

Removed functions:

    e_book_backend_get_backend_property_sync()
    e_book_backend_get_backend_property_finish()
    e_book_backend_sync_get_backend_property()
    e_data_book_respond_get_backend_property()

ECalBackend: Make the "cache-dir" property thread-safe.

New function:

    e_cal_backend_dup_cache_dir()

EBookBackend: Make the "cache-dir" property thread-safe.

New function:

    e_book_backend_dup_cache_dir()

Remove e_cal_backend_set_is_removed().

Not used by any current backend implementations, and I can't even make
sense of the documentation for it.

Remove e_cal_backend_is_removed().

Not used by any current backend implementations, and I can't even make
sense of the documentation for it.

Remove e_book_backend_set_is_removed().

Not used by any current backend implementations, and I can't even make
sense of the documentation for it.

Remove e_book_backend_is_removed().

Not used by any current backend implementations, and I can't even make
sense of the documentation for it.

ECalBackend: Remove set_backend_property() method.

No longer used.

EBookBackend: Remove set_backend_property() method.

No longer used.

ECalBackend: Remove CLIENT_BACKEND_PROPERTY_CACHE_DIR definition.

Backends should use the ECalBackend:cache-dir property directly.

ECalBackend: Remove CLIENT_BACKEND_PROPERTY_READONLY definition.

Backends should use the ECalBackend:writable property directly.

ECalBackend: Remove CLIENT_BACKEND_PROPERTY_ONLINE definition.

Backends should use the EBackend:online property directly.

EBookBackend: Remove CLIENT_BACKEND_PROPERTY_CACHE_DIR definition.

Backends should use the EBookBackend:cache-dir property directly.

EBookBackend: Remove CLIENT_BACKEND_PROPERTY_READONLY definition.

Backends should use the EBookBackend:writable property directly.

EBookBackend: Remove CLIENT_BACKEND_PROPERTY_ONLINE definition.

Backends should use the EBackend:online property directly.

e_data_cal_view_new: Add documentation.

e_data_book_view_new: Update documentation.

[l10n] Updated German translation

Fix implicit declaration of WIFEXITED.

https://bugzilla.gnome.org/show_bug.cgi?id=696858

Updated Hungarian translation

Simplify direct access to address books.

EBookBackend's new public API allows Direct Access Mode to bypass
EDataBook and talk to the EBookBackend directly, which is appropriate
since EDataBook is just glue code for remote D-Bus method invocations.

Modernize ECalBackend's public API.

This leaves ECalBackend and EDataCal engaging in a somewhat ridiculous
kind of ping-pong game for the moment, but the desired public API (in as
much as a backend has a public API) is starting to take shape.

With the scaffolding now in place, we can begin converting backends to
use modern GIO conventions one method at a time, and eventually remove
all the "respond" functions in EDataCal.

Modernize EBookBackend's public API.

This leaves EBookBackend and EDataBook engaging in a somewhat ridiculous
kind of ping-pong game for the moment, but the desired public API (in as
much as a backend has a public API) is starting to take shape.

With the scaffolding now in place, we can begin converting backends to
use modern GIO conventions one method at a time, and eventually remove
all the "respond" functions in EDataBook.

Updated Polish translation

Assamese translation updated

Updated Czech translation

Updated Spanish translation

Update French translation

EDataCalView: Unreference the EGdbusCalView in dispose().

Recent reference counting fixes uncovered this.  We must have been
leaking EDataCalView references for this to not have been an issue
until now.

EDataBookView: Unreference the EGdbusBookView in dispose().

Recent reference counting fixes uncovered this.  We must have been
leaking EDataBookView references for this to not have been an issue
until now.

EDataCalView: Prefer g_clear_object() in dispose().

EDataBookView: Prefer g_clear_object() in dispose().

EDataBookView: Remove private EDataBook member.

Alters the signature of e_data_book_view_new(), but it's a private
function only used by EDataBook.

e_cal_backend_add_view(): Keep a reference on the view being added.

Remove e_cal_backend_set_notification_proxy().

This is some kind of ancient relic whose purpose is unclear.

Seems to predate the Evolution / Evolution-Data-Server split circa 2003.
None of the current E-D-S backends use it, nor even evolution-groupwise.

Bug 696637 - Deadlock in data_cal_view_dispose().

Ownership semantics getting confused here.

e_cal_backend_remove_view() caused the view to be disposed, but
the view object was being too clever for its own good by calling
e_cal_backend_remove_view() from its own dispose().

EDataCal: Fix reference counting of views.

EDataBook: Fix reference counting of views.

EDataCal: Attach to ECalBackend during initialization.

The EDataCal will automatically attach itself to the ECalBackend,
like a parasite at birth.  This ensures the ECalBackend is able to
invoke class methods earlier during EDataCal initialization.

EDataBook: Attach to EBookBackend during initialization.

The EDataBook will automatically attach itself to the EBookBackend,
like a parasite at birth.  This ensures the EBookBackend is able to
invoke class methods earlier during EDataBook initialization.

e_cal_backend_remove_view(): Check for match before unreffing view.

e_book_backend_remove_view(): Check for match before unreffing view.

Bug #656488 - imapx: Memory leak of mi->server_user_flags

Bug #695915 - IMAP alert is not displayed

Bug #693101 - IMAPx can vanish and redownload folder summary

Bug 696285 - ownCloud: Check addressbook even if calendar fails

Updated Odia Language along with FUEL implementation

Updated gujarati file

Updated kn translations

Updated kn translations

[l10n] Updated German translation

updating Hindi Translation

Updated slovak translation

Updated kn translations

Updated kn translations

l10n: Update Japanese translation

Finnish translation update by Jiri Grönroos

Updated Slovenian translation

libedata-cal: Remove all deprecated backend APIs.

Clean the slate for the next round of changes.

All known 3rd party backends have been accounted for (ews/mapi/kolab),
so this should cause no disruption.

libedata-book: Remove all deprecated backend APIs.

Clean the slate for the next round of changes.

All known 3rd party backends have been accounted for (ews/mapi/kolab),
so this should cause no disruption.

Updated Odia Language along with FUEL implementation

Updated slovak translation

Let EDataCal track the "opened" state.

Due to the asymmetric nature of our backend APIs, the open() method
begins in ECalBackend, but the end result is reported to EDataCal.
So it makes more sense for EDataCal to track the backend's "opened"
state, at least until the APIs can be modernized.

This adds a new function -- e_data_cal_is_opened() -- which is just a
temporary hack and should only be called by e_cal_backend_is_opened().

This avoids EDataCal needing to call e_cal_backend_notify_opened(),
which is deprecated and about to be removed.

Let EDataBook track the "opened" state.

Due to the asymmetric nature of our backend APIs, the open() method
begins in EBookBackend, but the end result is reported to EDataBook.
So it makes more sense for EDataBook to track the backend's "opened"
state, at least until the APIs can be modernized.

This adds a new function -- e_data_book_is_opened() -- which is just a
temporary hack and should only be called by e_book_backend_is_opened().

This avoids EDataBook needing to call e_book_backend_notify_opened(),
which is deprecated and about to be removed.

Tamil Translations Updated

EDataCal: Keep a weak reference on its ECalBackend.

The ECalBackend now holds a strong reference on its EDataCal, so we need
to break the reference cycle.

New functions:

    e_data_cal_ref_backend()

Deprecated functions:

    e_data_cal_get_backend()

Create only one EDataCal per ECalBackend.

I've never understood why we create multiple EDataCal instances per
ECalBackend.  All it's doing is exporting a Calendar interface on the
session bus and handling incoming method invocations by forwarding them
to the ECalBackend.

The GDBusMethodInvocation identifies the client, so we don't need to
export a separate Calendar interface per client.  This seems to be a
relic of the Bonobo-era design.

Fixing this redundancy is a prerequisite to running backends in child
processes, which is one of my goals for 3.10.

New functions:

    e_cal_backend_ref_data_cal()
    e_cal_backend_set_data_cal()

Deprecated functions:

   e_cal_backend_add_client()
   e_cal_backend_remove_client()

EDataBook: Keep a weak reference on its EBookBackend.

The EBookBackend now holds a strong reference on its EDataBook, so we
need to break the reference cycle.

New functions:

    e_data_book_ref_backend()

Deprecated functions:

    e_data_book_get_backend()

EDataBook: Silence runtime warnings in direct access mode.

e_data_book_report_backend_property_changed() apparently has no way to
report backend property changes in direct access mode.  Return silently.

Create only one EDataBook per EBookBackend.

I've never understood why we create multiple EDataBook instances per
EBookBackend.  All it's doing is exporting an AddressBook interface on
the session bus and handling incoming method invocations by forwarding
them to the EBookBackend.

The GDBusMethodInvocation identifies the client, so we don't need to
export a separate AddressBook interface per client.  This seems to be
a relic of the Bonobo-era design.

Fixing this redundancy is a prerequisite to running backends in child
processes, which is one of my goals for 3.10.

New functions:

    e_book_backend_ref_data_book()
    e_book_backend_set_data_book()

Deprecated functions:

    e_book_backend_add_client()
    e_book_backend_remove_client()

EDataCalFactory: Rewrite client connection tracking.

Keep a hash table of client bus names mapped to a set of ECalBackend
references.  When the backend emits a closed() signal with the client's
bus name, we remove that ECalBackend reference from the data structure.

Also keep a hash table of client bus names we're watching so that if a
bus name vanishes we can remove all ECalBackend references associated
with it.

EDataBookFactory: Rewrite client connection tracking.

Keep a hash table of client bus names mapped to a set of EBookBackend
references.  When the backend emits a closed() signal with the client's
bus name, we remove that EBookBackend reference from the data structure.

Also keep a hash table of client bus names we're watching so that if a
bus name vanishes we can remove all EBookBackend references associated
with it.

EDataCal: Only cancel ops belonging to the sender when closing.

EDataBook: Only cancel ops belonging to the sender when closing.