Status: To Do
Affects Version/s: None
Fix Version/s: None
During work on
METRON-612, I noticed a lot of our warnings are from implicit use of the default Charset (e.g. when creating InputStreamReaders). Given that this value is platform dependent, we should at least consider explicitly using a particular Charset.
In addition, after fixing this, I would like to see it upgraded to a compile error, because in my experience almost everyone tends to use methods that use the implicit default Charset. This can be done by changing the compiler arg in the pom to instead be
My assumption is that we'll want UTF-8, but given a lack of expertise and that usually there's a lot of opinions on character set issues, we may want to consider other options.
https://docs.oracle.com/javase/8/docs/api/java/nio/charset/Charset.html lists standard Charsets and provides more details.