Re: JDK 16 RFR of JDK-8250237: Address use of default constructors in the javax.script package

2020-07-23 Thread Lance @ Oracle
+1 Best, Lance 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 iPad > On Jul 23, 2020, at 5:24 PM, Joe Darcy wrote: > > Hello, > > One class in the javax.script

Re: RFR 8250235: java/lang/invoke/lambda/LambdaFileEncodingSerialization.java should be on the problem list

2020-07-23 Thread Lance @ Oracle
+1 Best, Lance 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 iPad > On Jul 23, 2020, at 4:28 PM, Roger Riggs wrote: > > Please review adding an intermittent test to

Re: RFR: [15,docs] JDK-8247959,doclint errors in NIO code

2020-06-20 Thread Lance @ Oracle
+1 Best, Lance 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 iPad > On Jun 19, 2020, at 11:36 PM, Jonathan Gibbons > wrote: > > Please review some fixes to address

Re: RFR: [15, docs] JDK-8247899, HTML errors and warnings in threadPrimitiveDeprecation.html

2020-06-18 Thread Lance @ Oracle
Looks reasonable Jon Best, Lance 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 iPad > On Jun 18, 2020, at 8:19 PM, Jonathan Gibbons > wrote: > > Please review

Re: RFR [15,docs] JDK-8247880,bad HTML(href==...) in table

2020-06-18 Thread Lance @ Oracle
Looks good Best, Lance 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 iPad > On Jun 18, 2020, at 6:40 PM, Jonathan Gibbons > wrote: > > Please review a trivial docs

Re: RFR: 8244855 : Remove unused "getParent" function from Windows jni_util_md.c

2020-05-12 Thread Lance @ Oracle
+1 Best, Lance 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 iPad > On May 12, 2020, at 3:05 PM, Brent Christian > wrote: > > Hi, > > Please review this change to

Re: RFR: 8237508: Simplify JarFile.isInitializing

2020-01-20 Thread Lance @ Oracle
Looks good Claes Best, Lance 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 iPad > On Jan 20, 2020, at 7:12 AM, Claes Redestad wrote: > > Makes sense to keep even

Re: 8237186: Fix typo in copyright header of java/io/Reader/TransferTo.java

2020-01-14 Thread Lance @ Oracle
Looks ok Brian Best, Lance 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 iPad > On Jan 14, 2020, at 8:51 PM, Brian Burkhalter > wrote: > > Oops: correction. > >

Re: RFR (JDK10/JAXP Doc-only) 8184431: References to @sun.com

2017-12-21 Thread Lance @ Oracle
Hi joe Looks fine Best Lance 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 iPad > On Dec 21, 2017, at 6:48 PM, Joe Wang wrote: > > Hi Lance, >

Re: RFR: 8189331 [testbug] java/io/Serializable/maskSyntheticModifier/MaskSyntheticModifierTest.java failed incorrect serialVersionUID

2017-11-30 Thread Lance @ Oracle
+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 iPad > On Nov 30, 2017, at 10:20 AM, Roger Riggs wrote: > > Please review a test fix to copy

Re: Review Request JDK-8176333: jdeps error message should include a proper MR jar file name

2017-03-07 Thread Lance @ Oracle
+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 iPad > On Mar 7, 2017, at 8:45 PM, Mandy Chung wrote: > >

Re: RFR(xs): 8168096: markup error in "since" element spec of @Deprecated

2016-10-17 Thread Lance @ Oracle
+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 iPad > On Oct 17, 2016, at 7:20 PM, Stuart Marks wrote: > > Hi all, please review this tiny

Re: JDK 9 RFR of JDK-8151798: Mark java/util/TimeZone/Bug6772689.java as intermittently failing and demote to tier2

2016-03-14 Thread Lance @ Oracle
+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 iPad > On Mar 14, 2016, at 5:51 AM, Amy Lu wrote: > > java/util/TimeZone/Bug6772689.java > > This

Re: RFR: JDK-8043329: Wrong variable used in java.util.Collections javadoc code

2016-03-10 Thread Lance @ Oracle
Looks ok 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 iPad > On Mar 10, 2016, at 5:51 AM, Sergey Ustimenko wrote: > > Hi everyone! Could someone

Re: RFR (JAXP): 8072081 : Supplementary characters are rejected in comments

