Coding style and whitespace cleanup.

ECalBackendWeather: e_weather_source_new() is failable.

Backend logic was failing to take this into account and crashing if the
weather location code in the ESource was invalid.

I'm not sure if EWeatherSource is even worth keeping around, but I'll
come back to that later.  Not doing an all-out rewrite at the moment.

EWeatherSource cleanups.

Bug 699024 - [google/yahoo] Don't override username

When mail child sources are added to a google or yahoo collection, avoid
overriding an existing user name setting.  The IMAP or (especially) SMTP
configuration may have been modified to use a non-Google/Yahoo! server.

ECalClientView: Make D-Bus signal handlers thread-safe.

Seeing crashes now where handler functions for D-Bus signals like
"objects-added" are receiving finalized ECalClientView instances.
Possibly a side-effect of recent changes to GMainContext usage.

Use the trick of passing an allocated GWeakRef along with a suitable
GClosureNotify to g_signal_connect_data().  The handler function will
just return quietly if it can't obtain a strong client view reference.

EBookClientView: Make D-Bus signal handlers thread-safe.

Seeing crashes now where handler functions for D-Bus signals like
"objects-added" are receiving finalized EBookClientView instances.
Possibly a side-effect of recent changes to GMainContext usage.

Use the trick of passing an allocated GWeakRef along with a suitable
GClosureNotify to g_signal_connect_data().  The handler function will
just return quietly if it can't obtain a strong client view reference.

Bug 699223 - Emit view signals from context where view was started

This applies to both EBookClientView and ECalClientView.

Incoming D-Bus signals were being propagated to the EClient's main loop
context, but there may not be a running main loop on that thread, so the
main loop sources were never being dispatched and the view signals never
emitted.

Instead use the main loop context which was the thread-default at the
time the view was started, which is often the so-called "main" thread
where a main loop is more likely to be running.

IMAPX: Recognize RFC 5530 response codes.

Bug 699160 - IMAPX: Always flush debugging messages

Had a debugging scenario where key information was not written to a log
file until another IMAP command was issued.  Caused a bit of confusion.
This commit ensures that won't happen again.

camel_imapx_debug() cleanups.

Make camel_mime_message_dump() dump message contents

Previously unimplemented.  Dumps in hex and ASCII:

class: CamelMimeMessage
mime-type: text/plain; charset="UTF-8"
content class: CamelDataWrapper
content mime-type: text/plain; charset="UTF-8"
content len 967
    4f 6e 20 54 68 75 2c 20 32 30 31 33 2d 30 35 2d    On Thu, 2013-05-
    30 32 20 61 74 20 31 34 3a 31 37 20 2d 30 35 30    02 at 14:17 -050
    30 2c 20 44 61 6e 20 57 69 6c 6c 69 61 6d 73 20    0, Dan Williams
    77 72 6f 74 65 3a 0a 3e 20 4f 6e 20 54 68 75 2c    wrote:.> On Thu,
    20 32 30 31 33 2d 30 35 2d 30 32 20 61 74 20 31     2013-05-02 at 1
    31 3a 35 33 20 2d 30 37 30 30 2c 20 4c 69 6e 75    1:53 -0700, Linu
    73 20 54 6f 72 76 61 6c 64 73 20 77 72 6f 74 65    s Torvalds wrote
    3a 0a 3e 20 3e 20 4f 6e 20 54 68 75 2c 20 4d 61    :.> > On Thu, Ma
    79 20 32 2c 20 32 30 31 33 20 61 74 20 31 31 3a    y 2, 2013 at 11:

(02:09:38 PM) mbarnes: dcbw: nice, I forgot about camel_mime_message_dump() -- go ahead and commit that

Bug 699461 - Accept any 2xx HTTP code as successful DELETE

ESource: Fix a compiler warning.

Bug 698725 - ESourceRegistry: Handle service restarts gracefully

Check GDBusObjectManagerClient's "name-owner" property when handling
"object-added" and "object-removed" signals.  If the property is NULL,
implement a strategy that swaps out the ESource's old proxy object for
an equivalent replacement such that applications should remain unaware
of the service restart.

