Add new flag: `-H`/`--hold-pollute` - Githubissues

stefanheule / smartless

A pager similar to less, but smarter

Apache License 2.0

25 stars 5 forks source link

Add new flag: `-H`/`--hold-pollute` #15

Closed stdedos closed 5 years ago

stdedos commented 5 years ago

Instead of filling up the screen, print less-than-half of the screen. Works also with collated arguments (e.g. -iRH, -HiR, -iHR)

Minor changes:

No need for $@ for explicitly iterating input arguments
Useless Use Of Cat fix (cat ... | cmd === cmd < ... for most cases)
Fix printing input with no trailing \nEOF
Move the output functions closer to "output"
Unify output functions
Use printf instead of echo (portability)
Preserve input output format (no excessive \n added)
Left some personal unit tests as TODO comments

Signed-off-by: Ntentos Stavros 133706+stdedos@users.noreply.github.com

Some leading conversation regarding "naming" of the new flag: https://github.com/stdedos/smartless/commit/b2de6713ad8a665f6d5fac859eba157a9289806b#commitcomment-26483654

stefanheule commented 5 years ago

Sorry for taking so long to get to this, I really appreciate the contribution! Thank you.

stdedos commented 5 years ago

No worries, I am glad this appears to be useful after all!

I might have some other internal fixes, so, you may be expecting something today again.