On Sun, Jun 09, 2024 at 09:57:20AM +0200, Dragan Simic wrote: > Hello Ruben, > > On 2024-06-09 09:44, Rubén Justo wrote: > > Some concerns were raised about controlling when to send coloring escape > > sequences. Several ideas were discussed to address this, including > > introducing a new command ">", a modifier for "|": "||", and others. > > Alternatively, we could leave it up to the user to filter as needed. > > Or, simply, do not send escape codes at all. > > > > So, looking back at the ideas discussed in the thread, perhaps a > > reasonable next step might be to reintroduce the 'P<program>' command > > and let '|<program>' be the way to send raw, uncolored, chunks. > > Actually, it would be better to re-introduce the "P" option, without > any parameters, which would display the current hunk through the > already configured pager I'm sorry, but why limit the "P" command now? I understand the caution expressed in another message of this thread about playing it safe, but I think the user won't be surprised if we respect here the "color.diff" setting, just like we do with "p", and ... > and let "|<program>" be the new option > that pipes hunks _without_ coloring escape sequences to "<program>". ... we'll offer the command "|" to allow the user to process the raw chunk.
