On Sat, Jun 08, 2024 at 07:54:34AM +0200, Dragan Simic wrote: > When "|xyz" is used instead, the version of the hunk with no coloring > escape sequences should be piped to xyz. That is a sane and conservative approach, and I'm not opposed. However, giving the colorful version though a custom pager is a good thing to have, I think, i.e: allowing a simple "head" without losing the coloring. Let's recap a bit. Initially, this series aimed to enable sending chunks to the pager during "add -p" sessions. To reduce the blast radius of spawning a pager for each chunk, we introduced a new command "P". Junio suggested opening up the command to allow specifying a custom pager, in the form of "P<program>". The "P" command started to resemble a lot to the common pipe operator. Thus, we shifted to "|<program>". 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. This approach makes sense to me. I'll wait a bit before sending a reroll to gather feedback, though. Thanks.
