Richard Russon

> I use neomutt with qygwin in win 7 Ooh! That's interesting. Did you compile it yourself? (or is someone packaging it?) > When the app asks [yes/no], I can...

Hmm... that **is** tricky. Macros are very literal. If you had a conditional `` then what happens to the `z` that comes next? You'd need a `[my macro]` Or perhaps:...

Thanks for debugging, @gdkrmr. `op_main_entire_thread()` is trying to change folder and failing, causing `shared->mailbox` to become `NULL`. https://github.com/neomutt/neomutt/blob/1cabdee81c7d279b42b419c54942d32b374a2e79/index/functions.c#L2928 The simple fix is: ```diff -if (shared->mailbox->msg_count == 0) +if (!shared->mailbox ||...

OK. It's notmuch. Please can you do it ;-) Thanks

> Introduce threading Two words. Sounds so easy :-) It's something we've been working towards for six years. Mutt's code is a dreadful mess. The code is poorly designed and...

This function is doing a lookup of a name, say `send-hook`, in the array of Commmands. It's checking the `parse` member to avoid doing up to 78 string comparisons. The...

There are two sides to the hooks in that file: parsing and executing. `mutt_message_hook()` and `mutt_startup_shutdown_hook()` are the execution side. This issue is just about refactoring `mutt_parse_hook()`, to remove many...

> What is planned for this issue? Needs doing. Volunteer needed :heart:

> The function is long Nah! It's only 223 lines long -- it doesn't even make the top ten! - 1087 `mutt_buffer_select_file()` - 1013 `index_format_str()` - 950 `main()` - 850...

> The variable names are not realy telling: > `mutt_parse_hook(struct Buffer *buf, struct Buffer *s, intptr_t data, struct Buffer *err)` Yeah, I know. They're historic names that I haven't quite...