online-accounts: Mail sources need to be writable.

Unfortunately there is not a clean separation between GOA-dictated
settings and user preferences in mail account ESource triplets, so they
all need to be writable and it's up to the client application to defend
the settings which must not be changed.

ECollectionBackend: Minor cleanup.

Updated Belarusian translation.

Bug 559391 - Wrong number of unread mails in folder tree

This reverts part of the original fix for this bug which stopped reading
from the summary database and caused all kinds crazy message counts to
appear in Evolution's folder tree (often negative counts, or negative
counts cast to an unsigned integer).

Message counts are back to normal now.

Reverts part of commit 42489b26c45956c66186a5dc96083437cf7a579a.

Bug #578240 - Impossible to cancel search in search folders

Assamese translation updated

Updated Traditional Chinese translation(Hong Kong and Taiwan)

Preset timeout on SoupSession-s to 90 seconds

The default namespace is an empty string in IMAP settings

Updated POTFILES.in

Updated Galician translations

Bug #677886 - Runtime warning from ESourceAlarms

Camel: Use folder display name is status messages when possible.

And also put the folder name in quotes.

camel_folder_expunge_sync(): Push a status message.

CamelSubscribable: Connect before [un]subscribing.

CamelSubscribable: Push status messages for blocking operations.

Bug 678602 - Inform server of client-side auth cancellations.

server_side_source_remove(): Disregard G_IO_ERROR_NOT_FOUND.

CamelStoreClass: Rename compare_folder_name to equal_folder_name.

CamelStoreClass has the following method:

    GCompareFunc compare_folder_name;

Both the method name and call signature imply this is a strcmp()-style
function where a zero return value means the values are equal.  But in
fact this method is used as an equality test where a zero return value
means the values are NOT equal.

Rename the method and change its call signature like so:

    GEqualFunc equal_folder_name;

This is an API break but clearly a necessary one.

evolution-source-viewer.c: Fix a frequent runtime warning.

Updated Norwegian bokmål translation

Remove source_extension_constructed().

Was written before GObject implemented its own constructed() method.

CamelImapSettings: Change namespace default to empty string.

Revert "Bug #678901 - Fake ESource::changed notifications"

This reverts commit c982babcd69d1059ecbe01d2de9bade4073d9859.

Breaks "changed" notifications in EServerSideSource and causes
Bug 678842 - Enable GMail GOA Calendar doesn't add it to UI

evolution-source-viewer: Fix icon column alignment.

evolution-source-viewer: Change "UID" label to "Identity".

So it's consistent with the tree view column.

evolution-source-viewer: Add delete capability.

Easy way to get rid of botched or unwanted test accounts.

Bug 678903 - GConf migration omits so-called "send-only" accounts

Disregard G_FILE_ERROR_NOENT during GConf migration.

Not finding GConf XML files is a good thing!

Bug #559391 - Wrong number of unread mails in folder tree

Updated Bulgarian translation

Fix make distcheck

Bug #678901 - Fake ESource::changed notifications

Updated Greek translation

Remove redundancy from CRITICAL message.

camel_provider_list(): Emit CRITICAL if camel_provider_load() fails.

Otherwise CamelProvider modules may be silently dropped without so much
as a console message.

Updated Vietnamese translation

po/vi: import from Damned Lies

libedata-cal.pc.in: Add a 'libexecdir' variable.

libedata-book.pc.in: Add a 'libexecdir' variable.

Bug #677697 - Huge memory leaks

weather: update for newer GWeather

GWeather 3.6 changed API to be properly namespaced and GObject based.
Also, WeatherLocation was removed and replaced with GWeatherLocation
in all public API (with additionally the possibility to retrieve one
particular GWeatherLocation by station code)
At the same time, remove a old and now useless abstration by folding
EWeatherSourceCCF into EWeatherSource.

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

Fix configure warning about BUILT_SOURCES

ECollectionBackend: Add default method for dup_resource_id().

Now checks for an E_SOURCE_EXTENSION_RESOURCE by default and returns the
"identity" value if the extension if present.  Otherwise it returns NULL.

Add ESourceResource extension type.

Intended to be used by collection backends to note the server-assigned
name of a calendar, address book, or any other kind of remote resource
that an ESource can represent.

Avoid including <db.h> in a public header file.

Don't want Berkeley DB to be a requirement in libedata-book.pc.

build: libebook: Fix race condition in gen-western-table.py

In a parallel build, the normal compiler process will race with the
process writing to the table header.  Depending on timing, it may or
may not work.

Fix this in two ways:

