RFR 8220253: Fix Headings in java.sql.rowset

2019-03-12 Thread Lance Andersen
com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.

RFR 8220005: java/util/Arrays/TimSortStackSize2.java times out

2019-03-12 Thread Lance Andersen
gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engi

Re: (trivial doc fix) RFR: 8220262: fix headings in java.logging

2019-03-12 Thread Lance Andersen
+1 -- Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com Sent from my iPhone > On Mar 12, 2019, at 8:06 AM, Daniel Fuchs wrote: > > Hi, > > Please find below a

Re: RFR(jdk 13/java.xml) 8219705: Wrong media-type for a given serialization method

2019-03-11 Thread Lance Andersen
> On Mar 11, 2019, at 3:15 PM, Joe Wang wrote: > > > > On 3/11/19, 11:54 AM, Lance Andersen wrote: >> >> Hi Joe, >> >>> On Mar 11, 2019, at 2:41 PM, Joe Wang >> <mailto:huizhe.w...@oracle.com>> wrote: >>> >>&g

Re: RFR(jdk 13/java.xml) 8219705: Wrong media-type for a given serialization method

2019-03-11 Thread Lance Andersen
ersion: actual: 4.0, expected: 4.0 > indent: actual: yes, expected: yes > media-type: actual: text/html, expected: text/html > > Best, > Joe > > On 3/11/19, 11:31 AM, Lance Andersen wrote: >> >> Hi Joe >> >> I think this is fine. I might consider beefi

Re: RFR(jdk 13/java.xml) 8219705: Wrong media-type for a given serialization method

2019-03-11 Thread Lance Andersen
acle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Netw

Re: JDK 13 RFR of JDK-8220346: Refactor java.lang.Throwable to use Objects.requireNonNull

2019-03-08 Thread Lance Andersen
Suppressed exceptions not > recorded > return; > <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com <mailto:lance.ander...@oracle.com>

Re: [13] RFR: 8218948: SimpleDateFormat :: format - Zone Names are not reflected correctly during run time

2019-03-07 Thread Lance Andersen
nese locales between JDK and CLDR, in terms of > Simplified/Traditional scripts. > > Naoto <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.

Re: Review Request: JDK-8220282 Add MethodHandle tests on accessing final fields

2019-03-07 Thread Lance Andersen
e.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com <mailto:lance.ander...@oracle.com>

Re: [13] RFR: 8219890: Calendar.getDisplayName() returns empty string for new Japanese Era on some locales

2019-02-28 Thread Lance Andersen
p://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com <mailto:lance.ander...@oracle.com>

Re: Manifest ignores last line if not terminated by line break

2019-02-28 Thread Lance Andersen
racle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com <mailto:lance.ander...@oracle.com>

Re: JDK 13 RFR of JDK-8219561: Update jdeprscan to avoid the need for start-of-release changes

2019-02-22 Thread Lance Andersen
gn/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com <mailto:lance.ander...@oracle.com>

Re: Manifest ignores last line if not terminated by line break

2019-02-22 Thread Lance Andersen
> is a test case >> testJARManifestClassPathAttribute/testClassPathAttribute which I have >> no clue how this should really work. As far as I can tell the test is >> and was wrong but nevertheless passes. >> >> Is there a chance to find a sponsor for fixing this? >> >>

RFR: 8219548 Re: Faulty Null-Check Suspected in ToolProvider

2019-02-21 Thread Lance Andersen
t; Regards, > Philipp > > > On Fri, 2019-02-15 at 18:10 -0500, Lance Andersen wrote: >> Hi Philipp >> >> This probably makes sense to update. >> >> Can you also update the ToolProviderTest.java to add a test for the changes >> >> Thank

Re: Faulty Null-Check Suspected in ToolProvider

2019-02-20 Thread Lance Andersen
/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com <mailto:lance.ander...@oracle.com>

Re: Faulty Null-Check Suspected in ToolProvider

2019-02-15 Thread Lance Andersen
acle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive

Re: RFR: 8216363: NullPointerException in java.util.logging.Handler#isLoggable

