RE: RFR 8213031: (zipfs) Add support for POSIX file permissions

2019-02-13 Thread Langer, Christoph
Hi Lance, thanks for the detailed explanation, sounds great. I’ll work that in in my next edition 😊 Best regards Christoph From: Lance Andersen Sent: Mittwoch, 13. Februar 2019 23:53 To: Langer, Christoph Cc: Alan Bateman ; nio-dev ; Java Core Libs ; OpenJDK Dev list ; Volker Simonis Subj

Re: JDK-6982173: Small problem causing thousands of wasted CPU hours

2019-02-13 Thread Stuart Marks
On 2/13/19 7:22 PM, Alan Snyder wrote: If we take this route, how about changing the parameter type to Iterable? Won't work. Where I've ended up is that we need to iterate over "this" collection and, for each element, call contains() on the parameter. The AbstractCollection.removeAll() implem

Re: JDK-6982173: Small problem causing thousands of wasted CPU hours

2019-02-13 Thread Alan Snyder
If we take this route, how about changing the parameter type to Iterable? Alan > On Feb 13, 2019, at 7:12 PM, Stuart Marks wrote: > > Right, as I mentioned in my earlier emails [1][2] this is related to > JDK-6394757 [3] where the semantics shift depending on the relative sizes of > the c

Re: JDK-6982173: Small problem causing thousands of wasted CPU hours

2019-02-13 Thread Stuart Marks
Right, as I mentioned in my earlier emails [1][2] this is related to JDK-6394757 [3] where the semantics shift depending on the relative sizes of the collections. This has a complicated history. In JDK-6982173 [4] there is a lot of discussion about what heuristics to use for iterating this vs th

Re: JDK-6982173: Small problem causing thousands of wasted CPU hours

2019-02-13 Thread Stuart Marks
On 2/8/19 5:13 AM, Tagir Valeev wrote: I would argue that iterating the argument and calling remove() on "this" are the right semantics, because you want set membership to be determined by this set, not by whatever collection you pass as an argument. However, I note that AbstractCollection.rem

Re: RFR 8213031: (zipfs) Add support for POSIX file permissions

2019-02-13 Thread Lance Andersen
Hi Christoph > On Feb 13, 2019, at 5:30 PM, Langer, Christoph > wrote: > > Hi Lance, > > thanks for looking. > >> Just starting to take a peek at this and noticed one quick thing in your >> test: >> >> Paths.get(System.getProperty("test.dir", "."), "testPosix.zip") >> —— >>

RE: RFR 8213031: (zipfs) Add support for POSIX file permissions

2019-02-13 Thread Langer, Christoph
Hi Lance, thanks for looking. > Just starting to take a peek at this and noticed one quick thing in your test: > > Paths.get(System.getProperty("test.dir", "."), "testPosix.zip") > —— > > You do not need the test.dir property  or the permission added to test.policy > to access i

RE: RFR 8213031: (zipfs) Add support for POSIX file permissions

2019-02-13 Thread Langer, Christoph
Hi Alan, thanks for taking a first look into this new edition. > I think the javadoc changes will need a few iterations but we can get to > that once some of the finer details are sorted out. For example, "Posix > Support" isn't quite right as this is about optional support for the > POSIX view o

Re: RFR 8213031: (zipfs) Add support for POSIX file permissions

2019-02-13 Thread Lance Andersen
Hi Christoph, Just starting to take a peek at this and noticed one quick thing in your test: Paths.get(System.getProperty("test.dir", "."), "testPosix.zip") —— You do not need the test.dir property or the permission added to test.policy to access it, just reference the jar a

Re: RFR(JDK 13/java.xml) 8187697: Cleanup: irrelevant code in OutputPropertiesFactory

2019-02-13 Thread Joe Wang
On 2/13/19, 11:55 AM, Lance Andersen wrote: Hi Joe Overall the change seems Ok, assuming all the tests are happy :-) Yes, they all are happy. One minor suggestion for: src/java.xml/share/classes/com/sun/org/apache/xml/internal/serializer/OutputPropertiesFactory.java - line 150, plea

Re: RFR 8218228 : The constructor StringBuffer(CharSequence) violates spec for negatively sized argument

