Post-release version bump.

NEWS update for 3.5.2 release.

Updated Galician translations

Update API documentation.

Whitespace cleanups.

Bug #666327 - IMAP deadlock when moving message and checking new mail

with service lock removal the fix for bug #666327
went out (commit 74fcab535c0f50a27742c05e94036b8370ea9173).
Was a good thing as this version is less a hack.
To summarize:
imap folder deadlock:
1. do_copy :
.imap command on source lock the imap store
2. camel_imap_folder_fetch_data on destination:
. lock the destination folder
3. do_copy :
. call the hande user tag : wait for lock on the destination
folder cache .
4. camel_imap_folder_fetch_data on destination:
. wait for lock on the imap store to be freed.

Fix: get the lock on the destination folder cache before locking the
imap store (ie before calling the imap command).

Updated Spanish translation

Updated Spanish translation

Bug #673946 - Cannot delete search folders

Simplify vFolder's code and other bunch of related time optimizations

Bump goa_minimum_version to 3.2.

Just to make it a stable release number.

EFileCache cleanups.

Updated Slovenian translation

libebackend documentation touch ups.

Camel documentation touch ups.

Add e_util_strdup_strip().

Duplicates an input string and strips off any leading or trailing
whitespace.  The resulting string is returned unless it is empty or
NULL, in which case the function returns NULL.

Useful in "set" functions for string properties.
Helps avoid expressions like (str != NULL && *str != '\0').

camel_service_disconnect(): Missing break in switch statement.

Bug 667725 - imapx_untagged: code should not be reached

This code is evil.

When we scan a folder for new messages, we issue a 'FETCH 1:* (UID FLAGS)'
or similar command.

When we receive an untagged FETCH from the server telling us flags for a
message, we make a decision about whether that information was solicited
by such a command, or whether it was unsolicited.

If it was unsolicited, we process it normally as an asynchronous flags
update and all is well.

If it was solicited, we add the UID to a list. When the FETCH (UID FLAGS)
command *completes*, we'll sort that list and then fetch the full headers
for each message.

However, we weren't very good at telling when an update was solicited.
Assuming that only solicited messages will have a UID is bogus.

This was failing if an unsolicited update came in when the (UID FLAGS)
fetch had completed, and we were already fetching the message headers.
The "new" UID would be added to the end of the list, even if we were
already fetching that message or if we already had it in cache. We'd
issue a FETCH command for it, and the barf when the server complied,
because when the UID list wasn't sorted we wouldn't find the offending
uid when we looked for it.

The simple "fix" for this is to keep a boolean flag 'scan_changes' which
is TRUE only when that FETCH (UID FLAGS) command is running. If a flags
change comes in at any other time, it is definitely unsolicited and
should *not* be added to the uidset. This at least protects us from
having UIDs added after we've sorted the list and started to do other
things with it, which was causing the crash.

In fact, this whole 'solicited' vs. 'unsolicited' thing is a design
mistake. In imapx_untagged() we should never care about what we asked
for and what we didn't. That's why the responses are *untagged*. The
server tells us things about the state of the mailboxes, and we should
process that information into our own local cache — it shouldn't
*matter* what we asked for. But that's a more intrusive fix for another day.

In addition, we were reliably *triggering* this behaviour in some cases
because we had to issue a SELECT for the folder in question before
issuing the FETCH (UID FLAGS) command. And on completion of the SELECT,
if UIDNEXT had increased, we were automatically issuing a *new* FETCH
(UID FLAGS) command starting from the last-known-uid in our cache. This
was entirely gratiutous, so use the same scan_changes boolean flag to
avoid it in that situation.

Fix off-by-one in uidset handling for fetches

When we get to the end of the for() loop processing the uidset, the index
variable is one *more* than the last item. That's how for() loops in C
work.

But when we break out of the middle because we are limiting the batch
size, we were forgetting to do the same and we were storing the index
of the last item that we'd already fetched.

This was causing us to re-fetch the last message of one batch, as the
first message of the next batch. Mostly harmless, but spotted and fixed
as part of bug 667725.

Bug 676506 - imapx: After QRESYNC leaves us out of sync, a "full resync" doesn't work either.

Always fetch from the beginning of the folder.

Bug 676403 - imapx: Syncing folder after server-side deletion of many messages is *very* slow

