johnjaylward

Results 68 comments of johnjaylward

Here are the results of a developer survey published 1.5 years ago that show 3% of developers using Java7 or lower: https://snyk.io/blog/developers-dont-want-to-leave-java-8-as-64-hold-firm-on-their-preferred-release/

Now that Java 17 is the new LTS version (java 11 still being in LTS support), Here are some highlights of features/improvements added to Java 12-17 that may be applicable...

I think your function is the right idea, but not quite there. I'd implement this similar to what you have, but more like this: ```java /** * Converts a JSONArray...

personally, I wouldn't bother with toList at all in your use-case though. as in #672, I would just use the spliterator to get a stream and continue to use the...

if you want to make that as a helper method, it would look something like this: ```java public static Stream toStream(JSONArray array, Class cast) { return java.util.StreamSupport(jarr.spliterator(), false) .filter(Objects::nonNull) .filter(i->...

is it an empty "last value" or is it the missing new line `\n` causing the issue? For example, does the following data work: ``` final String SIMPLE_CSV_EMPTY_COLUMN_STRING = "Number,Name,Empty\n"...

I'm not sure about CSV specifically, but here is a good answer for text files in general: https://unix.stackexchange.com/a/18789

looks like CSV itself is lose on the ending new-line: https://www.csvreader.com/csv_format.php Feel free to submit a PR for a fix

> Feel free to submit a PR for a fix The JavaDoc for the CDL class does specify that every row ends with a NEWLINE. So if you do make...

> I found as well that the test emptyLinesToJSONArray doesn't check anything because last row in str is omitted so it ends up being a case of a CSV with...