2019-02-13 Thread Roger Riggs
Hi Ivan, The release note looks fine,  Resolve -> FixDelivered to pass it on to the tech writer for review. Yes, communication is sometimes the hard part. Thanks, Roger On 02/12/2019 08:21 PM, Ivan Gerasimov wrote: Uh, lots of paperwork, comparing to the actual change in the source code :-

Re: RFR(JDK 13/java.xml) 8187697: Cleanup: irrelevant code in OutputPropertiesFactory

2019-02-13 Thread Lance Andersen
Hi Joe Overall the change seems Ok, assuming all the tests are happy :-) One minor suggestion for: src/java.xml/share/classes/com/sun/org/apache/xml/internal/serializer/OutputPropertiesFactory.java - line 150, please put the ‘*/“ on its own line > On Feb 13, 2019, at 1:15 PM, Joe Wang wro

RFR(JDK 13/java.xml) 8187697: Cleanup: irrelevant code in OutputPropertiesFactory

2019-02-13 Thread Joe Wang
Please review a cleanup patch for the OutputPropertiesFactory. The original intention was to remove the (awkward) JDK 1.2-related code, line 235-258 in the old file. I then went a bit further to remove another nuisance that annoyed users with an error "Could not load the property file 'output

Re: RFR(L): 8218628: Add detailed message to NullPointerException describing what is null.

2019-02-13 Thread Mandy Chung
I have to say that the example NPE messages this patch has are like stuffing as much relevant information in the message but they are not developer-friendly as Roger and David has commented. Exposing bytecode-level details is very confusing to Joe's developer while, from your perspective, anythin

RFR: 8217216: Launcher does not defend itself against LD_LIBRARY_PATH_64 (Solaris)

2019-02-13 Thread Henry Jen
Hi, Please review the webrev[1] for 8217216. The fix makes sure on Solaris, when LD_LIBRARY_PATH_64 is set, we setup LD_LIBRARY_PATH based on that value and unset LD_LIBRARY_PATH_64 in the relaunched process. Same approach was used before JDK-6367077, and the override is expected behavior on S

Re: RFR(s): 8212828: (process) Change the Process launch mechanism default on Linux to be posix_spawn

2019-02-13 Thread Thomas Stüfe
Sorry, Roger, I must have messed up the bug id when pushing. Thank you for fixing! This would be a good thing to have a jcheck for. ..Thomas On Wed, Feb 13, 2019 at 4:56 PM Roger Riggs wrote: > Hi Thomas, > > This is a change that should have a release note, can you fill it in: > https://bugs

Re: RFR(s): 8212828: (process) Change the Process launch mechanism default on Linux to be posix_spawn

2019-02-13 Thread Roger Riggs
Hi Thomas, This is a change that should have a release note, can you fill it in: https://bugs.openjdk.java.net/browse/JDK-8218924 Also, I had to do some manual updates to the issues to get the changeset into the history. The issue number should have been 8213192 in the summary lineand on the e

Re: RFR 8213031: (zipfs) Add support for POSIX file permissions

2019-02-13 Thread Alan Bateman
On 12/02/2019 21:57, Langer, Christoph wrote: Hi Alan, all, here comes the next proposal for POSIX support in jdk.zipfs - which hopefully represents the converged solution, at least in its overall design. I don't have time to do a detailed code review right now but I did read the updated propo

RE: RFR(L): 8218628: Add detailed message to NullPointerException describing what is null.

2019-02-13 Thread Lindenmaier, Goetz
Hi Bernd, I think this is a feasible idea, while I'm not sure whether this is really critical information. The stack trace already contains the names of Classes, Inner classes etc.. Field names should be not that more sensible information I guess, if at all. > but maybe as a default? You mean

RE: RFR(L): 8218628: Add detailed message to NullPointerException describing what is null.

2019-02-13 Thread Lindenmaier, Goetz
Hi Mandy, Thanks for supporting my intend of adding the message as such! I'll start implementing this in Java and come back with a webrev in a while. In parallel, I would like to continue discussing the other topics, e.g., the wording of the message. I will probably come up with a separate we