* Write the file atomically, so #include either gets the whole thing,
  or nothing.
* Add it to BUILT_SOURCES so automake knows to build it first.

Updated Spanish translation

Incorrect check for CamelSettings value change

This was causing runtime warnings on evolution start from IMAP provider,
which reads values from settings for junk/trash real folder, but
the settings object was set as NULL, which means "create default settings
for this class", but the test I added for object pointer change did
succeed for the NULL object (both internal and passed in settings are NULL
at the beginning), thus the default settings object was not created.

CamelIMAPXServer: use g_return_if_fail() family of functions instead of g_assert()

* changed the function param sanity checking from assertions
  to the g_return*_if_fail() family of functions, so we can
  stumble along instead of barfing in front of the user

IMAPX: (utils) use g_return_if_fail() family of functions instead of g_assert()

* changed the function param sanity checking from assertions
  to the g_return*_if_fail() family of functions, so we can
  stumble along instead of barfing in front of the user

CamelIMAPXServer: exposed command run function in public API

* in order to actually make use of the untagged response
  handler extensions, CamelIMAPXStore derivatives will
  need to send custom commands to the IMAP server
* the newly exposed API function provides a means for this

IMAPX: (utils) made capabilities flags extensible

* added a hash table as an IMAP capabilities flags
  lookup mechanism for the capabilities parser util
  function
* added a capabilities strings registration function
  (to add new capabilities flags, which will then be
  known to the parser and assigned a new flag bit.
  This is returned by the registration function so
  the caller can test whether the bit is set in the
  CamelIMAPXServer capabilities bitstring)

CamelIMAPXServer: added documentation for untagged handler descriptor struct

CamelIMAPXServer: added untagged response handler registration API function

* register new untagged response handlers with this function
* can also be used to replace existing handlers
  (think twice before doing that), while keeping track
  of the original handler function

CamelIMAPXServer: do not hide possible programming issue

* do not free the CamelIMAPXServer private context
  structure at the beginning of imapx_untagged()
  as it is expected to be NULL
* if it isn't, the function got called concurrently
  for the same CamelIMAPXServer instance, in which
  case we need special locking mechanisms anyway
* thanks to Milan Crha for pointing this out

CamelIMAPXServer: use function table for untagged handler dispatching

* added a handler GHashTable to CamelIMAPXServer private data
* CamelIMAPXServer instance init populates the table with
  an initial set of handler functions (the ones previously
  existing in the class)
* using that function table in imapx_untagged() now to
  find the handler function registered for a given untagged
  response

IMAPX: added string definitions for untagged responses

CamelIMAPXServer: moved handler code for untagged responses to individual functions

* this is the first step towards an extensible CamelIMAPXServer
  w.r.t. untagged IMAP responses
* there is now one handler function per untagged response
* instead of one big switch statement, there will be a
  table with pointers to the new handler function, which
  will be indexed by the untagged response code
* this will eventually eliminate the switch statement
  in imapx_untagged()
* the gperf dependency will be dropped in the process

Fix install failure in google backend.

Instead of linking to libebook-google-utils.la, just add its source
files directly to the libebookbackendgoogle module.   So the code is
duplicated in the backend module and the noinst unit test library.

This was causing libtool to barf on bogus undefined reference errors
during install.  Still don't really understand what libtool's problem
was -- maybe it didn't like the noinst library?  Who knows.

Bug #678856 - Crash in add_instance() at e-cal-client.c:1311

Assamese translation reviewed

google: Remove an outdated workaround for an EBookBackendCache bug

The bug itself was fixed in commit d535d4b09c61a8277084ddce0cfafc737b69fab2,
two years ago.

google: Remove the option to not cache Google Contacts data locally

We always download the entire contacts list, so there’s little point in not
caching it on disk. This saves bandwidth and speeds up initial access to the
address book. The cache file shouldn’t get unreasonably big, given that users
will typically only have a few hundred contacts in their address book.

vala: Add custom overrides for e_contact_[get|get_const|set]()

