Maybe I should remove both the Reader and the InputStream versions,
since they cannot parse multiple log events from a stream, and are thus
not very useful.
On 2017-07-26 10:34, Jörn Huxhorn wrote:
It would maybe be a good idea to get rid of the Reader method altogether since
it mainly introduces an unnecessary point of failure if the Reader is using an
encoding other than UTF-8.
Using a FileReader would be an example for this since its limited c'tors are
always using Charset.defaultCharset().name().
The correct way to create an UTF-8 file reader is the less than obvious "new
InputStreamReader(new FileInputStream(file), StandardCharsets.UTF_8)”. That’s a very
common mistake.