2015-12-09 Thread Lance @ Oracle
Hi joe, Looks ok. Assume the jck runs clean as well? Best Lance 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 iPad > On Dec 9, 2015, at 5:37 PM, huizhe wang

Re: RFR 8143015/9: 5 tests fail with error "Can't find source for class: java.util.stream.OpTestCase"

2015-11-16 Thread Lance @ Oracle
+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 iPad > On Nov 15, 2015, at 10:44 PM, Felix Yang wrote: > > Hi, >please review the following

Re: 8142493: Utility methods to check indexes and ranges doesn't specify behavior when function produces null

2015-11-12 Thread Lance @ Oracle
Looks ok Paul Best Lance 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 iPad > On Nov 12, 2015, at 6:19 AM, Paul Sandoz wrote: > > Hi, > >

Re: [RFR]: 8131334: SAAJ Plugability Layer: using java.util.ServiceLoader

2015-11-12 Thread Lance @ Oracle
Hi, Overall it looks ok, I would however suggest adding a couple of tests to the jdk that run with/out a security manager as a sanity check. Best Lance Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803

Re: RFR [9] 8132478: [tidy] three new warnings from java docs (java.net, javax.annotation)

2015-08-18 Thread Lance @ Oracle
Hi Alexander Looks ok but you will need to make sure the jaxws changes get pushed to the standalone project as well Best, Lance 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

Re: JDK 9 RFR of 8098790: Improve cross references and wording in java.lang.reflect.AnnotatedFoo type

2015-06-15 Thread Lance @ Oracle
Looks fine Joe 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 iPad On Jun 15, 2015, at 8:12 PM, joe darcy joe.da...@oracle.com wrote: Hello, Please review the

Re: JDK 9 RFR of JDK-8086029: Fix doclint reference warnings in org.omg.CORBA

2015-06-08 Thread Lance @ Oracle
+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 iPad On Jun 8, 2015, at 9:27 PM, joe darcy joe.da...@oracle.com wrote: Hello, As an addendum to the

Re: JDK 9 RFR of JDK-8078136: Incorrect figure number in reference to Hacker's Delight book in Long.bitCount() method

2015-05-17 Thread Lance @ Oracle
Go for it joe 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 iPad On May 17, 2015, at 12:53 PM, joe darcy joe.da...@oracle.com wrote: Hello, Please review this

Re: RFR 9: 8048264 : StringBuffer's codePoint methods throw unspecified IndexOutOfBoundsException

2015-04-13 Thread Lance @ Oracle
...@oracle.com wrote: Thanks, Lance. My next stop is the CCC. :) -Brent On 4/13/15 3:53 PM, Lance @ Oracle wrote: Hi Brent, This looks fine You might need to fast track a CCC request even though this is minor Best Lance http://oracle.com/us/design/oracle-email-sig-198324

Re: RFR 9: 8048264 : StringBuffer's codePoint methods throw unspecified IndexOutOfBoundsException

2015-04-13 Thread Lance @ Oracle
Hi Brent, This looks fine You might need to fast track a CCC request even though this is minor Best Lance 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 iPad On Apr

Re: RFR (XXS) : 8050123: Incorrect property name documented in CORBA InputStream API

2015-04-09 Thread Lance @ Oracle
+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 iPad On Apr 9, 2015, at 9:42 AM, Seán Coffey sean.cof...@oracle.com wrote:

Re: Review request for JDK-8051559: JAXP function dom tests conversion

2015-03-31 Thread Lance @ Oracle
Hi frank Can you forward the other review request as I think I thought they were the same and deleted it I will look at this again today Best Lance Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803

Re: JDK 9 RFR of JDK-8075565: Define @intermittent jtreg keyword and mark intermittently failing jdk tests

2015-03-19 Thread Lance @ Oracle
Hi joe, This looks fine based on your proposal. Best Lance 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 iPad On Mar 19, 2015, at 8:12 PM, Joseph D. Darcy

Re: RFR: 8061293: Update javax/xml tests to remove references of jre dir

2015-03-05 Thread Lance @ Oracle
Looks fine frank 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 iPad On Mar 5, 2015, at 2:30 AM, Frank Yuan frank.y...@oracle.com wrote: Hi All Would you like

