Re: RFR: 8201274: Launch Single-File Source-Code Programs

2018-04-13 Thread Jan Lahoda
The javac part looks OK to me. A nit comment, in: launcher/Main.java/MemoryFileManager#createInMemoryClassFile, there is: return new FilterOutputStream(new ByteArrayOutputStream()) { ... It could I think be written as: return new ByteArrayOutputStream() { @Override public void close() th

Re: Clean-room implementation of Double::toString(double) and Float::toString(float)

2018-04-13 Thread Thomas Stüfe
On Sat, Mar 31, 2018 at 12:17 AM, Brian Burkhalter wrote: > Hi Raffaello, > > On Mar 30, 2018, at 2:57 PM, raffaello.giulie...@gmail.com wrote: > […] The new code also has a better specification than the current one, while being mostly compatible. Indeed, the current specificat

Re: Missing many locales support on AIX platform

2018-04-13 Thread Bhaktavatsal R Maram
Given that patch is big, I am sending patch as attachment again after changing some mail settings. Hopefully, it will make to community this time. Thanks, Bhaktavatsal Reddy -Bhaktavatsal R Maram/India/IBM wrote: - To: core-libs-dev@openjdk.java.net From: Bhaktavatsal R Maram/India

Re: Missing many locales support on AIX platform

