Fix crash in quicksearch keypress handling.

The fourth argument to g_signal_emit_by_name() in this case
is not the signal user data, but a pointer where to store the
return value.

Force initialization of buffer (fixes a valgrind case where buf is
used uninitialized).

Fix a memory leak in quicksearch code.

Fix wrong type for layout_mode var in struct.

Make Claws Mail conservative about color labels: ask for confirmation
when user clears or override existing color labels to messages.
Confirmation dialog is discardable.

Update French translation.

Fix warning: ‘struct sockaddr’ declared inside…

By including our utils.h instead of spamassassin homonymous file.

In file included from ../../../src/procmsg.h:32:0,
                 from spamassassin.c:42:
./utils.h:124:47: warning: ‘struct sockaddr’ declared inside parameter list will not be visible outside of this definition or declaration
 int timeout_connect (int sockfd, const struct sockaddr *serv_addr, size_t addrlen);
                                               ^~~~~~~~

Save and reuse Actions input/output dialog size.

Fix a couple of unused variable warnings

- libspamc.c:491:19: warning: variable ‘family’ set but not used
  (used in a #ifdef block, fix by surrounding by same #ifdef)
- libspamc.c:1911:35: warning: unused variable ‘addrp’
  (used in a commented code block, fix by commenting also declaration)

Really ask for confirmation when marking *all* messages as read or
unread from the message list (as it's done from the folder context
menu). This confirmation dialog is still discardable and still
obeys to the option "Confirm before marking all messages in a folder
as read or unread" in preferences.
A bit of code factorization.

Fix French translation (unexpected accelerator used in dialog title).

Save few warnings around mistyped uses of g_list_length().

Correct the label of the 'confirm before marking all messages in a
folder as read' option: it's about marking all as read or UNread.

Use a proper CharSet enum value for uninitialized state of static vars,
reindent tables for increasing readability.

One more fix for GENERIC_UMPC: summary menu entries list differ.

Fix compilation using GENERIC_UMPC.

Fix code compilation when configure --enable-generic-umpc code has been used.

Don't manage account list focus state when closing an account edit window,
when this window has been called from main window's "Preferences for
current account" menu item.
Closes bug #4022: Closing "Account Preferences" window opens "Edit Accounts"
window if "Edit Accounts" window has been opened before at least once.

Fix memory handling around SieveEditorPage.

Fix some plugins' code indentation.

Reindent lists and arrays.

Ignore the expander "hot spots" in folderview for childless folders.

Fix sensitivity of the main window's 'View/Message part/View as text' entry
according to the selected mime part.

Fix array type to match items

Fixes warnings:
• …from log.c:25: /usr/include/glib-2.0/glib/gmacros.h:284:14: warning:
  initialization makes pointer from integer without a cast
• log.c:189:29: warning: return makes integer from pointer without a cast

Don't add "Go to last error" to filtering log window's context menu.

Clean-up in comments (The Commit Of The Year).

Fix two missed return values in compose_write_to_file() from commit e9f610a4f.

Thanks to wwp for finding the issue!

Fix unwanted IMAP connections on startup caused by processing rules.

The culprit was a combination of folder_item_prefs_clear()
setting item->enable_processing to TRUE even though default
value should be FALSE, and initial_processing() not skipping
over root folders, which do not have this variable later set
to correct default value in xml_to_folder_item().

Closes bug #3993: Claws Mail connects to IMAP server when it should not

Fix for e9f610a4f for the rare cases the privacy system has no error string ready.

Rework and simplify how compose_queue() return values are handled

Also closes bug #3960 - Sends unencrypted emails when encryption fails
because we now return from compose_queue() with an error when
that happens, instead of ignoring it.

ClamAV plugin: manage read errors (fixes CID #1220483).

Fix another leak in smime plugin (CID #1220437).

Spamassassin plug-in: add support for compression (server must have
compression enabled, local spamc too).

Disable SSLv3 in libspamc.c (patch from Debian spamassassin package).

Fix sensitivity of few preferences widgets of the SA plug-in.

Add NOTICE file to EXTRA_DIST.

Add SpamAssassin's NOTICE file.

SA plugin: better way to make sure Unix sockets are not used in Windows,
from a config coming from Unix: do it when loading the configs, not
when handling the plug-in preferences in GUI.

fix bug 3978, '"From" column displays both name and email address for Outbox'

Merge branch 'master' of ssh+git://git.claws-mail.org/home/git/claws

Use G_OS_WIN32, the glib-proper way to check for WIN32.

Spamassassin plugin in Windows: don't allow Unix socket transport
mode (and switch to TCP if the config says Unix socket).

fix display of Key ID

Updated Apache SpamAssassin code (libspam.[ch], utils.[ch]) from 3.0.x
to 3.4.1.

Fix a leak, CID #1220437.

Fix indentation, silents up CID #1434201.

Fix wrong use of pointer-to-array as an array, CID #1434191.

Fix buffer overrun, always writing at buffer size + 1.
Fixes CID #1434188.

Fix CID #1434197, mismatch and unneeded display of unavailable folder class
in warning.

Add missing src/addrmerge.h file from 3.16.0-107-gf25722a.

Check for enchant-2 (necessary since enchant 2.1.3).

Merge branch 'master' of ssh+git://git.claws-mail.org/home/git/claws

Add a basic contact merging feature to the address book, thanks to
Charles Lehner <cel@celehner.com>.
Closes bug 3346: Contact merging

Merge branch 'master' of ssh+git://git.claws-mail.org/home/git/claws

addritem_person_get_picture() now returns something useful: a copy
of the addritem's picture full path (still to be free'd by the caller),
instead of a copy of the addritem's UID string.

addritem+person_get_picture now returns something useful: a copy
of the addritem's picture full path (still to be free'd by the caller),
instead of a copy of the addritem's UID string.

Remove the empty progress bar that may remain in status bar after
pop3 incorporation (if account is configured to leave messages
on server).
Closes bug 3985: an empty progress bar remains after POP mail check completes

replace deprecated 'NONE' token

Fix broken templates attachments due to commit 814b087,
which clears template internals (including attachments list)
wherever template contains any field to apply (To/Subject/etc.)
and before attachments could be processed. Simply process
attachments before attachments list may be cleared by fields
processing. Ideally we should change quote_fmt_reset_vartable()
or its uses in compose.c at least.

Fix broken build using GTK3

Signed-off-by: Michael Rasmussen <mir@datanom.net>

Change how main window's keyboard accelerators are bypassed when quicksearch has focus.

Instead of artificially forwarding the keypress to quicksearch
(only when Shift is pressed), we emit a new "key-press-event"
signal directly to the quicksearch entry, and block the original
signal before it reaches the accelerator handling.

Closes bug #3984: Copy-paste in find/filter field works incorrectly

Attachment remover: notify the user about what has been done when processing
multiple selections.

Adapt identical behaviour to native address book. See commit 47e0437b8b

Signed-off-by: Michael Rasmussen <mir@datanom.net>

Fix broken build from commit 47e0437b8b

Signed-off-by: Michael Rasmussen <mir@datanom.net>

Remove key and free attribute value only if they are valid,
clears CID 1220202.

Fixed hook_id declarations to be gulong instead of guint.
Made statusbar's hook_id static.
Rework hook_id magic value, now 0 instead of -1, and use a define (HOOK_NONE)
everywhere instead of 0. Any hook_id must be initialized to HOOK_NONE.
Hook_id that get invalidated or reset get the same HOOK_NONE value.

Don't compute charset unless necessary (vcalviewer_get_uid_from_mimeinfo()),
don't compute charset at all when not used (vcalviewer_get_request_values(),
vcalviewer_get_reply_values(), de facto solves CID 1372373), possible remnants
of an old implementation that I couldn't even found in log.

