Hi

Thanks for the diff,

n retains the sense of the last search (repeats in the same direction), so this
is to reverse the sense of the last search?

> +                     if ((data->searchtype == WINDOW_COPY_SEARCHUP)
> +                         == (cmd == MODEKEYCOPY_SEARCHAGAIN))

This should be &&, I expect.

I think I would rather have a separate if in each case, eg:

                case WINDOW_COPY_SEARCHUP:
                        if (cmd == MODEKEYCOPY_SEARCHAGAIN)
                                window_copy_search_up(wp, data->searchstr);
                        else
                                window_copy_search_down(wp, data->searchstr);   
                
                        break;

And similar for WINDOW_COPY_SEARCHDOWN.

> +                             window_copy_search_up(wp, data->searchstr);
> +                     else
> +                             window_copy_search_down(wp, data->searchstr);


On Sat, Feb 06, 2010 at 02:19:03PM -0800, Micah Cowan wrote:
> The attached patch bind "N" (both copy modes) to "search-reverse", a new
> command that does the same as "search-again", but in the reverse of the
> current direction (same as "N" normally does in the vi editor).
> 
> -- 
> Micah J. Cowan
> http://micah.cowan.name/


> ------------------------------------------------------------------------------
> The Planet: dedicated and managed hosting, cloud storage, colocation
> Stay online with enterprise data centers and the best network in the business
> Choose flexible plans and management services without long-term contracts
> Personal 24x7 support from experience hosting pros just a phone call away.
> http://p.sf.net/sfu/theplanet-com

> _______________________________________________
> tmux-users mailing list
> tmux-users@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/tmux-users


------------------------------------------------------------------------------
The Planet: dedicated and managed hosting, cloud storage, colocation
Stay online with enterprise data centers and the best network in the business
Choose flexible plans and management services without long-term contracts
Personal 24x7 support from experience hosting pros just a phone call away.
http://p.sf.net/sfu/theplanet-com
_______________________________________________
tmux-users mailing list
tmux-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/tmux-users

Reply via email to