2019-02-15 Thread Lance Andersen
t; I'd be surprised if there is a JCK test expecting NPE. Will see. >> I suggest to update @param record to say "a LogRecord or null" >> to be explicit. >> Mandy > <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/

Re: RFR: 8216363: NullPointerException in java.util.logging.Handler#isLoggable

2019-02-14 Thread Lance Andersen
Hi Daniel, > On Feb 14, 2019, at 2:22 PM, Daniel Fuchs wrote: > > Hi Lance, > > On 14/02/2019 20:18, Lance Andersen wrote: >> Documenting the current behavior as it has been makes the most sense to me >> as well >> Have you created a CSR yet? if so I will

Re: RFR: 8216363: NullPointerException in java.util.logging.Handler#isLoggable

2019-02-14 Thread Lance Andersen
; <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com <mailto:lance.ander...@oracle.com>

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

2019-02-13 Thread Lance Andersen
see http://openjdk.java.net/jtreg/tag-spec.html for the system properties. I do not see test.dir there. — I would just do: — Path foo = Path.of("test.zip"); System.out.println("test.zip path=" + foo.toAbsolutePath()); -- which results in the

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

2019-02-13 Thread Lance Andersen
quot;zip:permissions" and we'll document this. If a user of zipfs wants >> to see full PosixFileAttributeView support with default values, then we >> should allow for a creation attribute for the zipfs that can control this. >> Maybe >> we can even allow specifying default values for user, group and permissions >> via zipfs attributes. >> >> I'll work to develop the patch into this direction unless you tell me that >> this >> idea is bogus (if so, then I hope it be soon ) >> >> Thanks >> Christoph >> >> > <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com <mailto:lance.ander...@oracle.com>

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

2019-02-13 Thread Lance Andersen
p://cr.openjdk.java.net/~joehw/jdk13/8187697/webrev/ > > Thanks, > Joe <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://

Re: RFR 8182117 : Document Zip File System Properties

2019-02-08 Thread Lance Andersen
/oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff |

Re: RFR 8182117 : Document Zip File System Properties

2019-02-05 Thread Lance Andersen
stoph > <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com <mailto:lance.ander...@oracle.com>

Re: RFR 8182117 : Document Zip File System Properties

2019-02-05 Thread Lance Andersen
Thank you for the feedback. Please see below > On Feb 5, 2019, at 3:31 AM, Alan Bateman wrote: > > A few comments > > On 04/02/2019 19:36, Lance Andersen wrote: >> : >> >> + * The Zip file system provider treats a Zip or JAR file as a file system >>

Re: RFR(JDK 13/java.xml) 8180901: Transformer.reset() resets the state only once

2019-02-05 Thread Lance Andersen
t;http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Techn

Re: RFR 8182117 : Document Zip File System Properties

2019-02-04 Thread Lance Andersen
- > On Feb 4, 2019, at 1:50 PM, Roger Riggs wrote: > > Hi Lance, > > That looks ok; editorial suggestions below: > > > On 02/04/2019 12:40 PM, Lance Andersen wrote: >> Hi all >> >> Please review the fix for https://bugs.openjdk.java.net/browse

Re: RFR(JDK 13/java.xml) 8206132: DOM parser does not honor DocumentBuilderFactory.setExpandEntityReferences(false)

2019-02-04 Thread Lance Andersen
~joehw/jdk13/8206132/webrev/ >>> >>> Thanks, >>> Joe > <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com <mailto:lance.ander...@oracle.com>

Re: [13]: RFR: 8218386: Correct the SE version in j.l.Character

2019-02-04 Thread Lance Andersen
gt; > Naoto <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Princi

RFR 8182117 : Document Zip File System Properties

2019-02-04 Thread Lance Andersen
stemProvider * @moduleGraph * @since 9 $ <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle

Re: RFR (JDK 13/java.xml) 8186321: Cleanup: SourceTreeManager not used

2019-01-31 Thread Lance Andersen
Thanks, > Joe > <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal M

Re: RFR [13] 8218022: Repeated words typos in java.base (was: Duplicate words typos in comments/javadoc/strings)