Fix possible access to uninitilized folder pointer (CID 1402515),
add a warning if vcal folder's not found.

Save us 35 warnings about hooks_register_hook() return checks.

Few optimizations: avoid strlen when string length is known (read()>0 return);
end string at real end of string instead of at buffer boundary and
be sure we print 0-terminated string (clamd_stream_scan());
clamd_verify_email: discover a strange multi-line response, be sure we
print 0-terminated string (CID 1220483), debug_print all response lines.

Fix wrong malloc of clamd_socket struct, CID 1220477.

Handle IMAP search failure instead of ignoring it.

Error out on protocol error during search, disconnect
the IMAP session, and display an error dialog about it.

Closes bug 3986 - IMAP quick search using non-ASCII
characters creates an infinite loop

Clear statusbar's progressbar in case of NNTP xhdr/xover failure.

Apply the attribute list fix from 8017810c4 also when handling LDAP contacts.

Fix two crashes caused by bad GtkListStore management in editaddress.c.

Fix use of uninitialized variable in addressbook edit person dialog.

Make LDAP use log_print instead of log_message for uninteresting stuff.

This makes it less noisy on the console without --debug if
everything works OK.

Replace GtkSCTree in addressbook_foldersel.c with GtkTreeView.

Replace both GtkCMCLists in Edit Person dialog With GtkTreeView.