Fix this by introducing a new camel_folder_change_info_remove_uids()
function, and using it for the 'vanished' code. This reduces the time taken
from about 40ms per message, to about 50µs.

Bug 676108 - OpenBSD needs _XOPEN_SOURCE_EXTENDED for strptime(3)

strptime(3) was introduced in XPG4. On OpenBSD, just defining
_XOPEN_SOURCE gets you the older XPG3 which doesn't include strptime(3),
so we need to define _XOPEN_SOURCE_EXTENDED to get XPG4v2.

Updated Galician translations

google: Fix a gcc warning in the Google backend

Signed/Unsigned comparison.

google: Fix some gcc warnings in the GOA authoriser

Variable shadowing, missing default cases and signed/unsigned comparisons.

Bump Camel's shared object name.

For CamelService connect/disconnect API changes.

Remove camel_service_lock/unlock().

This has been the cause of many deadlocks, and is no longer needed with
CamelService's new asynchronous connect/disconnect API.

CamelService: Rewrite connect/disconnect API.

CamelService now has proper cancellable asynchronous functions for
connect and disconnect operations.  The way this works is as follows:

The first service connect request runs asynchronously internally, even
if camel_service_connect_sync() is called.  Subsequent connect requests
are queued until the first connect request finishes, then all requests
finish simultaneously with the same result.  There are no retries; if
the first request fails, they all fail with the same error.

If a service disconnect is requested with one or more connect requests
outstanding, the connect requests are all cancelled and the disconnect
request runs asynchronously and plays by the same queueing rules.

CamelService's "connection-status" property will immediately reflect any
connect or disconnect requests in progress.  Change notification signals
for this property, however, are emitted from CamelSession's main context.

Camel: Use camel_session_idle_add() for idle callbacks.

Ensures all idle callbacks are dispatched from the same GMainContext,
especially important for scheduling signal emissions.

CamelSession: Add camel_session_idle_add().

Schedules an idle callback on CamelSession's GMainContext.

CamelSession: Add a read-only "main-context" property.

Remove camel_folder_has_search_capability()

CamelFolder descendants are supposed to implement search API now.

Camel: Allow SSL certificate resave and use detailed errors from SSL stream

Updated Spanish translation

Updated Swedish translation

Updated Norwegian bokmål translation

Bug #671603 - Birthday calendar broken since 3.3.4

Bug 664639 - [IMAPX] Doesn't quit gracefully on account disable

Signal shutdown when imapx connection shutsdown so that the connection is
pruned from connection manager.

Remove unused variable

Bug #670279 - "Category" selector mnemonic not shown

Bug 675387 - Store certificates under $XDG_DATA_HOME

Move certificates from $HOME/.camel_certs to $XDG_DATA_HOME/camel_certs.

Use g_simple_async_result_set_check_cancellable().

Always call it immediately after g_simple_async_result_new().

Whitespace cleanups.

Updated Italian translation

Use locks in factory's weak object notify callback on data-book/cal

CalDAV: Also rename NoSuchCal to ObjectNotFound in caldav_server_put_object()

Updated Galician translations

Updated Spanish translation

Better error message for SOUP_STATUS_SSL_FAILED response

CalDAV: Report NoSuchCal only when opening, ObjectNotFound otherwise

Make sure data-book/cal is closed/freed in a separate thread

When EDataBook/EDataCal was unreffed in the main thread then
any later calls to the factory were blocked by the backend
till it freed its resources, possibly making client look like
frozen/unresponsive.

Updated POTFILES.in

Add EAsyncClosure.

EAsyncClosure provides a simple way to run an asynchronous function
synchronously without blocking a running GMainLoop or using threads.

Merging this API from the account-mgmt branch as it has proven stable
and quite convenient.  I've seen this pattern in use throughout GIO,
but to my knowledge it has never been formalized like this.

Add a extended response handler for unhandled imapx tokens to be handled
by the derived providers.

Split camel-imapx library and merge into camel so that providers can be written on top of imapx

Bug #606181 - Accepting bad SSL certificate applies to any hostname

Change the Camel certdb to look up certificates by expected hostname.

This way, accepting a bad certificate for one mail server does not give
it a free pass to impersonate the user's other mail servers.  Storing a
second bad certificate for the same server will replace the first, but
that should be OK (Mozilla PSM works the same way).

