Merge branch 'master' into next

minor sup-config output tweaks

sup-config whitespace tweaks

bugfix in yaml serialization

Objects that don't respond to before_marshal were being turned into false.
Whoops!

fail fast if config is not valid

Merge branch 'master' into next

bugfix: crypto return values when no gpg binary detected

Tweak the return values of CryptoManager#decrypt so that the notice widget is
the first thing returned regardless. Makes the code cleaner.

Merge branch 'preemptive-loading' into next

make load-more callbacks use a queue and be thread-safe

Replaced previous insane implementation with something that actually
makes sense.

ui responsiveness tweaks

preemptively load messages when scrolling

Merge branch 'alignment-tweaks' into next

rejigger alignment of messages when using 'n' and 'p'

minimize horizontal movement. 'z' can be used to force alignment of
current message with left column.

move open3 require to correct file

Merge branch 'master' into next

sup-sync: restore state on messages that don't already exist

Revert "Merge branch 'after-add-message-hook' into next"

This reverts commit 67214e30d648ac9351b9898f8e1587d065264787.

This hook was causing sup-sync to fail on large mailboxes,
because it was keeping every message around in memory.

Merge branch 'xapian-updates' into next

remove use of Object#tap

Merge branch 'logging' into next

make SUP_LOG_LEVEL self-documenting

The idea here is that if someone is looking at the log and not seeing
the information of interest, then the log itself should tell them
how to get more information, (by suggesting to set SUP_LOG_LEVEL
to the next lower level).

Merge branch 'master' into next

add UTF-8 encoding string for ArchLinux systems

in ArchLinux, UTF-8 encoding is identified by "utf8" instead of
"UTF-8".

Merge branch 'reply-all-keybindings' into next

reply all keybindings

Merge branch 'logging' into next

Conflicts:
lib/sup/hook.rb

minor: move hook load messages from info to debug

Merge branch 'hook-local-vars' into next

bugfix: clear cached lambda hook locals after hook call

Merge branch 'master' into next

remove obsolete Manifest.txt

Merge branch 'run-mailcap-fixes'

Conflicts:
lib/sup/message-chunks.rb

Merge branch 'buffer-rolling'

Merge branch 'locking-refactor'

Conflicts:
bin/sup
bin/sup-sync-back
bin/sup-tweak-labels
lib/sup.rb
lib/sup/suicide.rb

Merge branch 'ncurses-fixes'

Conflicts:
lib/sup/buffer.rb

Merge branch 'various-api-refactors'

Merge branch 'logging'

Merge branch 'xapian-updates' into next

Merge branch 'after-add-message-hook' into next

Conflicts:

lib/sup/poll.rb

add Message#load_from_index! shortcut

Message#parse_header is slow, taking up to 2/3 of the time spent loading
threads in thread-index-mode. This patch adds a shortcut method that the index
can use to efficiently initialize a message.

add an after-add-message hook

I want to do some unrelated processing on each message I receive, but I
don't want to block the message being added to the index.  This patch
adds a hook which runs /after/ the message is added to the index.

Merge branch 'hook-local-vars' into next

fix proc locals, which must use the old method trick

For hook locals that are specified as procs (usually for performance
reasons), use methods instead of local variables. Sadly you don't get
all the regular variable semantics, but it is the only option for
delayed evaluation AFAICT.

convert a couple of arrays to sets for labels

This fixes some crashes when using 'l' in thread-view-mode that
have been present since commit 7aea418a8a62b7070eee764475fcfc0bdd8d58dd
("maintain labels as Sets rather than arrays").

Merge branch 'hook-local-vars' into next

Conflicts:
lib/sup/hook.rb

rewrite hookmanager to use eval for locals

This should fix all the issues that were cropping up when locals were faked
with method calls (the setter issue and the nil issue).

Merge branch 'master' into next

fix garbaged text in textfield when using ncursesw

Apparently, field_buffer content is not initialized to blanks when using
the wide-character version of ncurses. Forcing a call to
set_field_buffer fix the problem.

Merge branch 'master' into next

fix a thread merging bug introduced by refactoring in 59f8fc2

Signed-off-by: Alex Vandiver <alex@chmrr.net>

Merge branch 'logging' into next

switch non-error imap messages to info from debug

minor logging output tweak

Merge branch 'xapian-updates' into next

xapian index format versioning

move all GDBM data into Xapian

