Decouple the sidebar from the Buffy list.

Change the sidebar to use an array of SBENTRY* instead.  Move the
"is_hidden" into SBENTRY.  Remove the added "prev" pointer from BUFFY.

This way, sorting the sidebar doesn't affect the BUFFY list order, and
we don't need elements inside BUFFY just for the sidebar presentation.

Fix sidebar-next for the case where the mailboxes are unsorted and
$sidebar_new_mail_only is set.  Change sorting not to clump hidden
mailboxes at the bottom, instead simply skip over hidden mailboxes in
sidebar-next/prev.

Fix missing sidebar documentation links. (closes #3847)

USE_SIDEBAR needed to be set in doc/makedoc-defs.h so that the sidebar
option documentation is generated whether mutt is configured with the
sidebar or not.

Remove unused SORT_DESC.

This came over with the sidebar patch, but I believe is only used by
the notmuch extension.

Remove $sidebar_refresh_time from Sample Sidebar Config

The $sidebar_refresh_time option was removed with the changeset
1f840760e6e0. Remove it from the sample sidebar configuration as well,
so that anyone using that file in its entirety won't have to deal with "unknown
variable" errors.

Change sidebar to consistently use realpath for context and buffy comparison.

The original sidebar patch contained a half-implemented attempt to use
realpath() mailbox paths for comparison.  (Presumably so the open mailbox
remains highlighted despite symlink issues).

Add realpath to the Context, and set it when opening a mailbox.
Remove sidebar ifdef for the buffy member, and always set it there too.

Change the sidebar to use the realpath for comparison everywhere.

mutt_buffy_check() is using stat device and inode for comparison.
Perhaps this can be changed to use realpath instead, but that's beyond
the scope of this patch.

Change sidebar next/prev-new to look at buffy->new too.

Look at new in addition to msg_unread count, to account for when
$mail_check_stats is unset or when the sidebar only shows the %n
status flag.

Add documentation to the "New Mail Detection" section of the manual.

Mention the behavior change with $mail_check_recent.

Add a section about $mail_check_stats.

Combine the basic and extended buffy functions.

Add a check_stats parameter to the mbox, maildir, and mh buffy
functions.  Use that parameter to determine whether to also count
total, new, and flagged messages.

This makes the functions a bit more complicated, but improves
efficiency (for maildir and mh).

Also includes the following cleanup/fixes:
* Move the orig-value counter reset to the beginnining of the loop,
  (before tmp->new is set to 0).

* Change trashed maildir messages to not be counted in msg_count

* Remove an incorrect setting of mailbox->new based on msg_count in
  maildir.  (I missed this one for 1f840760e6e0)

* Change mbox to use the context->mtime for stats_last_checked,
  removing a race condition.

* Fix mh to actually count the messages in order to generate msg_count.
  mh_sequences only covers the range of messages with some sort of
  flag.

Make extended buffy independent of the sidebar.

Add new boolean option $mail_check_stats (default off) and
$mail_check_stats_interval.  The first turns extended buffy on.  The
second sets the amount of time in between extended buffy checks
(defaulting to 60 seconds).

Remove the option $sidebar_refresh_time.

Change mutt_buffy_check() to only notify the sidebar to redraw if a
mailbox buffy value changes.

Remove the #ifdefs around the extended buffy functions.  The next
patch will merge these functions with the basic functions and pass a
parameter instead.

Imap is a special case, because it sends out the status in one batch.
Change this to perform the comparisons inside cmd_parse_status() and
flag the sidebar there.  It was previously directly assigning the
status counters (unsigned int) to the buffy->new (short).  Change this
to assign 1/0.

Change the default for sidebar_format to use %n.

The next series of patches is going to create a new option,
$mail_check_stats defaulting off.  When off, the extended buffy which
calculates total/new/flagged messages won't be run.  To help reduce
"sidebar is broken" bug reports, this changes the default format to
something not requiring extended buffy.

The previous default is documented in the option and preserved in the
sample sidebar muttrc in contrib.

Corrected OPS.SIDEBAR filename in EXTRA_DIST.

Start to fix sidebar buffy modifications.

The extended buffy for mh had incorrect placement of the loop
brackets.  The counters weren't being incremented in the loop.

Fix extended buffy for maildir to count a maildir message as new if it
doesn't have the info delimeter.

Remove shortcircuits added to the basic buffy stating there is new
mail when (msg_unread > 0).  This is not necessarily true, depending
on $mail_check_recent.

Note: the extended buffy still needs more fixes, which will be done
when it is refactored into its own option.

Various fixes to the sidebar logic.

Use strfcpy instead of strncpy.  The current logic could write past
the end of the buffer.

Don't mess with BUFFY next pointers during removal.  The
mutt_parse_mailboxes() is fine, but this is still not something that
should be done inside sidebar.c.

On removal, set next->prev since we can.

Fix unmailboxes logic:
  * only fix the prev pointers once.
  * if we unmailbox the open mailbox, set it to NULL.

Lastly, flag a redraw on mailboxes/unmailboxes.

Clean up the sidebar manual.

Remove the introduction section.  It's a nice effort, but reads more
like a tutorial than something that belongs in section 2.  Will
probably put in on the website instead.

Remove references to a "patch", neomutt, and the history.

Remove the sample muttrc from the manual.

Fix validation errors for the documentation.

Fix sidebar check_sec.sh warnings.

Use safe_malloc, FREE, and the safe_strcat functions.

Sidebar clean up: building and drawing logic.

Fix the autoconf/makefile.am changes to be consistent.

Create a global SidebarNeedsRedraw to indicate a redraw is needed,
instead of putting sb_draw() everywhere in the code.

Create a menu_redraw_sidebar() function and use the REDRAW_SIDEBAR
flag instead of piggy-backing it inside the index loop.

Fix curs_main.c and pager.c to be a bit cleaner by using the global and
REDRAW_SIDEBAR.

Start to clean up some of the buffy code, but this needs to refactored
and fixed.

fix sidebar/mail_check painting problem

The bug was triggered when $mail_check was set to 0.

When drawing the status line, the Sidebar lied about its size to
workaround a bug in mutt_FormatString().  Whilst in that function the
Sidebar was redrawn.

The Sidebar got confused over whether it should be visible or not and
promptly vanished.

travis: new build script

make the cursor resetting work in slang

fixes: neomutt/homebrew-neomutt#13

Fix the case of messages in OPS.MIX and OPS.SIDEBAR

Thanks to Vincent Lefèvre for spotting the discrepancy.

Add sidebar_format flag '%n' to display 'N' on new mail.

fix cursor position after sidebar redraw

If the user is entering data, e.g. mail subject, and a sidebar redraw
interrupts them, put the cursor back where it belongs.

Fixes: #49

Fix sidebar buffy stats updating on mailbox close.

Move the mutt_sb_set_buffystats() call from mx_fastclose_mailbox() to
the bottom of mx_close_mailbox().  Append-only mailboxes don't have
msgcount set, so fastclose was the wrong place to be doing these
updates.

Fix alignment in the compose menu.

Several of the compose-panel labels didn't use the TITLE_FMT to align
themselves.  This causes formatting problems when the width is changed,
e.g. by the NNTP patch.

temporarily disable $sidebar_refresh_time

Limiting the refresh frequency of sidebar updates was interfering with
other Mutt behaviour.  Upstream, Mutt is changing how refresh is
handled.  I'll adopt those changes when they're finished.

The option will be silently ignored, for now.

fix reflow properly

The SidebarWidth was subtracted *after* the width calculations.
This meant the pager text was too narrow.

fixes #26

Setting $sidebar_width to more than 128 would cause bad things to happen.

First, give the users more than enough space.
Second, don't pad out short strings beyond the buffer

---
 sidebar.c | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

bug-fixes from mutt/default

rename sidebar functions sb_* -> mutt_sb_*

sync whitespace to mutt/default

fix missing TITLE_FMTs

doc: check for installed patches

doc: css for sidebar intro

fix format=flowed text when the sidebar is enabled

fixes #26

Set COLUMNS to MuttIndexWindow->cols in mutt_create_filter_fd. (closes #3837)

This allows filters to automatically (or via a command line argument)
adjust to the width of the pager/index.  This is even more important
with the sidebar, since the width of the pager is smaller than the
terminal width.

Thanks to Vincent Lefèvre for his helpful information and suggestions,
and to Richard Russon for the initial patches.

bump dependency to 1.6.1

merge: upstream

 * Fix IDNA functions for systems without iconv.
 * Fix mutt_protect() when INLINE is set. (closes #3828)
 * Create a wrapper sys_socket.h to work around Solaris namespace issues.  (closes #3833)
 * automatic post-release commit for mutt-1.6.1

automatic post-release commit for mutt-1.6.1

Create a wrapper sys_socket.h to work around Solaris namespace issues.  (closes #3833)

Solaris includes "sys/stream.h" inside their "sys/socket.h".  This
include file adds many non-reserved macros to Mutt's namespace, two of
which conflict with existing Mutt macros.

The simplest fix would be to rename those macros in Mutt, however this
will cause difficulty with out-of-tree patches.  This fix creates a
wrapper include file that preserves those existing macros and prevents
the Solaris values from entering Mutt's namespace.

allow sidebar functions when sidebar is hidden

Simon Lundström (simmel) noted that <sidebar-next-new> didn't work when
the sidebar was hidden.  There was no good reason for it not to work.

fix typo

drop spurious attribute setting

travis: add config

[ci skip]

Make <sidebar-next-new> only find *new* mail

Previously, the <sidebar-next-new> function thought flagged emails were
"new".  This wasn't documented behaviour and was unexpected for the
user.

Fixes: #16

build fix

allow sidebar_divider_char to be longer

Originally '$sidebar_divider_char' could be an arbitrary string.

When I reworked Sidebar I limited it to 1 character (because I couldn't
see a good case for anything else).

It turns out that ' | ' isn't a bad choice for the divider.

fixes: #15

minor code tidy

fix crash caused by sidebar_folder_indent

Certain folder settings, combined with sidebar_folder_indent, caused an overflow
when trying to compare paths.

    set folder="~/mail/folder"
    mailboxes "~/mail/folder/"

Note the trailing / on the mailbox name.

Signed-off-by: Richard Russon <rich@flatcap.org>

allow the user to change mailboxes again

"unmailboxes *" used to cause Sidebar to crash.  The bug-fix for this
introduced another bug that prevented users from changing to a
non-Sidebar folder.

Signed-off-by: Richard Russon <rich@flatcap.org>

try hard to keep track of the open mailbox

If a user deletes the current mailbox with "unmailboxes", but then adds
it back with "mailboxes", then we will highlight it correctly.

fix refresh of pager

Sidebar didn't redraw properly when in the pager, or help.

fix formatting

Respect the TITLE_FMT when on the compose screen.

note config changes in the readme

correct sidebar's messages counts

Sidebar calculated the message counts slightly differently from Mutt.
This occasionally lead to the counts changing unexpectedly.

ignore case when sorting alphabetically

build fix for old compilers

change wiki references to website

Fix mutt_protect() when INLINE is set. (closes #3828)

The oppenc changes allow security bits to be set even when not
encrypting or signing (for instance, OPPENCRYPT and INLINE).

mutt_protect() assumed that if INLINE is set, then either ENCRYPT or
SIGN must also be set.  Specifically, it would end up inline-signing
the message even though neither was set.

Ensure mutt_protect() is a noop if neither SIGN or ENCRYPT are set.

In ci_send_message(), check for sign or encrypt before calling the
crypt_get_keys() / mutt_protect() block, and also in the fcc section
(since clear_content would be NULL if not).

The second change to the fcc part is somewhat redundant, but better to
be explicit and avoid the case where the subtype is somehow
"encrypted" or "signed" even though msg->security wasn't set thus.

Only sort the sidebar if we're asked to

fixes #11

Fix IDNA functions for systems without iconv.

The IDNA changes for SMTPUTF8 support introduced a bug for systems
without iconv.  For those systems, the local<->intl functions would
return an error due to the charset conversion failing.

Change mutt_idna.c back to being conditionally compiled, but this time
based on HAVE_ICONV.  If there is no iconv, stub out the functions in
mutt_idna.h.

add ident to PATCHES

add docs

refresh sidebar after timeout

Allow the sidebar to automatically refresh after the amount of time
specified via the "timeout" variable.

Signed-off-by: Stefan Assmann <sassmann@kpanic.de>

split out sidebar wipe

handle unmailboxes * safely

rename: lots of config for consistency with mutt

feature: separate colour for spoolfile

don't highlight spoolfile if not coloured

feature: separate sidebar indicator colour

default MT_COLOR_SB_INDICATOR to MT_COLOR_INDICATOR

expandos for deleted, limited, tagged

split up sb_draw

revamp sorting

distinguish between what's highlighted and what's actually open

refactor: visibility check

add: BUFFY notifications

refactor: split out initialise

add: colours for divider and highlights

add: BUFFY sorting

rename: vars

rename: msgcount to msg_count to match other sidebar variables

drop old sort

feature: add function sidebar-toggle-visible

fix refresh -- time overflowed short

rename SidebarLastRefresh to LastRefresh as it's only used in sidebar.c

fix brain-damaged function: mh_buffy_update

Since 1.5.19, mh_buffy_update hasn't worked.  It updated the values
of pointers rather than the destinations of the pointers.

build: conditional compilation

Make sidebar compilation conditional on a configure option.

Add --enable-sidebar option to configure
Alter Makefile to honour configure
Move our OPS into OPS.sidebar
Surround all our code with #ifdef USE_SIDEBAR

By using the unifdef program, all the sidebar code can be removed.
The result is code that differs only slightly from the original mutt
code.  Run this on each source file:

    unifdef -Bm -UUSE_SIDEBAR -UHAVE_FMEMOPEN <FILE>

Then run sed using this script:

    s/ *- *SidebarWidth//g
    s/ *+ *SidebarWidth//g
    s/SidebarWidth *+ *//g
    s/SidebarWidth/0/g

Compared to the original mutt code, there are some minor changes,
such as whitespace, and minor refactoring of variables.

Note: One sidebar global variable is left-behind: SidebarWidth.
This is deeply embedded in the mutt code.  This is because mutt
hard-codes a lot of values: 0's for the left-hand side of the
screen and COLS representing the width of the screen.

tidy: add comments

NOTE: this patch does NOT make any code changes.
It simply adds comment blocks.

Fully document all the sidebar config options.
Add comment-blocks describing each function -- what it does, the
parameters it takes, the return values.  These can be used by
doxygen.

tidy: rename functions

sort: sidebar functions

NOTE: this patch does NOT make any code changes.
It simply rearranges entire functions.

Reorder the functions in sidebar.c
Put the static (helper) functions together.

tidy: other code

Tidy up our code in mutt functions

Minor refactoring of code
Initialise variables
Check pointers before use
Bracket complicated logic

tidy: our functions

Tidy up code in our complete functions

Minor refactoring of code
Initialise variables
Check pointers before use
Bracket complicated logic

tidy: sidebar

Tidy up code in sidebar.c

Minor refactoring of code
Initialise variables
Check pointers before use
Bracket complicated logic
Add {} to unbalanced, or ambiguous, if's
Align = signs
Split up illegible uses of ternary operator ? :
Split up: if ((x = y()) == NULL)
Remove perl-like one-line if's
Compare pointers to NULL
Move our private data struct (sidebar_entry) from protos.h

fix: drop DrawFullLine

Drop global variable DrawFullLine

sidebar used a global variable in order to occasionally draw a
complete line of text on the screen.  By manipulating the
SidebarWidth, we can produce the same effect.

Removing this variable also removes the need for nearly 50 lines of
duplicated code in mutt_FormatString.

fix: drop unused

NOTE: this patch does NOT make any functional changes.

remove unused parameters from:
    draw_sidebar
    scroll_sidebar
    calc_boundaries
draw_sidebar has a return value that isn't used
struct MBOX_LIST isn't used
remove some unneeded #include's
remove defunct #ifdef BUFFY_SIZE
remove junk comments

indent: sync to surroundings

NOTE: this patch does NOT make any code changes.
If only adds/removes whitespace.

Tidy our code in mutt functions as best we can.
Importantly, match up the indent to the surrounding code.

Delete trailing whitespace
{} in compact style
Tidy whitespace around ()'s and maths operators

indent: our functions

NOTE: this patch does NOT make any code changes.
If only adds/removes whitespace.

Use whitespace uniformly across all of our code.

Delete trailing whitespace
Indent with tabs
Function {} to beginning of line
Other {} in compact style
Tidy whitespace around ()'s and maths operators

feature: refresh

Add new config: "sidebar_refresh"
Configure the minimum time between sidebar refreshes to reduce
network traffic.

feature: dotpathsep

Add new config: "sidebar_delim_chars"
Allow the user to configure how folder names are parsed into a
hierarchy.

feature: working utf8

Correctly handle utf8 character sequences.
Fix the distinction, in the code, between buffer length in bytes,
string length in wide-chars and screen width in cells.

feature: whitelist

Add new config: "sidebar_whitelist"
When displaying only-new mailboxes, force some mailboxes to be
always displayed

feature: prev/next wraparound

Add new config: "sidebar_next_new_wrap"
When searching for next-new/prev-new mailbox, wrap around if the end
is reached.

feature: new only

Add new config: "sidebar_newmail_only"
Only display mailboxes containing new mail.

Add two new functions and keyboard mappings.
"sidebar-next-new" / "sidebar-prev-new" select the next/previous
mailbox that contains new, or flagged, mail.