Updated POTFILES.in

CamelIMAPXCommandFunc: Add a GError parameter.

Completion callbacks now get passed a GError and return a boolean.
In most cases calling camel_imapx_command_set_error_if_failed() is
sufficient.  The GError pointer passed to the callback may be NULL
if the server already knows an error occurred.

Add camel_imapx_command_set_error_if_failed().

If the CamelIMAPXCommand's status indicates failure, set the given
GError and return TRUE.  If no status is present (e.g. command not
complete) or the status is "OK", return FALSE.

This also drops the GError member from CamelIMAPXCommand, so that all
GErrors are now set on the command's CamelIMAPXJob.

CamelIMAPXCommand: Lose the GCancellable member.

Pass it around explicitly, or use the one in CamelIMAPXJob.

imapx_command_run_sync(): Take a GCancellable and GError explicitly.

CamelIMAPXCommand: Encapsulate some struct members.

Use a private CamelIMAPXRealCommand struct to keep some members hidden.
Add functions as necessary to interact with private struct members.

CamelIMAPXCommand: Use a GString.

The TODO comment was right: building the command part with a GString
instead of a CamelMemStream is much easier (and probably faster, not
that it matters).

imapx_command_start(): Take a GCancellable and GError explicitly.

camel_imapx_run(): Take a GCancellable and GError explicitly.

Split off CamelIMAPXCommand to a separate file.

camel-imapx-server.c is too large for anyone to fully grok.

CamelIMAPXServer: Coding style fix.

store_get_inbox_folder_sync: Add a NULL check.

Verify the get_folder_sync() method is implemented before invoking it.

CamelStoreSummary: Schedule save rather than save immediately

There was a regression from commit 5c52fe678f6d, marking multiple
messages read/unread invoked save on each single message change, which
had a) performance impact, b) did unnecessary disk writes (with fsync()
calls). This change fixes it by scheduling save, postponing it
by 5 seconds from the last connected CamelFolderSummary's count change.

Updated Norwegian bokmål translation

CamelIMAPXConnManager: Add a parting thought.

CamelIMAPXConnManager: Use a reader/writer lock.

Replace CamelIMAPXConnManager's recursive lock with a reader/writer
lock.  Read operations should be short and simple, write operations
may be long and complex.

Also check for cancellation in camel_imapx_conn_manager_get_connection()
after acquiring the writer lock but before attempting to requisition a
CamelIMAPXServer.

CamelIMAPXConnManager: Similarly for connection list operations.

Define a bunch of simple thread-safe operations for the list:

   imapx_conn_manager_list_info
   imapx_conn_manager_lookup_info
   imapx_conn_manager_remove_info

* Note: lookup_info() returns a new ConnectionInfo reference, and
        list_info() returns a new GList of ConnectionInfo structs,
        each with a new reference

Also rewrite imapx_find_connection() to make the logic clearer.
(Had to stare at that function for a long time to understand it.)

CamelIMAPXConnManager: Give ConnectionInfo its own mutex.

Define a bunch of simple thread-safe operations for it:

   connection_info_is_available
   connection_info_has_folder_name
   connection_info_insert_folder_name
   connection_info_remove_folder_name
   connection_info_dup_selected_folder
   connection_info_set_selected_folder

CamelIMAPXConnManager: Give ConnectionInfo a reference count.

Add lifecycle functions for it:

   ConnectionInfo *  connection_info_new    (CamelIMAPXServer *is)
   ConnectionInfo *  connection_info_ref    (ConnectionInfo *cinfo)
   void              connection_info_unref  (ConnectionInfo *cinfo)

Now we can perform list operations without holding CON_LOCK for so long.

CamelIMAPXConnManager: Miscellaneous cleanups.

Always name CamelIMAPXServer variables 'is', to be consistent with
CamelIMAPXServer itself.

CamelIMAPXJob: Clarify 'with_operation_msg' field.

Rename it 'pop_operation_msg' and add a comment.

Remove imapx_job_can_operation_msg().

Not needed.  camel_operation_push_message() already gracefully handles
'cancellable' being either NULL or a plain GCancellable.

CamelIMAPXServer: Use g_list_free_full().

Use a consistent variable name for CamelIMAPXServer.

imapx: Simplify parse_contents().

imapx_step: Return a success boolean.

imapx_continuation: Return a success boolean.

imapx_untagged: Return a success boolean.

imapx_completion: Return a success boolean.

Updated Spanish translation

Updated Slovenian translation

CamelSession: Avoid spurious "notify::online" signals.

imapx_refresh_finfo: Push a status message.

Updated Spanish translation

CamelVeeFolder: Process folder changes more efficiently.

CamelVeeFolder submits a new background job every time it receives a
CamelFolder::changed signal.  So the background jobs tend to pile up,
and there's no guarantee the jobs will be dispatched in the order
they were submitted since they're dispatched from a thread pool.

Instead, run one background job at a time, and have the job process
FolderChangedData from a queue until the queue is empty.  If another
CamelFolder::changed signal is received while the background job is
still running, the changes are simply be appended to the queue.

CamelVeeFolder: Push a status message while updating.

Bug #666764 - SQL error migrating folders

Camel: Clarify ownership of returned CamelProviders.