2018-04-13 Thread Bhaktavatsal R Maram
No luck! :( How can I attach patch? Any suggestions? Thanks, Bhaktavatsal Reddy -Bhaktavatsal R Maram/India/IBM wrote: - To: core-libs-dev@openjdk.java.net From: Bhaktavatsal R Maram/India/IBM Date: 04/13/2018 03:05PM Subject: Re: Missing many locales support on AIX platform Given t

Re: Missing many locales support on AIX platform

2018-04-13 Thread Alan Bateman
On 13/04/2018 10:35, Bhaktavatsal R Maram wrote: Given that patch is big, I am sending patch as attachment again after changing some mail settings. Hopefully, it will make to community this time. Your patch was attached. If I read it correctly, you've switched IBM943C to a template but there

Re: RFR: 8201274: Launch Single-File Source-Code Programs

2018-04-13 Thread Maurizio Cimadamore
One small followup question - the test SourceLauncherTest is not covering any shebang cases - is that deliberate? I see that those seem to be covered in the launcher test too, but I wonder if we should have tests for clearly broken stuff, such as '#' '#!' '#!\n' Another small question - I s

Re: Missing many locales support on AIX platform

2018-04-13 Thread Bhaktavatsal R Maram
Hi Alan, Thank you for your response. I'm happy that my patch was attached. But, I don't see attachment. So, I inlined patch which contain diffs from 2 changesets in mail text. If a Jira bug is opened for this issue, probably I can attach complete and consolidated patch there. At high level, I

Re: Missing many locales support on AIX platform

2018-04-13 Thread Roger Riggs
Hi, The question has come up before:   [1] Most OpenJDK mailing lists, ..., allow these attachment types: text/plain text/x-diff text/x-patch message/rfc822 Attachments with the following filename extensions are dropped: exe bat cmd com pif scr vbs cpl Regards, Roger

Re: [PATCH] RFR Bug-pending: Enable Hotspot to Track Native Memory Usage for Direct Byte Buffers

2018-04-13 Thread Adam Farley8
Hi Alan, Peter, I see that native memory is tracked in java.nio.Bits, but that only includes what the user thinks they are allocating. When the VM adds extra memory to the allocation amount this extra bit is not represented in the Bits total. A cursory glance shows, minimum, that we round the

InetAddress.getByName/getAllByName for empty host string

2018-04-13 Thread Jaikiran Pai
The javadoc of InetAddress.getByName(host) and getAllByName(host) states that: If the host is null then an InetAddress representing an address of the loopback interface is returned. For non-null values the javadoc explains what the implementation does. However, there seems to be no mention o

Re: InetAddress.getByName/getAllByName for empty host string

2018-04-13 Thread Chris Hegarty
Hi Jaikiran, On 13/04/18 15:29, Jaikiran Pai wrote: The javadoc of InetAddress.getByName(host) and getAllByName(host) states that: If the host is null then an InetAddress representing an address of the loopback interface is returned. For non-null values the javadoc explains what the impleme

Re: Missing many locales support on AIX platform

2018-04-13 Thread Volker Simonis
Hi Bhaktavatsal Reddy, thanks for addressing this long standing issue. I've opened "8201540: [AIX] Extend the set of supported charsets in java.base" [1] to track this issue. As I wrote in the bug report, this problem is the consequence of an emergency fix (JDK-8081332) I did back in 2015 to fix

Re: InetAddress.getByName/getAllByName for empty host string

2018-04-13 Thread Jaikiran Pai
Hi Chris, Thank you creating that JIRA. If the fix involves just updating the javadoc, is this something that youwould like me to contribute as a patch? I have a signed and approved OCA, but I will need a sponsor if I do come up with the patch. -Jaikiran On 13/04/18 8:41 PM, Chris Hegarty

Re: Missing many locales support on AIX platform

2018-04-13 Thread Bhaktavatsal R Maram
Hi Volker, Thank you. I will take help of my colleagues and host webrev with all the default charsets that are missing for AIX. - Bhaktavatsal Reddy -Volker Simonis wrote: - To: Bhaktavatsal R Maram From: Volker Simonis Date: 04/13/2018 08:51PM Cc: Alan Bateman , Java Core Libs ,

Re: [11] RFR: 8201507: Generate alias entries in j.t.f.ZoneName from tzdb at build time

2018-04-13 Thread Erik Joelsson
Build changes look good. /Erik On 2018-04-12 17:34, Naoto Sato wrote: Hi, Please review the fix to the subject issue. While fixing 8189784 [1], I noticed that not only CLDR zones but also tzdb link entries are also hard coded. So I further modified j.t.f.ZoneName to generate tzdb entries a

Re: [11] RFR: 8201507: Generate alias entries in j.t.f.ZoneName from tzdb at build time

2018-04-13 Thread Roger Riggs
Hi Naoto, Looks fine. Regards, Roger On 4/12/18 8:34 PM, Naoto Sato wrote: Hi, Please review the fix to the subject issue. While fixing 8189784 [1], I noticed that not only CLDR zones but also tzdb link entries are also hard coded. So I further modified j.t.f.ZoneName to generate tzdb entr

Re: [PATCH] RFR Bug-pending: Enable Hotspot to Track Native Memory Usage for Direct Byte Buffers

2018-04-13 Thread Alan Bateman
On 13/04/2018 15:14, Adam Farley8 wrote: Hi Alan, Peter, I see that native memory is tracked in java.nio.Bits, but that only includes what the user thinks they are allocating. When the VM adds extra memory to the allocation amount this extra bit is not represented in the Bits total. A cursor

RFR: JDK-8194750, Console.readPassword does not save/restore tty settings

2018-04-13 Thread Xueming Shen
Hi, Please help review the change for JDK-8194750. issue: https://bugs.openjdk.java.net/browse/JDK-8194750 webrev: http://cr.openjdk.java.net/~sherman/8194750/webrev *fix has been manually verified. No new auto-regression test added. Thanks, Sherman

Re: RFR: JDK-8194750, Console.readPassword does not save/restore tty settings

2018-04-13 Thread Martin Buchholz
Emacs users thank you for working on this. To repro, try emacs -q, M-x shell and run your manual test in there. You'll see stty -a reports -echo. This is trickier than I expected, since you have to manage saving/restoring around each call to readPassword AND have an exit hook to restore in case

RFR: 8184693: (opt) add Optional.isEmpty

2018-04-13 Thread Vivek Theeyarath
Hi All, Please review. Bug: https://bugs.openjdk.java.net/browse/JDK-8184693 Webrev : http://cr.openjdk.java.net/~vtheeyarath/8184693/webrev.00/ The related jtreg test was run and the test passed . Regards Vivek