pidcat icon indicating copy to clipboard operation
pidcat copied to clipboard

add option to override word wrap width

Open niedzielski opened this issue 11 years ago • 2 comments

niedzielski avatar Jul 26 '14 04:07 niedzielski

Why do you need this? On Jul 25, 2014 9:37 PM, "Stephen Niedzielski" [email protected] wrote:


You can merge this Pull Request by running

git pull https://github.com/niedzielski/pidcat patch-1

Or view, comment on, or merge it at:

https://github.com/JakeWharton/pidcat/pull/64 Commit Summary

  • add option to override word wrap width

File Changes

  • M pidcat.py https://github.com/JakeWharton/pidcat/pull/64/files#diff-0 (16)

Patch Links:

  • https://github.com/JakeWharton/pidcat/pull/64.patch
  • https://github.com/JakeWharton/pidcat/pull/64.diff

— Reply to this email directly or view it on GitHub https://github.com/JakeWharton/pidcat/pull/64.

JakeWharton avatar Jul 28 '14 02:07 JakeWharton

I use this option for the following cases: 1 Disable word wrapping. 2 Handling pipelines (grep, wc, tee, etc) and file redirection which don't have a known desired width. 3 Start pidcat from a small terminal not yet maximized.

This option seemed like a simple, albeit crude, solution to all three:

1 This could be implemented as a standalone option.

2 Grepping with context isn't fun. Detection for the output stream could and probably should be added but I think this is still a nice control to have.

3 I don't like that the wrapping behavior gets stuck on the invocations size. Dynamic terminal sizing could be added, but I like the flexibility to assume a constant size, if specified.

niedzielski avatar Jul 28 '14 13:07 niedzielski