This is important to handle gracefully not only for crash recovery, but
also when the registry service receives a Reload() method invocation or
a SIGHUP, both of which cause it to temporarily release its well-known
bus name and reload key files.

e_source_ref_dbus_object: Use the property lock.

ESourceRegistry is going to swap out the GDBusObject as part of the
recovery strategy for evolution-source-registry restarts, so we need
to make sure e_source_ref_dbus_object() is properly thread-safe.

ESourceRegistry: object_path_table cleanups.

Add source_registry_dbus_object_dup_uid().

Handy internal utility for ESourceRegistry.

This also happens to fix an EDBusSource reference leak in
source_registry_create_sources_object_added_cb().

Updated Slovenian translation

Post-release version bump.

NEWS update for 3.9.1 release.

Fix reference leak in book_client_view_emit_objects_modified().

This was causing test-client-photo-is-uri to fail.

EGoaClient nitpicks.

Bug 698726 - Handle goa-daemon crashes/restarts gracefully

EGoaClient is an improved GoaClient that handles daemon crashes and
restarts gracefully so E-D-S accounts are not destroyed unnecessarily.

Updated Czech translation

Tamil Translation Updated

Updated Greek translation

Improve error handling in imapx_server_get_message().

If a GET_MESSAGE job for the same UID is already in progress, wait for
the job to finish, but don't assume it was successful.  Try retrieving
the message from cache, but if that fails then we have to re-fetch the
message from the IMAP server.

ECalClient test: test-client-get-revision not broken anymore

Not sure how this is passing again, but it is.

Do not consider PGP and S/MIME subparts as attachments

test-client-custom-summary.c: Optimize test.

Reuse the same two addressbooks for every test, so that only the
first 2 tests populate the addressbooks with contacts and the remaining
tests only create a client and execute a query.

Also add all test contacts with a single batch command and dont fetch
the test contacts, also removed the #if 0'd case which is the only
case which required the local contact copies.

test-server-utils: Allow reuse of existing addressbook.

If committing the new scratch source ends with an error that
the addressbook/calendar already exists, then just use the existing
source instead of bailing out.

This is needed for tests which reuse the addressbook from a previous test.

Conflicts:

tests/test-server-utils/e-test-server-utils.c

Updated Spanish translation

[l10n] Updated Catalan (Valencian) translation

[l10n] Update Catalan translation

ECalClientView: Remove unnecessary fetching of ECalClient.

Left over from the D-Bus error stripping work.

EBookClientView: Remove unnecessary fetching of EBookClient.

Left over from the D-Bus error stripping work.

Bug #698261 - Skip invalid flags in imapx_parse_flags()

Previous patch had incorrectly implemented skip of the broken flag,
which fixes this change.

Bug #698261 - Skip invalid flags in imapx_parse_flags()

CamelStoreSettings::filter-inbox: Change default to TRUE

LC_ADDRESS is a GNU extension. Treat it as such.

Unconditional use of the GNU extension LC_ADDRESS causes compiler
errors on non-GNU systems.

In test cases, use LC_MESSAGES if LC_ADDRESS is unavailable.
Also fail gracefully with a log message if setlocale() fails.

ESource: Move "remember-password" to Authentication extension.

It's a more suitable place since the option is only relevant to
EAuthenticationSession.

Bug #691939 - Password prompt has "save password" always checked

Remove backward-compatibility cruft for goa-1.0 < 3.8.

In particular, GOA's Google provider uses OAuth 2.0 now, so we can
drop the OAuth 1.0a support in EGDataGoaAuthorizer and with it the
last lingering goa-1.0 dependency outside of the GOA module.

Bump minimum goa-1.0 requirement to 3.8.

Still an optional dependency.  This is to purge backward-compat cruft.

Switch NULL to empty string in e_util_utf8_{data_}make_valid()

It's used on various places which expect a valid string, but which
doesn't check for NULL values of the string, which can cause a crash,
like xtian found with e_data_cal_respond_get_object(), which can have
passed in a NULL object. This change is also to avoid NULL-checking
on too many places.