They were using void* in Vala, which is generally a bad idea. In this case,
it was resulting in unavoidable memory leaks in libfolks (bug #678781).

This changes the functions to be simple generics instead.

Post-release version bump.

NEWS update for 3.5.3 release.

Camel string pool improvements.

Keep the pointer to the allocated string and the allocated string's
reference count together in a new StringPoolNode struct.  This way a
StringPoolNode can be used as both the key and value in a GHashTable,
allowing the table to save some memory.  Also use the slab allocator
for the fixed-size StringPoolNode instances.

Didn't do any real formal measurements for this so the actual reduction
in memory usage may be insignificant.  If nothing else the code is more
readable now, and manipulating reference counts is less awkward.

Updated Galician translations

Coding style and whitespace cleanup.

libebook: Reference address books by UID instead of URI.

Breaks EContact and EVCard APIs, but merely making the names accurate.

google: Remove use of ESourceExtensionSecurity

Google have required the use of SSL connections to Google Contacts for a
while, and libgdata has used SSL connections exclusively for even longer.
The ‘use_ssl’ option was completely ineffective and redundant.

google: Remove an unused variable

It was always the case that: live_mode == (priv->bookviews != NULL)

google: Add a new custom vCard property containing Google system groups

This can then be picked up by libfolks (and other interested clients) to
allow them to detect which system groups a Google contact is in. Previously,
they had to infer this from the categories the contact was in, which are
localised. This was tricky and fragile.

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

Deprecate EDbHash.

Apparently nothing uses it anymore.

ESource: Use a collation key for display name comparisons.

e_source_compare_by_display_name() gets called a lot.  We've been
calling g_utf8_collate() each time, which is known to be slow when
comparing large numbers of UTF-8 strings.

We now stash a collation key during e_source_set_display_name()
by passing the UTF-8 display name to g_utf8_collate_key().  Then
in e_source_compare_by_display_name() we can simply call strcmp()
on the internal collation keys to compare display names.

Didn't take any real performance measurements other than seeing
e_source_compare_by_display_name() topping the charts in sysprof.

Add missing linker flags for libebook-google-utils.

Updated Telugu Translation

Bug 678067 - Localized display names get stuck

ESource: Minor code cleanup.

camel-provider.h: Fix a typo.

CAMEL_PROVIDER_IS_STORE_AND_TRANSPORT: Handle NULL gracefully.

Evaluates to FALSE if the 'provider' pointer is NULL.

ESourceContacts: Only include local address books by default.

A more reasonable default than including all address books by default.
Weeds out large, remote address books that can take forever to query.

Bug 678618 - Runtime warning when clicking on top-level sidebar item

source_idle_changed_cb: Avoid a runtime warning.

Apparently it's possible for the 'changed' GSource to already be NULL
when we go to unref it in source_idle_changed_cb().  Not sure how that's
happening... possibly racing with dispose()?  In any case just check for
NULL first.

EServerSideSource: Don't write changes to disk until exported.

If a collection backend is configuring an EServerSideSource from a
worker thread, avoid writing changes to disk prematurely.

Explicitly calling e_source_write_sync() still works as advertised
whether the source is exported or not.

EServerSideSource: Add an "exported" property.

Read-only boolean property indicates whether the source has been
exported over D-Bus.  Basically just queries the registry server
for itself.

Updated Spanish translation

Fix build failures when $(top_srcdir) != $(top_builddir).

Really need to get distcheck working again...

Assamese translation reviewed

ESource: Read/Write int64/uint64

Improve imapx_command_fetch_message_done().

Using a corrupted database table as an opportunity to improve code
readability and error handling.

camel_data_cache_get: Improve empty file handling.

When an empty cache file is detected, set a GError with a descriptive
error message in addition to returning NULL.

e_book_backend_file_open: Fix directory existence test.

In e_book_backend_file_open() we seem to be bailing out if the data
directory does not exist, but yet we don't create the directory until
further down.

Not sure if this is correct, but it seems to me we only want to check
for the data directory if 'only_if_exists == TRUE'.

EDataBookFactory: Improve error message for GOA connection failure.

Fallout from a debugging session.

The printed error message was simply

   libedata-book-WARNING **: The given address is empty

which was not very helpful in tracking down the source.

camel_data_cache_get_filename: Remove unused GError parameter.

Don't want this parameter to result in unnecessary error handling.

imapx_command_fetch_message_done: Fix an error handling issue.

Bug 678378 - ESourceRegistry causes deadlock in gnome-shell

Avoid scheduling idle callbacks while populating the registry for the
first time during initialization, and therefore having to iterate the
main context to force those idle callbacks to be dispatched.

Instead call source_registry_add_source() directly, just during the
initial population.  This will emit "source-added" signals from the
object manager thread, but no one will be listening.

ESourceRegistry: Split ESource creation into a standalone function.

Split ESource creation out of the "object-added" signal handler to
a standalone function -- source_registry_new_source() -- which the
signal handler now calls.

ESourceRegistry: Use g_main_context_ref_thread_default().

Slightly more convenient than g_main_context_get_thread_default(), which
requires checking for NULL before referencing the main context ourselves.