2019-01-29 Thread Lance Andersen
AlgorithmChecker, > * and DisabledAlgorithmConstraints. > */ > public class ConstraintsParameters { > <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig

Re: Duplicate words typos in comments/javadoc/strings

2019-01-28 Thread Lance Andersen
t; - * OK/CANCEL confirmation confirmation option. > + * OK/CANCEL confirmation option. > * > * An underlying security service specifies this as the > * {@code optionType} to a {@code ConfirmationCallback} > Index: src/java.base/share/classes/jdk/internal/

Re: RFR (JDK 13/java.xml) 8216408: XMLStreamWriter setDefaultNamespace(null) throws NullPointerException

2019-01-25 Thread Lance Andersen
JBS: https://bugs.openjdk.java.net/browse/JDK-8216408 >>> webrevs: http://cr.openjdk.java.net/~joehw/jdk13/8216408/webrev/ >>> >>> Thanks, >>> Joe >>> >> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/

Re: RFR: JDK-8217393 Re: Clarification in Attributes equal

2019-01-25 Thread Lance Andersen
represent the same mappings. * * @param o the Object to be compared * @return true if the specified Object is equal to this Map - > On Jan 25, 2019, at 2:53 PM, Lance Andersen wrote: > > Hi Joe > > yes I checked it in my local build and it does

Re: RFR: JDK-8217393 Re: Clarification in Attributes equal

2019-01-25 Thread Lance Andersen
> On 1/25/2019 11:44 AM, Lance Andersen wrote: >> Thank you Joe. >> >> So the change is (see bolded change): >> >> $ hg diff >> diff -r 6130409b923e >> src/java.base/share/classes/java/util/jar/Attributes.java >> --- a/src/java.base/share/cl

Re: RFR: JDK-8217393 Re: Clarification in Attributes equal

2019-01-25 Thread Lance Andersen
:24 AM, Roger Riggs wrote: >> Looks fine, Lance >> >> Roger >> >> On 01/25/2019 02:22 PM, Lance Andersen wrote: >>> The CSR review suggested a slight update to the proposed wording: >>> >>> ——— >>> $ hg diff >>>

Re: RFR: JDK-8217393 Re: Clarification in Attributes equal

2019-01-25 Thread Lance Andersen
* @return true if the specified Object is equal to this Map */ — Best Lance > On Jan 22, 2019, at 7:47 PM, Lance Andersen wrote: > >> >> On Jan 22, 2019, at 12:02 PM, Alan Bateman wrote: >> >> On 19/01/2019 12:46, Lance Andersen wrote: >

Re: RFR (JDK 13/java.xml) 8216408: XMLStreamWriter setDefaultNamespace(null) throws NullPointerException

2019-01-25 Thread Lance Andersen
/~joehw/jdk13/8216408/webrev/ > > Thanks, > Joe > <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-

Re: RFR: 8210469: Missing doPriviledged block and permission for jdk.zipfs module

2019-01-25 Thread Lance Andersen
> On Jan 25, 2019, at 3:55 AM, Alan Bateman wrote: > > > > On 25/01/2019 00:11, Lance Andersen wrote: >>> On Jan 24, 2019, at 6:20 PM, Mandy Chung wrote: >>> >>> >>> >>> On 1/24/19 1:55 PM,

Re: RFR: 8210469: Missing doPriviledged block and permission for jdk.zipfs module

2019-01-24 Thread Lance Andersen
> On Jan 24, 2019, at 6:20 PM, Mandy Chung wrote: > > > > On 1/24/19 1:55 PM, Lance Andersen wrote: >> >>> Alternatively you can simply do: >>> PrivilegedExceptionAction pea = () -> >>> tempPath.toRealPath(); >&

Re: RFR: 8210469: Missing doPriviledged block and permission for jdk.zipfs module

2019-01-24 Thread Lance Andersen
Hi Mandy, > On Jan 24, 2019, at 4:50 PM, Mandy Chung wrote: > > > > On 1/24/19 12:48 PM, Lance Andersen wrote: >> Hi all, >> >> Please review the following patch which addresses >> https://bugs.openjdk.java.net/browse/JDK-8210469 >> <https:/

Re: RFR: 8210469: Missing doPriviledged block and permission for jdk.zipfs module

2019-01-24 Thread Lance Andersen
empPath.toRealPath() > 327 , null, > 328 new java.util.PropertyPermission("user.dir", "read")); > thanks for catching that, I agree, it should not be there, I think I just saw what I wanted to see. :-) I updated the webrev in place as there is no other change Best La