Re: JDK 9 RFR of JDK-8073952: Spec of j.l.r.Method.toString/toGenericString need to be clarified

2015-02-26 Thread Lance @ Oracle
+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 iPad On Feb 26, 2015, at 8:28 PM, joe darcy joe.da...@oracle.com wrote: Hello, Please review the small changes to

Re: RFR: 8073394: Clock.systemUTC() should return a constant

2015-02-24 Thread Lance @ Oracle
Looks good Daniel. 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 iPad On Feb 24, 2015, at 5:59 AM, Daniel Fuchs daniel.fu...@oracle.com wrote: On 23/02/15 21:40,

Re: Review request: 8073374: Re-examine jdk.xml.ws dependency on java.xml.ws SOAPNamespaceConstants

2015-02-19 Thread Lance @ Oracle
+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 iPad On Feb 19, 2015, at 1:24 PM, Mandy Chung mandy.ch...@oracle.com wrote: This eliminates the dependency from

Re: JDK 9 RFR of JDK-8071959: java.lang.Object uses implicit default constructor

2015-01-29 Thread Lance @ Oracle
+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 iPad On Jan 29, 2015, at 7:02 PM, joe darcy joe.da...@oracle.com wrote: Hello, Please review the patch below to

Re: JDK 9 RFR of 6481080 : (ann) @Deprecated annotation has no effect on packages

2015-01-06 Thread Lance @ Oracle
Hi joe, The update is clear Best Lance 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 iPad On Jan 6, 2015, at 6:12 PM, joe darcy joe.da...@oracle.com wrote: Hello,

Re: RFR: JDK-8051563: Convert JAXP function tests in xslt components: : org.apache.qetest.dtm package, org.apache.qetest.trax package to openjdk

2014-12-30 Thread Lance @ Oracle
Hi Tristan, I will look at this but doubt I will get to this tomorrow Best, Lance 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 iPad On Dec 30, 2014, at 5:28 PM,

Re: RFR: 8067377: My hobby: caning, then then canning, the the can-can

2014-12-13 Thread Lance @ Oracle
Good eyes Daniel! 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 iPad On Dec 13, 2014, at 7:27 AM, Daniel Fuchs daniel.fu...@oracle.com wrote: On 12/12/14 10:34 PM,

Re: JDK 9 RFR of JDK-8066643: (zipfs) Suppress deprecation warnings in jdk.zipfs module

2014-12-08 Thread Lance @ Oracle
+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 iPad On Dec 8, 2014, at 1:24 AM, joe darcy joe.da...@oracle.com wrote: Hello, Please review the simple patch below

Re: JDK 9 RFR of JDK-8066641: Suppress deprecation warnings in jdk.naming module

2014-12-04 Thread Lance @ Oracle
+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 iPad On Dec 4, 2014, at 9:23 PM, joe darcy joe.da...@oracle.com wrote: Hello, Please review the small fix below

Re: RFR 8066188: BaseRowSet default value for escape processing is not correct

2014-11-29 Thread Lance @ Oracle
Thank you Alan. Yes, the spec is silent on thread safety. Once I finish with adding tests, I will look to update the javadocs to clarify this. Best, Lance Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803

Re: RFR(s): 8056313: TEST_BUG: java/util/Timer/NameConstructors.java fails intermittently

2014-11-20 Thread Lance @ Oracle
Look fine Stuart Best Lance 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 iPad On Nov 20, 2014, at 6:51 PM, Stuart Marks stuart.ma...@oracle.com wrote: Hi all,

Re: RFR (XS) 8065096: java.net.Authenticator.theAuthenticator should be properly synchronized

2014-11-17 Thread Lance @ Oracle
Looks fine aleksey 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 iPad On Nov 17, 2014, at 5:35 AM, Aleksey Shipilev aleksey.shipi...@oracle.com wrote: Hi, Please

Re: 8025690: Default FileHandler constructor doesn't throw NullPointerException if pattern is empty and count 1

2014-10-01 Thread Lance @ Oracle
Hi Daniel, The revised changes look fine. Best, Lance 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 iPad On Oct 1, 2014, at 6:00 AM, Daniel Fuchs

Re: RFR: 8054555, javadoc clean-up in java.sql, javax.sql.*

2014-08-07 Thread Lance @ Oracle
Hi joe, Thank you for the quick review. I will change those to {@code Inputstream} Best, Lance 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 iPad On Aug 7, 2014, at