Keeping everything in Xapian means much better consistency in case of a crash.
Thread killing is now supported.

Merge branch 'various-api-refactors' into next

minor misc code tweaks

have mbox, maildir and imap sources (de)serialize labels nicely

At output time, rather than emitting a YAML Set (which is nasty), emit an array
of strings. At input time, normalize into a Set of labels.

add hooks for before- and after- yaml marshalling

Merge branch 'console-mode' into next

Conflicts:
lib/sup/modes/console-mode.rb

add intro help text and 'e' command to console mode

'e' restarts evaluation once you've stopped it with ctrl-g.

Merge branch 'console-mode' into next

Conflicts:
bin/sup

console: clear_hooks

console: reload

console: index internals accessor

console: add/remove labels

console mode

Merge branch 'master' into next

add 'I' keybinding to raise Inbox buffer

Merge branch 'buffer-rolling' into next

skip system buffers when rolling

Merge branch 'master' into next

maildir: allow ', ' in the unique-name portion of a maildir filename

The maildir specification says the following about unique names:

    A unique name can be anything that doesn't contain a colon (or
    slash) and doesn't start with a dot.

    [http://cr.yp.to/proto/maildir.html]

So disallowing a unique name to have a comma breaks maildir import on
systems where there is a comma in the names.

Merge branch 'ncurses-fixes' into next

ncurses cursor pops on after refresh. disable that.

Yet more shotgun programming.

Merge branch 'logging' into next

bugfix:relay event when new message is added to index

made this fancy method, and then i forgot to use it!

clean up some minor conflict mis-resolutions

Merge branch 'logging' into next

Conflicts:
bin/sup
bin/sup-sync-back
bin/sup-tweak-labels
lib/sup.rb
lib/sup/buffer.rb
lib/sup/message-chunks.rb
lib/sup/suicide.rb

rewrite logging to have multiple levels: debug, info, etc.

Level is now set by SUP_LOG_LEVEL environment variable.

Big patch here, but most of it is replacing the old Redwood::log calls.

Some work on poll-mode, which subclasses log-mode. Remind me never to use
inheritance again.

rewrite Singleton to not require i_am_the_instance

The flip side is that you have to use .init instead of .new.

minor code style tweaks and whitespace removals

refactor index access into three methods and rewrite PollManager#each_message_from

Couple big changes in this commit, but they're all tied together.

Index.sync_message is refactored into three separate methods: add_message,
update_message and update_message_state. The intention is that add_message is
called for new messages only, update_message is called for changing the method
body on disk (e.g. when we see multiple copies of the same message, or by
DraftManager when the text of a draft is changes), and update_message_state is
called when the labels on a message change. So indexes that differentiate those
operations can exhibit more natural performance characteristics.

Also, PollManager.add_messages_from has been renamed to each_message_from and
changed significantly. It now *only* yields successive messages; it does not
load the index version of the message, and it does not auto-add the message to
the index.  (In fact, it ignores the result of the block.) There's also a new
method called add_new_message that calls Index.add_message and then relays the
update to other GUI elements.

There was a lot of refactoring of sup-sync that was part of this. Probably
not strictly necessary but it was too hard to untangle the changes.

maintain labels as Sets rather than arrays

Merge branch 'ncurses-fixes' into next

restore keypad mode after we force ncurses to refresh the whole screen

This also happen to fix a regression after a call to run-mailcap, since
for some reason a screen resize event is triggered when we get control
back...

Merge branch 'master' into next

when writing mbox files, output From_ line date as utc

Otherwise, parsing the From_ lines later will fail!

Merge branch 'run-mailcap-fixes' into next

use BufferManager.shell_out to call run-mailcap instead of system

This ensure ncurses state is correctly restored upon command completion.

don't redirect stderr to /dev/null when calling run-mailcap

Some programs will fail to work correctly if this is done, like for
example: w3m --dump, to view HTML files.

Merge branch 'master' into next

move xapian index loading into load_index

Merge branch 'master' into next

allow sup-sync to use --start-at with --restored

Merge branch 'locking-refactor' into next

Merge branch 'ncurses-fixes' into next

refactor index locking interaction and replace suicidemanager

Since SIGTERM now works, we have a better way of killing a remote
process than SuicideManager. So replace that with a $die global.
Also refactor the interactive locking code into a separate module
so as not to pollute index.rb with such trivial concerns.

die gracefully upon SIGKILL