On Sat, Aug 06, 2016 at 03:03:58PM -0700, Kevin J. McCarthy wrote:
> I would like some feedback on this patch.  Please read the patch summary
> for details.

Actually, I realized the patch can be much simpler.  I approached the
problem backwards: first making the switch in the pager, then adding the
cleanup in the index for the cases where it doesn't redirect, and
finally realizing I could consolidate the cleanup after the switch in
the index.

It looks like all I really need, though, is the
mutt_clear_pager_position() call after the index switch.  That catches
all the cases, and avoids having to specify the redirect operations!

I'll post a revised patch later today after I test for a bit.

-- 
Kevin J. McCarthy
GPG Fingerprint: 8975 A9B3 3AA3 7910 385C  5308 ADEF 7684 8031 6BDA

Attachment: signature.asc
Description: PGP signature

Reply via email to