]> sigrok.org Git - libsigrok.git/commit
input/csv: move channel creation to after format parsing
authorGerhard Sittig <redacted>
Fri, 18 Oct 2019 23:07:28 +0000 (01:07 +0200)
committerGerhard Sittig <redacted>
Sat, 21 Dec 2019 17:20:04 +0000 (18:20 +0100)
commit05719d75aa31627a25375567b3641d8787423567
tree430ff97322b99060013d89c8443e0de1be12604a
parentcd7c5f9655b4edd766bec5b3d1926609f1e736f3
input/csv: move channel creation to after format parsing

The previous implementation incompletely handled arbitrary data type
oders in mixed signal input files. Rearrange the logic such that all
format specs get parsed first, then all channel creation details get
determined, then all channels get created. It appears to be essential
that all logic channels get created first, resulting in index numbers
starting at 0 and addressing the correct position in bitfields. The
analog channels get created after all logic channels exist. Adjacent
number ranges for channel types also results in more readable logic in
other locations.

This was tested with -I csv:column_formats=t,2a,l,a,x4,a,-,b3 example
data, and works as expected.
src/input/csv.c