Ticket #2630 (new defect)

Opened 7 years ago

Last modified 7 years ago

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.

Change History

comment:1 Changed 7 years ago by keasy

  • Component changed from mc-core to mcview
Note: See TracTickets for help on using tickets.