Avoid possible crash when url->path is NULL

Add a comment explaining CamelStore's mode field.

I was about to get rid of this field thinking it was unused, but as
usual it's a GroupWise-specific hack for non-writable proxy accounts.
Add a comment to the header file for future reference.

Remove g_thread_init() calls.

g_type_init() already calls it, and has for several GLib releases.

IMAP: Split off a new mutex for commands and responses.

Define a new public GStaticRecMutex named "command_and_response_lock"
on CamelImapStore for IMAP command and response operations to use.

Fixes some deadlocks I was encountering due to the overreliance on
CAMEL_SERVICE_REC_CONNECT_LOCK, particularly during authentication.

camel_imap_store_connected() cleanup.

If camel_service_connect_sync() set a GError, don't overwrite it with a
generic "you must be working online" message.  The connect_sync() error
message has useful information to help identify the real problem.

Updated Slovenian translation

Bug 662524 - POP3: Use a safer default value for "delete-after-days"

Default to not deleting mail kept on the POP3 server at all rather than
automatically deleting after X days.  Helps avoid accidental data loss.

Updated Spanish translation

Bug #660182 - Error dialog on LDAP query with multiple referrals

Bug 662643 - camel_url_decode_path() crashes on empty string

Move remaining CamelURL elements to CamelSettings.

This is a continuation of the CamelSettings work I started in 3.1.

CamelService no longer stores a CamelURL.  The URL components are being
moved to CamelSettings.

For network services, the host, port, user and authentication mechanism
URL components now live in CamelNetworkSettings.

For local services, the path URL component (aka, the file path to the
root of the local mail store) now lives in CamelLocalSettings.

For now these new settings are configured the same as other settings:
through camel_settings_load_from_url().  But again, that function will
be dropped as soon as we move to key files for account data storage.

Also:

- Remove camel_service_get_url(), which is no longer used.

- camel_session_add_service() now takes a protocol name instead of a
  URL string.  The protocol name must match a registered CamelProvider.

- camel_service_get_camel_url() is renamed camel_service_new_camel_url().
  It now returns a new CamelURL which must be freed with camel_url_free().
  The returned CamelURL is built from user, host, port and path settings,
  and does not have any parameters.  You MUST access settings through
  CamelSettings now.

- CamelServices need to be prepared for CamelSettings values changing or
  even the entire CamelSettings object being replaced at any time.  Most
  services were already prepared, but I caught a few services caching
  settings during initialization.  Always consult CamelSettings when a
  setting value is needed so the service is acting on a fresh value.

- Camel's SONAME is hereby bumped (yet again) for 3.3.2.

Call g_thread_init() conditionally

Post-release version bump.

NEWS update for 3.3.1 release.

Avoid a crash in camel_session_get_service_by_url().

Check that we found a CamelProvider before dereferencing it.

Add camel_folder_refresh_info()/_finish().

I forget to write wrapper functions for CamelFolderClass.refresh_info()
and CamelFolderClass.refresh_info_finish() even though the class methods
have been there for awhile.

Thanks to Srini for pointing this out.

Updated Slovenian translation

Added test to show that fields-of-interest is working in libecal

Test is based on test-client-get-view.c, this version of the test
sets fields-of-interest to UID, RECURRENCE-ID and LAST-MODIFIED
fields, tests that all those fields are returned and asserts that
the event summary (which is indeed set) is always NULL.

Updated Polish translation

Updated Polish translation

Updated Polish translation

Bug #660870 - Empty Contacts list when pressing "To:" in composer

Revert "Remove of -Werror-deprecated-function-declaration", it doesn't work

Bug #660656 - Fix e_iterator_get annotation

Updated Belarusian translation.

Remove -Werror-deprecated-function-declaration to make it build for the 3.4 moduleset

Bug #527614 - Valgrind shows a nice amount of lost memory

Updated Spanish translation

Updated Spanish translation

Fix compiler warnings from GOA code in book factory

Really implement e_cal_client_view_set_fields_of_interest().

As discussed on bug https://bugzilla.gnome.org/show_bug.cgi?id=652180,
this patch adds many '_component' variants to functions that are named
with '_object', all the '_component' variants take an 'icalcomponent *'
instead of an ical string.

The magic filtering happens in e_data_cal_view_get_component_string()
which creates an ical string representation while omitting properties
that are not mentioned in the fields of interest.

