]> sigrok.org Git - libsigrok.git/commitdiff
input/csv: Update developer comment (fix for last EOL marker)
authorGerhard Sittig <redacted>
Mon, 5 Jun 2017 17:33:42 +0000 (19:33 +0200)
committerUwe Hermann <redacted>
Tue, 6 Jun 2017 21:28:15 +0000 (23:28 +0200)
src/input/csv.c

index ab22f0754f14ac788e1de61f1d78bbb6b1b3249a..1d230fdcad0e2c3777f329943fc1ddc219a6de0f 100644 (file)
  *   all of robustness and flexibility and correctness.
  *   - The current implementation splits on "any run of CR and LF". Which
  *     translates to: Line numbers are wrong in the presence of empty
- *     lines in the input stream.
- *   - The current implementation insists in the presence of end-of-line
- *     markers on _every_ line in the input stream. "Incomplete" text
- *     files that are so typical on the Windows platform get rejected as
- *     invalid.
+ *     lines in the input stream. See below for an (expensive) fix.
  *   - Dropping support for CR style end-of-line markers could improve
  *     the situation a lot. Code could search for and split on LF, and
  *     trim optional trailing CR. This would result in proper support
@@ -89,9 +85,7 @@
  *     input stream is scanned for the first occurance of either of the
  *     supported termination styles (which is good). For the remaining
  *     session a consistent encoding of the text lines is assumed (which
- *     is acceptable). Potential absence of the terminator for the last
- *     line is orthogonal, and can get handled by a "force" flag when
- *     the end() routine calls the process_buffer() routine.
+ *     is acceptable).
  *   - When line numbers need to be correct and reliable, _and_ the full
  *     set of previously supported line termination sequences are required,
  *     and potentially more are to get added for improved compatibility