On Tue, Jan 10, 2017 at 20:53 -0000, Karl Palsson wrote:
>
> Anything further?
Since you Cc'ed me specifically, I'll repeat to the ML what I've
said before on IRC.
I don't oppose your cleanup of how sigrok-cli handles decoder
stacks. The improvement is welcome. After the reply to the ML
I became aware that changing the stdout format is beneficial and
improves robustness and usability even for a single stack with
multiple decoders and differing annotations.
There was some brain storming and mockup how stacked decoders'
output could get presented in the GUI, but that probably does not
translate to the CLI incarnation.
Somebody mentioned that -S should not completely get axed, but
should work as before for a single stack, and emit an appropriate
message for multiple stacks. Can't tell whether you addressed
this concern.
Haven't dug deeper into the patches myself, neither read them
thoroughly nor ran the version here so far. Neither have I come
to a (final) conclusion what I'd expect from -A as a user in the
presence of multiple stacks. Being able to create multiple
stacks in the first place is an improvement nevertheless.
So many of my concerns have become obsolete. The general thought
of proper support for multiple stacks is very much appreciated.
While some food for thoughts remain about backwards compatibility
and maybe about what users want to pick / filter from the output
and how to most appropriatelly translate that to command line
options.
virtually yours
Gerhard Sittig
--
If you don't understand or are scared by any of the above
ask your parents or an adult to help you.
------------------------------------------------------------------------------
Developer Access Program for Intel Xeon Phi Processors
Access to Intel Xeon Phi processor-based developer platforms.
With one year of Intel Parallel Studio XE.
Training and support from Colfax.
Order your platform today. http://sdm.link/xeonphi
_______________________________________________
sigrok-devel mailing list
sigrok-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/sigrok-devel