One for email list, one for attribute list.

Fix potential crashes in gtkut_tree_view_get_selected_pointer().

Remove unused struct member in SummaryView

In the end it wasn't a very important color… ;-)

Fix a dumb double-free on Windows.

Closes bug 3957: Claws-Mail 64bit crashes when saving a draft

Fix license version in Address Keeper plugin

Update uses of gtkut_tree_view_get_selected_pointer().

Make the gtkut_tree_view_get_selected_pointer() more versatile.

It can now be used also for non-trivial use cases, where
the caller needs to further work with the model and/or
selection, or if a string value needs to be read from the
model.

Add gtkut_tree_view_get_selected_pointer() helper function.

It's meant for the simple cases when you just want to get
a pointer out of a treeview's selected row and you're not
going to do anything with the view, nor the model.

Fix summary_select_all when there is nothing selected.

Due to how GtkCMCList works, we first need to focus and
select a row.
Fixes bug #3973, ''select all' in summaryview does not
automatically focus the summaryview'

Revert "fix bug 3973, ''select all' in summaryview does not automatically focus the summaryview'"

This reverts commit a3e48bd6e7e62dedfef8ddedbeeb90400b5234b3.
That commit introduced slowness when selecting all rows.

Replace GtkCMCList in browseldap.c with GtkTreeView.

fix bug 3973, ''select all' in summaryview does not automatically focus the summaryview'

The problem is apparent when 'default selection on entering a folder' is
set to 'none' and the first user action on entering the folder is
'select all'. This visually selected all the messages but summaryview
did not have the focus, so any actions performed on that list would
fail.

Removed summary_select_all(), which didn't quite do as advertised and
instead use gtk_cmclist_select_all() which selects and focuses.

Merge needless three gtk_tree_model_get() calls into one.

Fix memory leaks from results of gtk_tree_model_get().

G_TYPE_STRING values need to be freed, G_TYPE_PIXBUF
(or anything else derived from GObject) needs to be
unreferenced. This was forgotten in some places.

Unreference GtkTreeModel pointers after they've been added to a tree view.

This was forgotten in only a few cases.

Replace GtkCMCList in editldap_basedn.c with GtkTreeView.

Replace GtkCMCTree in addressadd.c with GtkTreeView.

RSSyl: fix deleted item checking when modified or published time is missing

Fixes bug #3971.

Replace GtkCMCList in pgpcore select-keys.c with GtkTreeView.

While we're at it, activating a row now finishes the
selection dialog without having to press the Select button.

Replace GtkCMCList in LDIF import dialog with GtkTreeView.

Replace GtkCMCList in addrgather dialog with GtkTreeView.

Replace GtkCMCList in folder sort dialog with GtkTreeView.