Integrated: 8263560: Remove needless wrapping with BufferedInputStream

2021-03-29 Thread Сергей Цыпанов
On Sat, 13 Mar 2021 22:29:23 GMT, Сергей Цыпанов wrote: > In some cases wrapping of `InputStream` with `BufferedInputStream` is > redundant, e.g. in case the wrapped one is `ByteArrayOutputStream` which does > not require any buffer having one within. > > Other cases are related to reading ei

Re: New candidate JEP: 408: Simple Web Server

2021-03-29 Thread Krzysztof K.
Hi, Is there a plan to change the package name? I always thought that com.sun.* package was for old internal code and I assume I'm not the only one. Regards, Krzysztof Krason On Mon, Mar 29, 2021 at 9:16 PM wrote: > https://openjdk.java.net/jeps/408 > > Summary: Provide a command-line tool to

New candidate JEP: 408: Simple Web Server

2021-03-29 Thread mark . reinhold
https://openjdk.java.net/jeps/408 Summary: Provide a command-line tool to start a minimal web server that serves static files in the current directory. This low-threshold utility will be useful for prototyping, ad-hoc coding, and testing purposes, particularly in educational contexts. - M

Re: RFR: 8263506: Make sun.net.httpserver.UnmodifiableHeaders unmodifiable [v2]

2021-03-29 Thread Michael McMahon
On Mon, 29 Mar 2021 09:09:00 GMT, Julia Boes wrote: >> The fix makes the map in sun.net.httpserver.UnmodifiableHeaders unmodifiable >> by wrapping it in an unmodifiable view. > > Julia Boes has updated the pull request incrementally with two additional > commits since the last revision: > > -

Re: RFR: 8263506: Make sun.net.httpserver.UnmodifiableHeaders unmodifiable [v2]

2021-03-29 Thread Daniel Fuchs
On Tue, 16 Mar 2021 14:35:12 GMT, Michael McMahon wrote: >> Julia Boes has updated the pull request incrementally with two additional >> commits since the last revision: >> >> - wrap List with unmodifiable map and update test >> - remove map wrapping > > Changes requested by michaelm (Reviewe

Re: RFR: 8263506: Make sun.net.httpserver.UnmodifiableHeaders unmodifiable [v2]

2021-03-29 Thread Julia Boes
On Mon, 22 Mar 2021 16:41:10 GMT, Daniel Fuchs wrote: >> Actually, I just noticed that the `List`s are still modifiable. The >> constructor might have to create an intermediate `Map>` >> with unmodifiable Lists and then you create an unmodifiable Map from that. >> >> As regards the comment ab

Re: RFR: 8263506: Make sun.net.httpserver.UnmodifiableHeaders unmodifiable [v2]

2021-03-29 Thread Julia Boes
On Mon, 22 Mar 2021 16:36:55 GMT, Daniel Fuchs wrote: >> I guess this presumes that no reference to `headers` escapes? Can that be >> assumed? > > Yes - it's private - and that wasn't a problem before - was it? The problem > was the put method that still put things into the map - and that will

Re: RFR: 8263506: Make sun.net.httpserver.UnmodifiableHeaders unmodifiable [v2]

2021-03-29 Thread Julia Boes
> The fix makes the map in sun.net.httpserver.UnmodifiableHeaders unmodifiable > by wrapping it in an unmodifiable view. Julia Boes has updated the pull request incrementally with two additional commits since the last revision: - wrap List with unmodifiable map and update test - remove map wr