Re: RFR JDK-5077522 : Duration.compare incorrect for some values

2014-06-21 Thread Lance @ Oracle
Agree this is better and cleaner! 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 iPad On Jun 21, 2014, at 4:27 AM, Daniel Fuchs daniel.fu...@oracle.com wrote: Thanks

Re: RFR: 8046389: Add missing @since tag under javax.sql.**

2014-06-16 Thread Lance @ Oracle
This is fine Henry Best Lance 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 iPad On Jun 9, 2014, at 5:39 PM, Henry Jen henry@oracle.com wrote: Hi, Please

Re: 8043958: Remove unused com.sun.pept classes from jdk repository

2014-05-26 Thread Lance @ Oracle
+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 iPad On May 26, 2014, at 7:11 AM, Alan Bateman alan.bate...@oracle.com wrote: As part of modularity motivated

Re: RFR (JAXP): 8037259: Xerces Update: XPointer update

2014-04-11 Thread Lance @ Oracle
Overall it looks ok. I think you will get some unchecked warnings from com/sun/org/apache/xerces/internal/xpointer/XPointerHandler.java Any reason not to address this for your HashMap and ArrayList? Best Lance Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java

Re: RFR: 8035284: (xs) Remove redundant null initialization

2014-04-11 Thread Lance @ Oracle
Looks fine mike 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 iPad On Apr 11, 2014, at 3:22 PM, Mike Duigou mike.dui...@oracle.com wrote: Hello all; This is a simple

Re: RFR (JAXP): 8037259: Xerces Update: XPointer update

2014-04-11 Thread Lance @ Oracle
made a similar fix in ElementSchemePointer.java. If you see anything else, then let me know. http://cr.openjdk.java.net/~dli/8037259/webrev/ Thanks, David On 4/11/2014 12:23 PM, Lance @ Oracle wrote: Overall it looks ok. I think you will get some unchecked warnings from com/sun

Re: JDK 9 RFR of JDK-8039864: Fix fallthrough lint warnings in other libs

2014-04-10 Thread Lance @ Oracle
+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 iPad On Apr 10, 2014, at 1:02 AM, Joe Darcy joe.da...@oracle.com wrote: Hello, I've started looking at cleaning up

Re: Core Libs Dev[9] Review Request for 8039488: tidy warnings cleanup for javax.sql

2014-04-09 Thread Lance @ Oracle
I think this looks ok but a couple of suggestions src/share/classes/javax/sql/rowset/spi/package.html - The URL links at the bottom of the page need updated, please address them as part of this putback if possible src/share/classes/javax/sql/package.html - I would use trade; as part of the

Re: Core Libs Dev[9] Review request for 8039172: Tidy warnings cleanup for java.net, java.math, java.time, java.rmi

2014-04-04 Thread Lance @ Oracle
Looks ok but have one concern/question as to why you added the summary attribute to the table tag as it has been deprecated and believe it is not supported in HTML 5. I would probably not include it Best Lance Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java

Re: Core Libs Dev[9] Review request for 8039172: Tidy warnings cleanup for java.net, java.math, java.time, java.rmi

2014-04-04 Thread Lance @ Oracle
, Thank you for the note; the summaries were removed, please see the updated webrev: http://cr.openjdk.java.net/~yan/8039172/webrev.01/ Regards, Alexander On 04.04.2014 15:16, Lance @ Oracle wrote: Looks ok but have one concern/question as to why you added the summary attribute to the table

Re: Core Libs Dev[9] Review request for 8039041: tidy warnings cleanup for javax.naming

2014-04-03 Thread Lance @ Oracle
Looks fine 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 iPad On Apr 3, 2014, at 5:01 AM, alexander stepanov alexander.v.stepa...@oracle.com wrote: Hello, Could

Re: JDK 9 RFR of JDK-8039038: Fix serial lint warnings in com.sun.jmx.snmp

2014-04-02 Thread Lance @ Oracle
+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 iPad On Apr 2, 2014, at 2:39 AM, Joe Darcy joe.da...@oracle.com wrote: Hello, *fingers crossed* The patch below

Re: Core Libs Dev[9] Review Request for 8038493: tidy warnings cleanup for java.sql

