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.

Use color buttons' macro also here

Add support for the iso-8859-8-i encoding

Support this encoding by aliasing it to iso-8859-8.

Make GtkCMCList/Tree use "text", not "fg" rc property for text color.

Closes bug #3880.

Cast to GSourceFunc instead of deprecated GtkFunction when using g_timeout_add().

Also correct return type of fancy_show_mimepart_prepare()
callback function.

Update few occurrences to the new alertpanel() API (and by the way,
solve a leaking button_label strinc allocation).

Migrate from deprecated GtkTooltips to GtkTooltip in clamd plugin.

Fix few forgotten GDK_ to GDK_KEY_ renames in plugins.

Forgotten changes in commit 09301b6e73e4af1b03b6f19d340dd909cf527bcc

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

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

Do not put enchant's CFLAGS into main CFLAGS make variable.

Handle its CFLAGS just like all other libraries'.

Rework of alertpanel default button focus handling.

Instead of using "+" prefix with button labels and/or
the default_value parameter, we introduce "focus"
parameter, which will point to first, second or third
button.

Advantages:
- we got rid of the "+" prefix, which won't work with GTK3
  with GTK3
- instead of having two ways to indicate which button
  should have focus, which can be in conflict, and
  are not very straightforward, there is just one,
  arguably more intuitive
- some small memory leaks fixed along the way
  (using g_strconcat() directly as function argument)

Fix ‘GtkWidget {aka struct _GtkWidget}’ has no member named ‘style’

Thanks to Abhay S. Kushwaha for reporting!

Fix ‘GtkLabel {aka struct _GtkLabel}’ has no member named ‘label’

Force -DGTK_DISABLE_SINGLE_INCLUDES -DGSEAL_ENABLE in GTK_CFLAGS.

Remove direct GTK struct member access from pdf_viewer plugin.

Remove the last offender against -DGTK_DISABLE_SINGLE_INCLUDES

Get rid of final direct GTK struct member access from ClawsSpellEntry.

We have to track preedit_length internally, because this
bit of GtkEntry is not available anymore.

ClawsSpellEntry does not need to implement GtkEditable

We're "subclassing" GtkEntry, which already implements it.

Replace gtk_style_unref()

gtk_style_unref() is a remnant from gtk+1 and the compatibility function
in gtk+2 is gone in gtk+3, use g_object_unref() instead.

Signed-off-by: Ralph Sennhauser <ralph.sennhauser@gmail.com>

Add missing header to acpi_notifier

When building against gtk+3 the compiler complains about implicit
function declaration, so include the required header for using the
gtk_cmoption_menu.

Signed-off-by: Ralph Sennhauser <ralph.sennhauser@gmail.com>

Drop no longer used gtk+2 utility functions

With requiring gtk+2 >=2.24 the following functions are obsolete:
  gtkut_widget_set_mapped
  gtkut_widget_set_realized
  gtkut_widget_set_can_default
  gtkut_widget_set_receives_default
  gtkut_widget_set_can_focus
  gtkut_widget_set_has_window

Replace all occurrences with the underlying function.

Signed-off-by: Ralph Sennhauser <ralph.sennhauser@gmail.com>

Drop ifdefs for old gtk+2

Signed-off-by: Ralph Sennhauser <ralph.sennhauser@gmail.com>

Require gtk+ >= 2.24

gtk+-2.24.0 was released 2011-01-30 and has seen over 30 maintenance
releases since. Drop support for gtk+2 < 2.24 to ease porting to gtk+3
and to cleanup existing compatibility code for old gtk+2 releases.

Signed-off-by: Ralph Sennhauser <ralph.sennhauser@gmail.com>

A few instances of combobox manipulation still used the old API.

Patch by Ralph Sennhauser.

Removed almost all direct GTK struct member accesses.

Use accessor functions instead.

The few remaining offenders are in spell_entry.c, which
will require some rework.

Replaced some deprecated GTK+ API calls in gtkaspell.c.

Fix some copyright headers

Refactor colors handling with more arrays

This also makes possible to add more colors easily.

Move all colors into an array

RSSyl: Handle 404 and other fetch failures better.

Move syntax help after entry

Remove more unnecessary colons in labels

Use spacing more uniformly on themes prefs

Also remove colons from labels (not used in other places), and align
them to the right, to be visually consistent with other preference
pages.

Fix wrong article in translated string

Do not consider missing passwordstorerc file as error.

Just continue with empty password store.

Prune more gtk_widget_set_size_request calls

Handle password store config_version update without the global gint variable.

Handle failed/aborted folderlist config_version update in main.c