RFR: 8210469: Missing doPriviledged block and permission for jdk.zipfs module

2019-01-24 Thread Lance Andersen
-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com <mailto:lance.ander...@oracle.com>

Re: [testbug] RFR: 8217353: java/util/logging/LogManager/Configuration/updateConfiguration/HandlersOnComplexResetUpdate.java fails with Unexpected reference: java.lang.ref.WeakReference

2019-01-24 Thread Lance Andersen
00. > (webrev updated in place). > http://cr.openjdk.java.net/~dfuchs/webrev_8217353/webrev.00/ > > Hopefully that will reduce the frequency of failures. > Re testing in progress... > > best regards, > > -- daniel > > On 24/01/2019 12:14, Lance Andersen wrote: >&

Re: [testbug] RFR: 8217353: java/util/logging/LogManager/Configuration/updateConfiguration/HandlersOnComplexResetUpdate.java fails with Unexpected reference: java.lang.ref.WeakReference

2019-01-24 Thread Lance Andersen
il-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com <mailto:lance.ander...@oracle.com>

Re: RFR: JDK-8217393 Re: Clarification in Attributes equal

2019-01-22 Thread Lance Andersen
> On Jan 22, 2019, at 12:02 PM, Alan Bateman wrote: > > On 19/01/2019 12:46, Lance Andersen wrote: >> Hi all, >> >> Please review the fix for JDK-8217393 which updates the javadocs for >> Attriibutes::equals to clarify its behavior to match its implementatio

Re: Manifest copy constructor does not deeply copy individual section Attributes

2019-01-21 Thread Lance Andersen
e >> attached patch. >> >> Regards, >> Philipp > <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.

RFR: JDK-8217393 Re: Clarification in Attributes equal

2019-01-19 Thread Lance Andersen
-value mappings. * * @param o the Object to be compared * @return true if the specified Object is equal to this Map ——— Best, Lance > On Jan 17, 2019, at 6:50 PM, Lance Andersen wrote: > > Hi Philipp, > > I suspect the behavior dates back to the original

Re: Review Request JDK-8213932: [TESTBUG] assertEquals is invoked with the arguments in the wrong order

2019-01-18 Thread Lance Andersen
l-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com <mailto:lance.ander...@oracle.com>

Re: Clarification in Attributes equal

2019-01-17 Thread Lance Andersen
if> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com <mailto:lance.ander...@oracle.com>

Re: RFR [test] 8217340: Compilation failed: tools/launcher/Test7029048.java

2019-01-17 Thread Lance Andersen
tp://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Tec

Re: RFR 8211919: ZipDirectoryStream should provide a stream of paths that are relative to the directory

2019-01-15 Thread Lance Andersen
> On Jan 15, 2019, at 3:55 PM, Alan Bateman wrote: > > > > On 15/01/2019 18:59, Lance Andersen wrote: >> >> OK thank you. I made a ‘minor update to the comments and removed “4" > Looks good. I also skimmed the update test and it looks okay too except s

Re: RFR 8202675 : Replace process-wide terminology in serial filtering to be consistent

2019-01-15 Thread Lance Andersen
<http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com <mailto:lance.ander...@oracle.com>

Re: RFR 8211919: ZipDirectoryStream should provide a stream of paths that are relative to the directory

2019-01-15 Thread Lance Andersen
Lance > > Best regards > Christoph > <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324

Re: RFR 8211919: ZipDirectoryStream should provide a stream of paths that are relative to the directory

2019-01-15 Thread Lance Andersen
> On Jan 15, 2019, at 4:07 AM, Alan Bateman wrote: > > On 15/01/2019 00:13, Lance Andersen wrote: >> : >> >> Thank you Alan pointing out this example which the previous fix also did not >> address. I updated the change which addresses the above example