2014-03-28 Thread Lance @ Oracle
Looks fine 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 iPad On Mar 28, 2014, at 6:58 AM, alexander stepanov alexander.v.stepa...@oracle.com wrote: Hello, Could

Re: RFR (JAXP): 8035437: Xerces Update: xml/serialize/DOMSerializerImpl

2014-03-28 Thread Lance @ Oracle
With the change to stringbuilder I am ok with it Best Lance 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 iPad On Mar 26, 2014, at 1:44 PM, Aleksej Efimov

Re: JDK 9 RFR of JDK-8036744: Fix raw lint warnings in java.lang.reflect.WeakCache

2014-03-05 Thread Lance @ Oracle
+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 iPad On Mar 5, 2014, at 7:52 PM, Joe Darcy joe.da...@oracle.com wrote: Hello, Please review the simple patch below

Re: RFR [8023022] Some more typos in javadoc

2014-02-16 Thread Lance @ Oracle
Looks fine 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 iPad On Feb 16, 2014, at 2:07 PM, Ivan Gerasimov ivan.gerasi...@oracle.com wrote: Hello! It was reported

Re: A hole in the serialization spec

2014-02-15 Thread Lance @ Oracle
IMHO, I think that is a tall task. There is such a large base of code that uses it or java.sql.Date, that having warnings for deprecation will cause a lot of angst. That code is not going to migrate quickly, if at all to the new types. Best Lance Lance Andersen| Principal Member of

Re: Time to remove sun.misc.Service?

2014-02-11 Thread Lance @ Oracle
+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 iPad On Feb 11, 2014, at 6:19 AM, Alan Bateman alan.bate...@oracle.com wrote: It was never meant to be used by

Re: RFR (JAXP): 8033980 : Xerces Update: datatype XMLGregorianCalendarImpl and DurationImpl

2014-02-11 Thread Lance @ Oracle
Hi joe It looks like you changed the serialversionuid in Durationimpl, did it get changed incorrectly previously? 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 iPad On

Re: JDK 9 RFR of JDK-8033527: Fix raw type lint warning in sun.nio.ch

2014-02-04 Thread Lance @ Oracle
+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 iPad On Feb 4, 2014, at 1:09 AM, Joe Darcy joe.da...@oracle.com wrote: Hello, Please review this small fix to

Re: RFR java.time cleanup of javadoc and messages

2014-02-03 Thread Lance @ Oracle
+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 iPad On Feb 3, 2014, at 5:43 PM, roger riggs roger.ri...@oracle.com wrote: Please review this group of java.time

Re: RFR: (8031737) CHECK_NULL and CHECK_EXCEPTION macros cleanup

2014-02-01 Thread Lance @ Oracle
Looks fine Which releases are you think of including this in if any besides 9? 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 iPad On Feb 1, 2014, at 1:03 PM, roger riggs

Re: RFR (JAXP): 8027359: XML parser returns incorrect parsing results

2014-01-03 Thread Lance @ Oracle
Looks ok joe 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 iPad On Jan 3, 2014, at 1:07 PM, huizhe wang huizhe.w...@oracle.com wrote: Hi, This is an issue with the

Re: RFR: 8029236: Update copyright year to match last edit in jdk8 jaxp repository for 2013

2013-12-23 Thread Lance @ Oracle
Looks ok assuming they were all modified 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 iPad On Dec 23, 2013, at 12:56 PM, huizhe wang huizhe.w...@oracle.com wrote:

Re: Review Request for 8029216: (jdeps) Provide a specific option to report JDK internal APIs

2013-11-26 Thread Lance @ Oracle
+1 Mandy 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 iPad On Nov 26, 2013, at 6:59 PM, Mandy Chung mandy.ch...@oracle.com wrote: This is a simple patch that adds a

Re: RFR (JAXP) 8028822 : Error in the documentation for newFactory method of the javax.xml.stream factories

2013-11-25 Thread Lance @ Oracle
+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 iPad On Nov 25, 2013, at 4:51 PM, huizhe wang huizhe.w...@oracle.com wrote: Hi, This is a patch to fix an error in

Re: RFR: JDK-8028628 - java/nio/channels/FileChannel/Size.java failed once in the same binary run

2013-11-19 Thread Lance @ Oracle
Looks ok 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 iPad On Nov 19, 2013, at 6:57 PM, Dan Xu dan...@oracle.com wrote: Hi All, Please review the simple fix towards

