Request for Review and Sponsor needed: JDK-8167648: java.io.PrintWriter should have PrintWriter((String|File), Charset) constructors

Patrick Reinhart patrick at reini.net
Wed Dec 21 21:08:13 UTC 2016


Hi Roger,

I tried to put your suggested changes, into the following webrev:

http://cr.openjdk.java.net/~reinhapa/reviews/8167648/webrev.01


>>>> - 375:  Can this use the new private constructor that will handle psOut.
>>> Here I can not get hold on the encoding at this point or have I missed something here?
>> True, not easily, it is available as a String from OutputStreamWriter.getEncoding() but it would suffer from 
>> having to lookup it by name again.
> 
> Right, even the Charset is actually available within the StreamEncoder implementation but not provided to the outside world.
> 
> Also the OutputStreamWriter is in wrapped in a BufferedWriter and would be needed to be extracted from there again in the first place. 

Here I left the invocation as it was beforeā€¦

-Patrick



More information about the core-libs-dev mailing list