Re: RFR 8211919: ZipDirectoryStream should provide a stream of paths that are relative to the directory

2019-01-14 Thread Lance Andersen
> On Jan 14, 2019, at 2:05 PM, Alan Bateman wrote: > > On 12/01/2019 20:13, Lance Andersen wrote: >> Hi all, >> >> The following patch addresses the issue where the stream of paths where not >> relative to the specified directory as needed. >> &g

Re: RFR 8080569: (process) java/lang/ProcessBuilder/DestroyTest.java fails with "Process terminated prematurely"

2019-01-14 Thread Lance Andersen
net/~rriggs/webrev-destroytest-8080569-2/ >>> >>> Thanks, Roger >>> >>> [1] https://bugs.openjdk.java.net/browse/JDK-8080569 >>> [2] https://bugs.openjdk.java.net/browse/JDK-8216990 >>> > <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com <mailto:lance.ander...@oracle.com>

Re: RFR 8080569: (process) java/lang/ProcessBuilder/DestroyTest.java fails with "Process terminated prematurely"

2019-01-14 Thread Lance Andersen
anks, Roger > > [1] https://bugs.openjdk.java.net/browse/JDK-8080569 > [2] https://bugs.openjdk.java.net/browse/JDK-8216990 > <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracl

Re: RFR 8211919: ZipDirectoryStream should provide a stream of paths that are relative to the directory

2019-01-14 Thread Lance Andersen
rather be: if (ds.iterator().hasNext()) throw… The check is fine as it is but I changed it to make it clearer similar to DirectoryStream/Basic.java Again, thank you for your feedback. Updated webrev is here http://cr.openjdk.java.net/~lancea/8211919/webrev.01/index.html Best Lan

RFR 8211919: ZipDirectoryStream should provide a stream of paths that are relative to the directory

2019-01-12 Thread Lance Andersen
://cr.openjdk.java.net/~lancea/8211919/webrev.00/ Best Lance <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-1983

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

2019-01-12 Thread Lance Andersen
ave to resort to >>>> throwing an IOE? >>>> >>> Maybe although it would be a bit awkward to deal with. The issues around >>> this remind me a bit about mounting fat32 file systems on Linux or Unix >>> systems where the fields in the stat structure are populated with >>> default values. PosixFileAttributeView::readAttributes is essentially >>> the equivalent of a stat call. This might be something to look at for >>> the file owner at least. >>> >>> -Alan <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com <mailto:lance.ander...@oracle.com>

Re: [13] 8216172: File.renameTo(File dest) should check for NPE at the very beginning

2019-01-10 Thread Lance Andersen
did not know it would be worthwhile. I would think do none or do > all. > > Thanks, > > Brian > >> On Jan 10, 2019, at 2:20 PM, Lance Andersen > <mailto:lance.ander...@oracle.com>> wrote: >> >> Looks fine Brian. >> >> One question would

Re: [13] 8216172: File.renameTo(File dest) should check for NPE at the very beginning

2019-01-10 Thread Lance Andersen
sInvalid()) { > return false; > } > <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/ora

Re: [RFR] 8216362: Incorrect jar file error message when there is an invalid manifest

2019-01-09 Thread Lance Andersen
he end of the string > on line 321 to be consistent with the other messages. I will fix before I push > > > On Wed, 2019-01-09 at 15:42 -0500, Lance Andersen wrote: >> Here is the webrev for the changes: >> >> http://cr.openjdk.java.net/~lancea/8216362/webrev.

Re: [RFR] 8216362: Incorrect jar file error message when there is an invalid manifest

2019-01-09 Thread Lance Andersen
<http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com <mailto:lance.ander...@oracle.com>

Re: [RFR] 8216362: Incorrect jar file error message when there is an invalid manifest

2019-01-08 Thread Lance Andersen
esystem. >> >> --Sean > <8216362.patch> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/desig

Re: [PATCH] for error message not containing file name of jar with bad manifest