[l10n] Updated German translation

[l10n] Updated German translation

Avoid possible deadlock with CamelStoreSummary locks

Recently added store_summary_sync_folder_summary_count_cb() could
cause deadlock when locking SUMMARY_LOCK first, and then calling
functions which lock REF_LOCK before SUMMARY_LOCK. Locking REF_LOCK
first avoids this.

Bug #661549 - Avoid fetching from backend when calculating summary

Bug #656603 - Add support for generating vCard 2.1 in libebook

Updated Norwegian bokmål translation

Updated Galician translations

Updated Japanese translation

l10n: updated Italian translation

Update libedata-book API docs.

Update libebook API docs.

Update libedata-cal API docs.

Update libedataserver API docs.

Update Camel API docs.

Updated Korean translation

Updated British English translation

smtp: Adapt to Camel's new authentication API.

pop3: Adapt to Camel's new authentication API.

nntp: Adapt to Camel's new authentication API.

imapx: Adapt to Camel's new authentication API.

imap: Adapt to Camel's new authentication API.

Camel: New authentication API.

Currently each provider has to implement its own authentication loop.

The idea for these changes is add a CamelSession method that executes
a common authentication loop for all providers:

    CamelSessionClass:

    gboolean  (*authenticate_sync)   (CamelSession *session,
                                      CamelService *service,
                                      const gchar *mechanism,
                                      GCancellable *cancellable,
                                      GError **error);

    void      (*authenticate)        (CamelSession *session,
                                      CamelService *service,
                                      const gchar *mechanism,
                                      gint io_priority,
                                      GCancellable *cancellable,
                                      GAsyncReadyCallback callback,
                                      gpointer user_data);

    gboolean  (*authenticate_finish) (CamelSession *session,
                                      GAsyncResult *result,
                                      GError **error);

Each CamelService is then responsible for implementing a new
authenticate method that makes only ONE authentication attempt and
returns an appropriate status code:

    CamelServiceClass:

    CamelAuthenticationResult
              (*authenticate_sync)   (CamelService *service,
                                      const gchar *mechanism,
                                      GCancellable *cancellable,
                                      GError **error);

    void      (*authenticate)        (CamelService *service,
                                      const gchar *mechanism,
                                      gint io_priority,
                                      GCancellable *cancellable,
                                      GAsyncReadyCallback callback,
                                      gpointer user_data);

    CamelAuthenticationResult
              (*authenticate_finish) (CamelService *service,
                                      GAsyncResult *result,
                                      GError **error);

The status codes are defined by the CamelAuthenticationResult enum:

    CAMEL_AUTHENTICATION_SUCCESS    : auth credentials were accepted
    CAMEL_AUTHENTICATION_FAILURE    : auth credentials were rejected
    CAMEL_AUTHENTICATION_ERROR      : something went horribly wrong!

CAMEL_AUTHENTICATION_SUCCESS and CAMEL_AUTHENTICATION_ERROR both
terminate the authentication loop, but CAMEL_AUTHENTICATION_FAILURE
cycles the loop and asks the CamelService to retry authentication,
usually after prompting the user to enter a different password.

Updaged Galician translations

Require libgdata >= 0.10.

Drop all the backward-compatibility cruft.

Updated Spanish translation

Bug #661448 - e_util_copy_*_slist(): Avoid using append function

The current implementation calls g_slist_append() for each element
of the list to copy. However, g_slist_append() is expensive
because it iterate over the whole destination list.

This patch alters the behavior to first make a deep copy of the
list and then concatenate the destination list and the list copy.
The concatenation only iterates of the destination list once.

Updated Hungarian translation

EBookClient: Avoid useless copying of valid UTF-8 strings

Make use of e_util_ensure_gdbus_string() instead of
e_util_utf8_make_valid() whenever possible to avoid useless
copying/freeing of valid UTF-8 strings (likely case).

e-book-backend-file: Don't share DB_ENV between address books from different sources

A DB_ENV is associated to the folder when the database is stored, in
order to store transaction logs. As a consequence, a DB_ENV should
never be shared between address books from different sources, since
they are kept in separate folders.

ENameSelectorEntry::user_delete_text: Delete whole text when user deleted it

In cases like setting an empty string to a GtkEntry the event
passes -1 as end_pos, where ENameSelectorEntry didn't behave properly.

Bug #661432 - 'Any field contains' search doesn't work in contact view