Re: RFR 8028092 Lint cleanup of java.time.format

2013-11-09 Thread Lance @ Oracle
Looks good roger 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 iPad On Nov 9, 2013, at 2:14 PM, roger riggs roger.ri...@oracle.com wrote: Please review this lint

Re: RFR (JAXP): 8016500 : Unlocalized warnigs.

2013-10-17 Thread Lance @ Oracle
+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 iPad On Oct 17, 2013, at 5:51 PM, huizhe wang huizhe.w...@oracle.com wrote: Hi, This is a fix to use the existing

Re: JDK 8 RFR 8026832: Clean up straggling doclint warnings in java.math

2013-10-17 Thread Lance @ Oracle
+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 iPad On Oct 17, 2013, at 6:00 PM, Brian Burkhalter brian.burkhal...@oracle.com wrote: Please review … Issue:

Re: RFR: JDK-8025686 : (s) Update jdk repo netbeans projects to support NetBeans 7.4 for Java 8 support

2013-10-01 Thread Lance @ Oracle
Looks ok to me as well 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 iPad On Oct 1, 2013, at 5:32 AM, Chris Hegarty chris.hega...@oracle.com wrote: The changes look ok

Re: JDK 8 RFR java.nio doclint fixes

2013-07-20 Thread Lance @ Oracle
Looks fine joe 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 iPad On Jul 20, 2013, at 1:17 PM, Joe Darcy joe.da...@oracle.com wrote: Hello, Please review the latest

Re: JDK 8 code review request for doclint issues in java.lang.instrument

2013-07-01 Thread Lance @ Oracle
Looks good 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 iPad On Jul 1, 2013, at 2:24 PM, Joe Darcy joe.da...@oracle.com wrote: Hello, Yet another found of doclint

Re: JDK 8 code review request for doclint issues in java.util.spi

2013-07-01 Thread Lance @ Oracle
Looks fine 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 iPad On Jul 1, 2013, at 4:35 PM, Joe Darcy joe.da...@oracle.com wrote: Hello, There are a few doclint issues

Re: RFR (JAXP): 8012683 : Some ObjectFactory classes should be removed.

2013-05-20 Thread Lance @ Oracle
+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 iPad On May 20, 2013, at 7:26 PM, huizhe wang huizhe.w...@oracle.com wrote: Hi, This is a quick fix to remove a copy

Re: RFEs implementing JEP 170

2013-05-02 Thread Lance @ Oracle
Yes I am away right now but will follow up when I am back mid next week Best Lance 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 iPad On May 2, 2013, at 11:45 AM, Alan

Re: RFR: 8006036, (process) cleanup code in java/lang/Runtime/exec/WinCommand.java

2013-04-05 Thread Lance @ Oracle
I can look to do this Monday for you for Sunday eve I am away from my primary system right now 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 iPad On Apr 5, 2013, at 5:29

Re: 8011373: Property java.runtime.profile should be removed (left-over code)

2013-04-03 Thread Lance @ Oracle
+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 iPad On Apr 3, 2013, at 7:14 AM, Alan Bateman alan.bate...@oracle.com wrote: I need a reviewer for a trivial change

Re: review request 8006139, add missing methods to javax.sql.rowset.serial.SQLInput/OutputImpl

2013-01-14 Thread Lance @ Oracle
Yes sorry if that was not clear but that is my plan with the ccc and javadoc update I will get the JBS entry and ccc submitted later today and make a few of the minor suggestions from ulf and push out the revised webrev later today or Tuesday Best Lance Lance Andersen| Principal Member of

Re: review request 8006139, add missing methods to javax.sql.rowset.serial.SQLInput/OutputImpl

2013-01-13 Thread Lance @ Oracle
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 iPad On Jan 13, 2013, at 5:56 AM, Ulf Zibis ulf.zi...@cosoco.de wrote: Am 12.01.2013 22:56, schrieb Lance Andersen -

Re: review request 8006139, add missing methods to javax.sql.rowset.serial.SQLInput/OutputImpl

2013-01-13 Thread Lance @ Oracle
Yes I just noticed this The code was written before my time I will add them though 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 iPad On Jan 13, 2013, at 7:28 AM, Ulf