2019-01-08 Thread Lance Andersen
else to create one and I don't have the > slightest idea how to get such a privilege. Could you give me a hint how to > proceed? > > Philipp > > On Mon, 2019-01-07 at 18:05 -0500, Lance Andersen wrote: >> Hi Philipp, >> >> I would like to suggest a ne

Re: [PATCH] for error message not containing file name of jar with bad manifest

2019-01-07 Thread Lance Andersen
m/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burling

Re: RFR 8216205: Java API documentation formatting error in System.getEnv

2019-01-07 Thread Lance Andersen
s, its >>>>* {@link SecurityManager#checkPermission checkPermission} >>>>* method is called with a >>>> - * {@code {@link RuntimePermission}("getenv.*")} permission. >>>> + * {@link RuntimePermission Runtime

Re: RFR 8216205: Java API documentation formatting error in System.getEnv

2019-01-07 Thread Lance Andersen
urityException} being thrown. > * > * When passing information to a Java subprocess, > > > Thanks, Roger <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-e

Re: 8215798: [javadoc] Use {@systemProperty} for org.openjdk.java.util.stream.tripwire system property

2019-01-04 Thread Lance Andersen
primitive subtype > specializations. > * > > Thanks, Roger <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracl

Re: RFR(JDK 13/java.xml) javax.xml.catalog.CatalogResolverImpl: GroupEntry.matchURI fails to match

2019-01-03 Thread Lance Andersen
324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com <mailto:lance.ander...@oracle.com>

Re: 8216134 (process) ProcessBuilder startPipeline does not hide piped streams

2019-01-03 Thread Lance Andersen
<http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com <mailto:lance.ander...@oracle.com>

Re: RFR: 8182992 Typo in DatagramPacket constructor API doc

2019-01-03 Thread Lance Andersen
er >* marker in the SQL statement >* >* @throws SQLFeatureNotSupportedException if the JDBC driver does not > support this method > > looks fine otherwise <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/o

Re: [12] RFR: 8214567 and 8214569: Use {@systemProperty} for definitions of system properties

2018-12-25 Thread Lance Andersen
ating line 1867 in ClassLoader.java to avoid that > single long line sticking out would be good. > > Mandy <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-19832

Re: RFR: JDK-8066619: String(byte[],int,int,int) in String has been deprecated in Manifest and Attributes

2018-12-20 Thread Lance Andersen
does not yet have access to cr.openjdk.java.net. > > $.02, Roger > > On 12/20/2018 09:17 AM, Lance Andersen wrote: >> Hi Philipp, >> >> The tests look better. Any chance you can post the webrev in the future vs >> the patch as it makes it easer to review. >> >

Re: RFR: JDK-8066619: String(byte[],int,int,int) in String has been deprecated in Manifest and Attributes