EBookBackendSqliteDB: Avoid summary introspection on initial creation.

The summary introspection does not introspect whether regular prefix
indexes are created, which is acceptable since it does not effect queries.

However since we introspect before creating the indexes, we lose the
configured index information which is needed to initially create those
indexes.

This patch simply checks if the table exists when initially creating
the summary and avoids the introspection if the table is being created
for the first time.

Automatically strip D-Bus info from error messages.

I mistakenly assumed code generated by the gdbus-codegen tool would
automatically strip off the D-Bus error name from error messages, but
that apparently is not possible according to my discussion with David
Zeuthen in [1], since it would break g_dbus_error_is_remote_error().

Distinguishing between local and remote errors strikes me as a rarely
needed corner case, and is certainly not worth the extra step imposed
on applications when dealing with something as fundamental as GError.

This commit prevents the D-Bus error name from leaking through in any
Evolution-Data-Server client-facing library functions.

Also rename any stack-allocated GError pointers to 'local_error' to
help distinguish them from GError "out" parameters which are always
named 'error'.  This is just to improve variable name consistency.

[1] https://bugzilla.gnome.org/697819

Bug 695578 - Trailing space in translatable string

Bug 697705 - e_cal_client_remove_objects_sync() crashes on empty UID

Test suite cleanups.

build: Fix srcdir != builddir from git

We can't do (cd $(srcdir); ...) and inside reference $(top_srcdir)
because that variable uses a *relative* path.  Thus we copy the
approach from gnome-shell of explicitly using addprefix to append the
source directory.

Bug #696757 - Do not use dynamic keys in g_object_set_data()

Bug #681837 - Make enum type registration thread safe

Bug 697734 - Memory leak in local calendar backend

We were leaking an icalproperty while seeding the "revision" D-Bus
property during instance initialization, because the backend does
not yet have an icalcomponent to hold the icalproperty.

This commit avoids the leak during instance initialization, but also
makes sure the "revision" property gets updated once the backend does
have an icalcomponent.

ECalBackendFile: Indentation cleanups.

Tabs, not spaces.

Updated Hungarian translation

CamelIMAPXServer: Rewrite duplicate message fetching.

The g_cond_wait() call in imapx_server_get_message() seems to be the
source of many deadlocks.  Possibly because some of the error handling
branches in that function fail to broadcast the GCond being waited on,
and cancellation is a very common error condition, especially when the
user is rapidly browsing through messages.

The code is attempting to wait for the completion of a GET_MESSAGE job
for the same UID which is already queued.  It's doing this by defining
its own GMutex and GCond, but it turns out we already have a function
for this: camel_imapx_job_wait().

Let's try using camel_imapx_job_wait() there.  It can't do worse than
the current situation, and we can kill off the specialized GMutex and
GCond variables which were unfortunately made public.

Bug #271262 - Allow Send/Receive of local stores in offline

EBookBackendWebdav: Defer to default get_contact_list_uids_sync().

EBookBackendGoogle: Defer to default get_contact_list_uids_sync().

EBookBackend: Make get_contact_list_uids_sync() optional.

This method is now optional.  By default, get_contact_list_uids_sync()
simply calls get_contact_list_sync() and extracts UID strings from the
matched EContacts.  Subclasses may override this if they can implement
it more efficiently.

EBookBackend: Minor code rearrangement.

EBookBackendGoogle: Implement the new backend API.

At the same time I converted back to using synchronous GData functions,
which makes the logic flow much easier to follow (in my opinion).

I'm not sure why the asynchronous functions were used in the first place
when each backend method is invoked from a worker thread where it's safe
to block.  Maybe the threading model was different at the time, I really
don't know.  The commit history does not reveal the rationale.

EBookBackendGoogle cleanups.

Updated Danish translation

EBookBackendWebdav: Remove unused macros.

EBookBackendWebdav: Implement the new backend API.

EBookBackendWebdav cleanups.

Updated POTFILES.in

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.