[PATCH v3] VIM: Improve moving between messages in a thread

Bartosz Telenczuk telenczuk at unic.cnrs-gif.fr
Mon Jan 12 15:47:19 PST 2015


Hi Ian,

The patch looks good. I tested it on my system and it works fine.  I just have one suggestion regarding coding style.

> +function! s:show_prev_msg()
>  function! s:show_next_msg()

Since these functions are almost the same, you could avoid code repetition by replacing them by a function which takes (positive or negative) increment as an argument.

Cheers,

Bartosz



More information about the notmuch mailing list