| org.apache.commons.csv.Assertions |
Line |
| Replace usage with {@link Objects} when we switch to Java 7. |
23 |
| org.apache.commons.csv.CSVFormat |
Line |
| --No comment-- |
502 |
| --No comment-- |
504 |
| --No comment-- |
506 |
| --No comment-- |
508 |
| --No comment-- |
510 |
| where did this rule come from? |
1037 |
| org.apache.commons.csv.CSVParser |
Line |
| This is not great, throw an ISE instead? |
443 |
| org.apache.commons.csv.CSVParserTest |
Line |
| this may lead to strange behavior, throw an exception if iterator() has already been called? |
713 |
| org.apache.commons.csv.CSVPrinter |
Line |
| Is it a good idea to do this here instead of on the first call to a print method? It seems a pain to have to track whether the header has already been printed or not. |
66 |
| org.apache.commons.csv.CSVRecord |
Line |
| Maybe make this public? |
244 |
| org.apache.commons.csv.Lexer |
Line |
| escape handling needs more work |
313 |
| is this correct? |
341 |
| is this correct? Do tabs need to be escaped? |
342 |
| is this correct? |
343 |
| org.apache.commons.csv.LexerTest |
Line |
| is this correct? Do we expect <esc>TAB to be unescaped? |
312 |
| is this correct? Do we expect <esc>BACKSPACE to be unescaped? |
327 |
| is this correct? Do we expect <esc>FF to be unescaped? |
341 |
