Hi,
On Mon, Apr 02, 2018 at 03:12:49PM +0200, Gero Treuner wrote:
> Tested
>
> - Mailbox types: see above
> - General compilation: on Linux with/without **1 inotify
> - Sidebar compilation: breaks with same message in original and patched
> source with my configuration/system (current git master HEAD).
Regarding sidebar compatibility:
Compiling ("touch OPS" helped) and using it (first time for me) works
without problems. IMO that is the option where I expected most
interference,
Attached is a 2nd patch with cosmetic cleanups.
Gero
commit 279fb09a565924f3491dbe61ccd3976d232a4c80
Author: Gero Treuner <[email protected]>
Date: Mon Apr 2 17:51:55 2018 +0200
Tiny cleanups
diff --git a/monitor.c b/monitor.c
index f5847816..888df3db 100644
--- a/monitor.c
+++ b/monitor.c
@@ -214,7 +214,6 @@ int mutt_monitor_poll ()
if (INotifyFd != -1)
{
fds = poll (PollFds, PollFdsLen, -1);
- /* exits with !0 before notifier get written - permanent interrupt? */
if (fds == -1)
{
@@ -324,8 +323,7 @@ static int monitor_resolve (MONITORINFO *info, BUFFY *buffy)
if (fmt)
{
- snprintf (info->_pathbuf, sizeof(info->_pathbuf),
- info->magic == MUTT_MAILDIR ? "%s/new" : "%s/.mh_sequences", info->path);
+ snprintf (info->_pathbuf, sizeof(info->_pathbuf), fmt, info->path);
info->path = info->_pathbuf;
}
if (stat (info->path, &sb) != 0)