Bug #660829 - Allow 'Mark as read' in right click menu on spam folder

Revert "gdbus_proxy_call_sync: stop using g_usleep()"

This reverts commit cf78ee827d3c5157e44a1e4c32d395942a816461.

EBookClientView: Use e_util_free_object_slist() utility function

e-book-client: Refactor bulk methods code

Add a contact_slist_to_utf8_vcard_array() utility function to
avoid code duplication in EBookClient bulk methods.

gdbus_proxy_call_sync: stop using g_usleep()

Instead, tell g_main_context_iteration() that it may block if no
event source is ready.

Don't free vcard memory in e_data_book_view_notify_update*_vcard()

Instead, the vcard memory is freed by the caller which feels more
consistent with the rest of the code and less error-prone.

e_book_backend_sqlitedb_new: Fix database connection reuse

Remove EBookBackendSqliteDB objects from the db_connections
hash table in their dispose function. It used to be done in
their finalize function but this would cause
g_hash_table_lookup() to sometime return EBookBackendSqliteDB
GObjects whose ref_count is 0, under heavy load.

This patch also makes sure the database mutex is properly unlocked
in case of error. It also properly reports errors when
book_backend_sqlitedb_load() fails.

Updated Telugu Translations

libebook: Add unit test for bulk modification function

e-book-client: Add bulk contact modification methods.

This reduces the number of DBus round-trips and enables
optimizations in the backend.
This patch also makes use of Berkeley DB transactions in the file
backend.

Workaround possible crash in imap_body_decode()

Updated Vietnamese translation

vi.po: import from Damned Lies

Updated Norwegian bokmål translation

Updated Swedish translation

Updated Spanish translation

configure.ac: Bump ADDRESS_BOOK_DBUS_SERVICE_NAME to 2

The DBus API has changed with the addition of bulk contact
addition methods.

addressbook: Improve bulk contact removal code

The patch makes use of Berkeley DB transations in the file backend
and roll back the transaction in case of error to make the
behavior consistent with bulk addition.

This patch also makes the webdav backend behave as expected since
it does not support bulk removal and the implementation was broken.

libebook: Add test for bulk methods

e-book-client: add e_book_client_add_contacts*() methods

Enables optimizations for batch additions of contacts such as
less DBus round trips and use of database transactions in file
backend.

build_sqlitedb(): remove useless free() for vcard_dbt.data

Updated POTFILES.in

build_sqlitedb: Fix memory leaks and optimize function

This patch fixes 2 possible memory leaks in build_sqlitedb()
function, makes use of e_util_free_object_slist() utility function,
slightly improves error checking and makes some minor optimizations.

Make all backends return NULL when calling get_contact() and the contact is not found.

Some backends returned NULL (google, webdav) while other returned
an empty string (ldap, file, vcf). The behavior should be
consistent and I believe the best is to return NULL since it is
an error case (NOT_FOUND error is returned), this avoids useless
string allocation and possible leaks.

gdbus_proxy_call_sync: check sync flag more frequently

This patch lowers the value used in g_usleep() from 250000us
to 1000us in order to decrease latency.

CamelFolderSummary API changes

Prefer g_simple_async_result_take_error().

Slightly more efficient and convenient than:

    g_simple_async_result_set_from_error (simple, error);
    g_error_free (error);

One less GError to copy and destroy.

Increase dependency on gtk+ to 3.2

Add missing arguments checks to CamelCipherContext

Bug #660884 - Missing libebackend LIBADD in calendar/backends

Provide EClient's GSList utility functions within e-data-server-util.h

Bug #660615 - Make POP3 provider more cancellable ready

Fix a little memory leak

Bug 660533 - Missing GOA cflags/libs in libedata-book

Bug #620382 - Do not delete also directory for custom local calendars

Bug #601898 - SOCKS proxy does not work with mailer

Add unit test for lazy contact vcard parsing

The test was mostly written by Milan Crha.

addressbook: Optimizations to avoid vcard parsing

Now that vcards are lazily parsed, it makes sense to optimize
the rest of the code to avoid vcard parsing as much as possible.

In particular, it is now possible the pass the vcard UID when
constructing an EVCard object by using e_vcard_construct_with_uid().
This will avoid useless vcard parsing whenever getting the vcard
UID later. Moreover, setting the UID attribute on a EContact no
longer requires vcard parsing.