Re: JDK-8067661: transferTo proposal for Appendable

2017-11-13 Thread Patrick Reinhart
Hi Brian See latest changes here: http://cr.openjdk.java.net/~reinhapa/reviews/8067661/webrev.03 -Patrick > Am 14.11.2017 um 01:34 schrieb Brian Burkhalter : > > Hi Patrick, > > More editorial comments ... ;-) > No problem :-) > In Readable.java these lines > > 73 * Depending on wh

Re: RFR(M) 8189116: Give the jdk.internal.vm.compiler.management only the permissions it really needs to expose the bean

2017-11-13 Thread Jaroslav Tulach
I tried the same test with changeset: 47679:d85284ccd1bd user:sspitsyn date:Fri Nov 03 17:09:25 2017 -0700 summary: 8189731: Disable CFLH when there are no transformers and it also yields the exception. E.g. the problem is certainly not result of my changes. -jt PS: I tr

Re: JDK-8067661: transferTo proposal for Appendable

2017-11-13 Thread Brian Burkhalter
Hi Patrick, More editorial comments ... ;-) In Readable.java these lines 73 * Depending on which class implements the appendable, there may be a limit 74 * of data that can written to which in turn could lead to an exception. are a bit unclear to me. Do you mean to say a limit o

Re: JDK-8067661: transferTo proposal for Appendable

2017-11-13 Thread Patrick Reinhart
Latest changes: http://cr.openjdk.java.net/~reinhapa/reviews/8067661/webrev.03 -Patrick > Am 13.11.2017 um 15:35 schrieb Roger Riggs : > > Hi Patrick, > > Comments on http://cr.openjdk.java.net/~reinhapa/reviews/8067661/webrev.02: > > Readable.java: > 74: „to an error" is a bit overloaded, p

Re: RFR(M) 8189116: Give the jdk.internal.vm.compiler.management only the permissions it really needs to expose the bean

2017-11-13 Thread Jaroslav Tulach
Hello Mandy, this was a good test: > > > > ./build/linux-x64/jdk/bin/java -XX:+UnlockExperimentalVMOptions -XX: > > +EnableJVMCI -XX:+UseJVMCICompiler -jar ... > > You can also try running the above command with -Djava.security.manager > as a sanity test (the application may need additional perm

Re: Faster Math ?

2017-11-13 Thread joe darcy
Hello, A few comments on this thread: As Paul noted, a portion of fdlibm has been ported from C to Java. I do intend to finish the port at some point. The port gives an implementation speedup by avoiding Java -> C -> Java transition overheads. However, the same algorithms are being used of co

RE: 8184777: Factor out species generation logic from BoundMethodHandle

2017-11-13 Thread Christoph Dreis
Hey Claes, though far away from being an expert on the subject matter, I have some very minor comments if you don't mind. ClassSpecializer.java L510: * For example, a concrete species for two reference and one integral bound values have a shape like the following: Should be imho: L510: * For ex

Re: RFR: 8184777: Factor out species generation logic from BoundMethodHandle

2017-11-13 Thread Claes Redestad
Thanks Vladimir! /Claes On 2017-11-13 17:38, Vladimir Ivanov wrote: Looks good! Best regards, Vladimir Ivanov On 11/13/17 7:34 PM, Claes Redestad wrote: Hi,   this patch factors out the BoundMethodHandle species data class generation to a new ClassSpecializer facility.   While currently se

Re: RFR: 8184777: Factor out species generation logic from BoundMethodHandle

2017-11-13 Thread Vladimir Ivanov
Looks good! Best regards, Vladimir Ivanov On 11/13/17 7:34 PM, Claes Redestad wrote: Hi,  this patch factors out the BoundMethodHandle species data class generation to a new ClassSpecializer facility.  While currently semantically neutral, this will make it possible to reuse the facility in

RFR: 8184777: Factor out species generation logic from BoundMethodHandle

2017-11-13 Thread Claes Redestad
Hi,  this patch factors out the BoundMethodHandle species data class generation to a new ClassSpecializer facility.  While currently semantically neutral, this will make it possible to reuse the facility in other places.  Webrev: http://cr.openjdk.java.net/~redestad/8184777/open.00/  Bug: https

Re: Change in properties for logging: deliberate?

2017-11-13 Thread Daniel Fuchs
Hi Jason, On 13/11/2017 15:14, Jason Mehrens wrote: Hi Daniel, Sorry for the late reply I was offline for the long weekend. Hot reloads of the LogManager have always been a problem. I think you are running into https://bugs.openjdk.java.net/browse/JDK-8033661 in your testing and that is goi

Re: Change in properties for logging: deliberate?

2017-11-13 Thread Jason Mehrens
Hi Daniel, Sorry for the late reply I was offline for the long weekend. Hot reloads of the LogManager have always been a problem. I think you are running into https://bugs.openjdk.java.net/browse/JDK-8033661 in your testing and that is going to give you troubling results on what is recreated a

Re: JDK-8067661: transferTo proposal for Appendable

2017-11-13 Thread Roger Riggs
Hi Patrick, Comments on http://cr.openjdk.java.net/~reinhapa/reviews/8067661/webrev.02: Readable.java: 74: "to an error" is a bit overloaded, perhaps "to an exception" X-Buffer.java.template: 1567: "this buffer":  do you mean the "out buffer" 1578:  "The out buffer is this buffer": will read

Re: Incorrect validation of DST in java.util.SimpleTimeZone

2017-11-13 Thread Venkateswara R Chintala
Thanks David, Peter for your review and comments. As I am new to the community, can you please help me to open a bug and integrate the changes into code base? -Venkat On 12/11/17 2:19 AM, Peter Levart wrote: Hi David, Venkat, On 11/11/17 21:15, Peter Levart wrote: For example, take the foll