CamelStoreSummary: Save count changes from CamelFolderSummary immediately

Post-release version bump.

NEWS update for 3.3.4 release.

Updated Galician translations

Bug 667388 — Introspection annotation fixes for nullable return types

Add some missing (allow-none) annotations, as well as some (transfer full)
annotations, to libebook and libedataserver. Update the documentation to
match, and to be a little more descriptive in places.

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

Bug 667698: vapigen: fix builddir != srcdir issue

Updated Belarusian translation.

Bug #644817 - Slow addressbook cache

Updated Norwegian bokmål translation

Updated Spanish translation

Bug #638810 - Crash in camel_folder_search_set_folder with IMAP account

imap_folder_dispose(): Disconnect the summary first.

Disconnect the folder summary before destroying any internal members,
since disconnecting may take awhile to run.

CamelFolderSearch: Add argument guards to set() functions.

Bug #666757 - [CalDAV] Manage authentication requests even after open

Bug #665036 - Memory leaks spot in Contacts view

Camel: Use thread-safe accessors for string settings.

Camel: Add thread-safe accessors for string settings

Bug #661743 - Return maildir Inbox sibling folders only when requested

Updated Japanese translation

Bug 666770 - Add gmodule-2.0 to libedataserver.pc

Bug #592906 - Birthdays from WebDAV contacts do not show up in calendar

Updated Spanish translation

Bug 662888 - Use --include-uninstalled for internal .gir files

Otherwise we end up with a dependency on an installed pkg-config file.

imapx_run_job(): Initialize cancel_id.

Bug #482171 - IMAP warnings are presented without folder context

Bug #357175 - IMAP alerts still too intrusive

Updated Norwegian bokmål translation

Bug #666327 - [IMAP] Missing unlock in camel_imap_folder_fetch_data

Add misssing service unlock in the case of fast switch between folders
with non cached messages. Missed from previous commit:
74fcab535c0f50a27742c05e94036b8370ea9173 .

...And another iconv flush fix

Fixed another iconv() flush case.

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

Grow the output buffer as needed when flushing iconv

Updated Spanish translation

post release version bump

News update for 3.3.3 release

Bug #331361 - Allow GPG decryption with anonymous recipient set

Updated Hungarian translation

Bug #665253 - "New folder" dialog does not show IMAP accounts

Bug #662068 - Crash in e-cal-backend-http.c:retrieval_done

Fix leaks around categories editing

e_categories_selector_get_checked: Return them always in the same order

Bug #666106 - libical 0.47 handles CATEGORIES as multiple parameters

Revert patch for bug #514041

See the bug for more information:
https://bugzilla.gnome.org/show_bug.cgi?id=514041#c9

Contacts calendar - consider also other than only file-as field

Bug #652173 - EContact's file-as getter doesn't populate empty value

Add CamelSession.remove_service() method.

So subclasses can be notified when a service is removed.

Changes camel_session_remove_service() slightly.  Now takes a
CamelService instead of a UID, and does not return a gboolean.

Minor ABI + API break; soname bumped.

Updated Slovenian translation

Fix glib-gen.mak rules.

The %-enumtypes.h pattern causes problems when an "*-enumtypes.h"
from elsewhere (e.g. Camel) is found in the preprocessor search path.

Instead specify the output filename explicitly through a new variable:
glib_enum_output.  (e.g. glib_enum_output=e-data-book-enumtypes)

Call gettext on the short descriptions of port numbers.

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

Updated Spanish translation

Add --wait-for-client factory processes command line option

This is much better than --keep-running for cases when debugger
needs to be sure that everything opened is freed (the factory
closes itself automatically), while client's run takes longer
than 10 seconds (which is easy when running both/all under
valgrind, especially with evolution as the client).

Fix a bug in camel

imapx_command_run_sync(): Set a GError if cancelled.

Getting segfaults on cancellation without this...

Small correction to imapx_command_run_sync().

CamelIMAPXCommand: Make imapx_command_run_sync() more responsive to cancellations.

Connect a GCancellable::cancelled signal handler that broadcasts the
GCond that imapx_command_run_sync() is waiting on so it can proceed
immediately.

If imapx_command_complete() is called sometime later by the thread
executing the command, the GCond will broadcast again but no one will
be listening for it.

CamelIMAPXCommand: Reference the command when running synchronously.

Reference is dropped in imapx_command_complete().

CamelIMAPXCommand: Remove imapx_command_free() with imapx_command_unref().

CamelIMAPXCommand: Add imapx_command_ref().

CamelIMAPXCommand: Make IMAPX command functions static.

CamelIMAPXCommand: Add a reference count.

CamelIMAPXCommand: Allocate with g_slice_new().

Slab allocator is more efficient for fixed-size structs.

CamelIMAPXJob: Make imapx_job_run() more responsive to cancellations.

Connect a GCancellable::cancelled signal handler that broadcasts the
GCond that imapx_job_run() is waiting on so it can proceed immediately.

If imapx_job_done() is called sometime later by the thread executing the
job, the GCond will broadcast again but no one will be listening for it.

Giving CamelIMAPXJob a reference count allows the imapx_job_run() caller
to drop its job reference and proceed without worrying about whether the
job is still active in the parser thread, since the parser thread will
have its own job reference.