2018-12-20 Thread Lance Andersen
8 -0800, Martin Buchholz wrote: >> On Tue, Dec 18, 2018 at 1:13 PM Lance Andersen > <mailto:lance.ander...@oracle.com>> >> wrote: >> >>> >>> >>> >>> Is there a reason you did not use >>> >>> @Test(

Re: RFR: JDK-8066619: String(byte[],int,int,int) in String has been deprecated in Manifest and Attributes

2018-12-18 Thread Lance Andersen
work. >>> >>> >>> In the test ValueUtf8Coding, just a mention of a method to create a string >>> with repeats. >>> "-".repeat(80); >>> > <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://

Re: RFR (S): 8215472: Cleanups in implementation classes of jdk.zipfs and tests

2018-12-17 Thread Lance Andersen
e.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com <mailto:lance.ander...@oracle.com>

RFR 8215372: test/jdk/java/nio/file/DirectoryStream/Basic.java not correct for validating the use of a glob

2018-12-13 Thread Lance Andersen
ream ds = newDirectoryStream(dir, filter)) { $ -- <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/

Re: RFR: JDK-8215217: OpenJDK Source Has Too Many Swear Words

2018-12-13 Thread Lance Andersen
-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com <mailto:lance.ander...@oracle.com>

Re: RFR 8215309 : Convert package.html files to package-info.java files

2018-12-12 Thread Lance Andersen
e.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com <mailto:lance.ander...@oracle.com>

Re: 8214696: Module class should be filtered by core reflection

2018-12-12 Thread Lance Andersen
design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com <mailto:lance.ander...@oracle.com>

Re: Review Request JDK-8215238: (jdeps) update jdk8_internals.txt per the removal of javafx, corba, EE modules

2018-12-11 Thread Lance Andersen
8324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering

Re: RFR 8214971 : Replace use of string.equals("") with isEmpty()

2018-12-06 Thread Lance Andersen
agement.agent >>>>jdk.naming.dns >>>>jdk.rmic >>>> >>>> >>>> Webrev: >>>> http://cr.openjdk.java.net/~rriggs/webrev-string-isempty-8214971-1/ >>>> >>>> Issue: >>>> https://bugs.openjdk.java.net/browse/JDK-8214971 >>>> >>>> Thanks, Roger >>> >> > <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com <mailto:lance.ander...@oracle.com>

Re: RFR 8214971 : Replace use of string.equals("") with isEmpty()

2018-12-06 Thread Lance Andersen
s, Roger <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Pri

Re: RFR 8214498 : java/util/Locale/bcp47u/SystemPropertyTests.java wrong locale default

2018-12-05 Thread Lance Andersen
browse/JDK-8214498 > > Thanks, Roger > <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198

Re: [12] RFR: 8214770: java/time/test/java/time/format/TestNonIsoFormatter.java failed in non-english locales.

2018-12-04 Thread Lance Andersen
m/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com <mailto:lance.ander...@oracle.com>

Re: RFR: XXXS: JDK-8214745: Bad link in coll-reference.html

2018-12-03 Thread Lance Andersen
t. > > <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Princ

Re: RFR: [XXS] 8214744: Unnecessary tags in java.util.zip.Deflater

2018-12-03 Thread Lance Andersen
* is required. > - * > * @param input the input data bytes > * @see Deflater#needsInput > */ > <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/orac

Re: RFR(JDK 12/java.xml) 8213734: SAXParser.parse(File, ..) does not close resources when Exception occurs.

2018-11-29 Thread Lance Andersen
.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com <mailto:lance.ander...@oracle.com>

Re: [test] 8214445 : java/net/URL/HandlerLoop has illegal reflective access

2018-11-28 Thread Lance Andersen
/oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com <mailto:lance.ander...@oracle.com>

Re: RFR(JDK 12/java.xml) 8213117: adoptNode corrupts attribute values

2018-11-28 Thread Lance Andersen
24.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com <mailto:lance.ander...@oracle.com>

Re: RFR 8214014 : Remove vestiges of gopher: protocol proxy support

2018-11-28 Thread Lance Andersen
is patch assumes a previous patch cleaning up property > initialization (4947890) > <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://o

Re: RFR: JDK-8197398 (zipfs) OutOfMemoryError when talking contents of empty JAR file

2018-11-27 Thread Lance Andersen
r jdk 12 handle a case where we can update an > existing entry in jar with "/" (absolute path). As in case of jdk 8 we > cannot update the existing zipentry as doing so we get > "FileAlreadyExistsException". Therefore, fix for jdk 8u does not consider > this case. >

Re: RFR (JDK 12/java.xml) 8177286: AttributeSet: attempt to compare Qname and String

2018-11-26 Thread Lance Andersen
oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com <mailto:lance.ander...@oracle.com>

RFR: 8210454 jar tool does not allow setting the module version without also setting the main class

2018-11-26 Thread Lance Andersen
/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com <mailto:lance.ander...@oracle.com>

Re: RFR: JDK-8214223: tools/jdeps/listdeps/ListModuleDeps.java failed due to missing Lib2 file

2018-11-22 Thread Lance Andersen
lic static long getPid() { > +return ManagementFactory.getRuntimeMXBean().getPid(); > +} > +} <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-19

Re: 8214195: Align stdout messages in test/jdk/java/math/BigInteger/PrimitiveConversionTests.java

2018-11-22 Thread Lance Andersen
the same. > > Thanks, > > Brian <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gi

<    4   5   6   7   8   9   10   11   12   13   >