The camel-cert.db format is unchanged except that it can now contain
multiple entries for the same certificate with different hostnames, and
if it contains multiple certificates for the same hostname, all but the
last will be dropped (becoming permanent the next time the certdb is
saved).

Users who were taking advantage of evolution-data-server's previous,
vulnerable behavior of accepting a certificate for a hostname other than
the originally user-approved one will get bad certificate dialogs and
will need to re-approve the certificate for the desired hostname(s).

Note: Case insensitive compare of host names added by mcrha.

Remove extra test on Google server photo data validity

It sometimes can happen that a contact has filled photo's etag,
while the photo itself is not present. In that case backend skipped
the contact, instead of passing it without photo.

Bug #669191 - Address book dialog has incorrect default size

Require libsoup >= 2.38.1.

To pick up the fix for bug 673678:
"Property SSL_USE_SYSTEM_CA_FILE's value is ignored"

Updated Galician translations

Updated Norwegian bokmål translation

Post-release version bump.

NEWS update for 3.5.1 release.

Update API documentation.

Updated Galician translations

Bug #669085 - Saving draft on Google server emits runtime warning

Updated Spanish translation

Bug #664639 - [IMAPX] Doesn't quit gracefully on account disable

Bug #674257 - Recheck attachment availability on message download

Bug #669034 - [IMAP] Shows "Unknown background operation" on start

Fix Google Talk vCard attribute name definition

Bug #668989 - Add a twitter username to IM in contacts

Revert unintentional API break in libedataserver.

Partially reverses commit 90eb7da3ab9b00f504f5fed1f740c59485356f41.

Apparently this unintentionally broke language bindings.

Instead of renaming the e_source_peek_uid() and e_source_peek_name()
functions themselves, leave the old functions alone and define aliases
for the new names:

   #define e_source_get_uid            e_source_peek_uid
   #define e_source_get_display_name   e_source_peek_name

Using the new names throughout Evo and EDS is very helpful in reducing
diff noise between "master" and the "account-mgmt" branches so I can see
the important differences, so I don't want to revert the commit entirely.

Bug #672398 - CalDAV backend doesn't respect "Copy for offline" option

Bug #668679 - Crash on insertion of NULL in e_destination_set_contact()

Bug #668549 - Crash in objects_added_cb() in e-book-client-view.c

Bug #671537 - Option to validate SLL certificates with libsoup

Remove deprecated API from ESourceSelector

ESourceComboBox: Add e_source_combo_box_ref_active().

Replaces e_source_combo_box_get_active().

Returns a new ESource reference that the caller must unreference.

Remove e_source_combo_box_get/set_active_uid().

Adapt ESourceComboBox to use GtkComboBox's "active-id" property.

To get or set the active ESource UID, use:

   gtk_combo_box_get_active_id()
   gtk_combo_box_set_active_id()

Diff noise reduction.

Comparing 'master' and 'account-mgmt' branches.

Nudge ESource toward its new API.

Rename e_source_peek_uid() to e_source_get_uid(), and
e_source_peek_name() to e_source_get_display_name().

The old names are retained for now as deprecated aliases.

ESourceSelector: Add e_source_selector_ref_primary_selection().

Replaces e_source_selector_get_primary_selection().

Returns a new ESource reference that the caller must unreference.

ESourceSelector: Add e_source_selector_ref_source_by_path().

Replaces e_source_selector_get_source_by_path().

Returns a new ESource reference that the caller must unreference.

Bump libedataserverui shared object name.

In anticipation of some API breaks.

EBookBackendLDAP: Miscellaneous tweaks.

Eliminating diff noise with the account-mgmt branch.

Forgot one.

Add yet more missing linker flags.

Add another missing linker flag.

Add missing linker flag.

Bug #665381 - LDAP backend crash when query_ldap_root_dse() fails

Updated Galician translations

Update API documentation.

Remove some redundant argument checks.

Bug #674309 - Crash when calling oauth_init_nss() from two threads

Updated Norwegian bokmål translation

EDataFactory: Readability tweaks.

Add e_data_factory_ref_backend_factory().

Just a way to get access to the EBackendFactory itself.

Needed on the account-mgmt branch to check whether a suitable E-D-S
backend factory is available to handle a given GNOME Online Account
provider type such as "exchange".

Bug #667891 - Rapid message list regen when downloading a folder