Ticket #2630 (new defect)
Store the growbuffer / Rename Parse into Refresh for DS_*_PIPE
Reported by: | keasy | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | Future Releases |
Component: | mcview | Version: | master |
Keywords: | Cc: | ||
Blocked By: | Blocking: | ||
Branch state: | no branch | Votes for changeset: |
Description
When std pipes, data that comes our way is very feeble. It might be a one-at-time result of some work, unique in some way.
When user presses Raw/Parse?, he expects to Raw/Parse? that same set of data. That is happening for regular files, but not pipes! Growbuffers are freed, and ...
If it's a STDIN pipe, the data just goes away and on a second read, nothing is there. Not good at all.
If it's a STDIO pipe (i.e. data is output of some command), *the command is re-executed* and only then parsed. Meaning it's actually a different set of data.
Therefore, the Raw/Parse? button actually acts as a Refresh button when pipes are involved. Easy fix is to just rename the button to "Refresh" for pipes, harder one is to store the growbuffers and use THEM instead of re-querying original data again.