Re: [PATCH v3 7/7] mergetools: vimdiff: restore selective diff mode

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Aug 9, 2022 at 12:26 AM Fernando Ramos <greenfoo@xxxxxx> wrote:
>
> On 22/08/08 08:08PM, Felipe Contreras wrote:
> >
> > Notice that after reorganizing the tab handling my layout becomes really simple:
> >
> >   set hidden diffopt-=hiddenoff | silent execute 'bufdo diffthis' | 4b
>
> Thanks! I really like this new approach.
>
> I also manually tested your new series (with and without .vimrc) and it works as
> expected in all listed test cases.
>
> This last series gets the best of both worlds:
>
>   - Highglihting works in single window tabs even when there is just one tab
>
>   - Single tab layouts are not treated differently
>
>   - Generates shorter (easier to understand!) vim command strings (nice!)
>
>   - Opens the gate for a future configuration option that lets you enable "all
>     buffers diff mode" even if they are not visible in a given tab (maybe this
>     could be a new syntax token, *, that "marks" a tab to work in this mode?)
>
> So, definitely a great work. Thanks for the deeper look at the problem and this
> brilliant solution :)

It probably can be cleaned up a bit more, but the important thing is
the idea: parse tabs especially (which they are anyway).

Now if only the maintainer cared about fixing the regression that
would be great.

-- 
Felipe Contreras



[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux