proxy for I1, the generated proxy class includes an
>> instance method named "foo" and no static method is generated in the
>> proxy class.
>>
>> Mandy
>>
>> On 10/2/17 11:44 AM, Eric McCorkle wrote:
>>> Hello everyone,
>>>
>>&g
Hello everyone,
A colleague of mine discovered what seems to be a bug in
java.lang.reflect.Proxy#newProxyInstance. I'd like to confirm that this
is indeed incorrect behavior before I go and fix it.
Consider the following two interfaces:
interface I1 {
static I1 foo() {
return null;
}
}
line breaks), you can extract a test method since the test just
> operates on two arrays of Type[] and call it with the parameter type
> array and the answer array.
>
> cheers
> /Joel
>
> On 2014-11-05, Eric McCorkle wrote:
>> Thanks, Joel. I've applied your fixes a
Are there any more comments on this patch? This is an 8u40 issue, so it
needs to be committed soon.
Thanks,
Eric
On 11/05/14 16:57, Eric McCorkle wrote:
> A word on the tests: I don't think there's a stable way to check the
> types on getParameterizedType() and getAnnotatedType(
gnature (or non at all). This is a discussion we can
> defer however.
>
> If you flesh out the test and remove the caching I think this is a fine fix
> for both 8u and 9.
>
> cheers
> /Joel
>
> On 31 okt 2014, at 17:15, Eric McCorkle wrote:
>
>> Hello,
>&
ver.
>
> If you flesh out the test and remove the caching I think this is a fine fix
> for both 8u and 9.
>
> cheers
> /Joel
>
> On 31 okt 2014, at 17:15, Eric McCorkle wrote:
>
>> Hello,
>>
>> Please review this patch which fixes issues that aris
This issue is an 8u40 issue, so it needs to be reviewed ASAP in order to
backport it in time.
On 10/31/14 12:15, Eric McCorkle wrote:
> Hello,
>
> Please review this patch which fixes issues that arise with
> getGenericParameterTypes() and getAnnotatedParameterTypes() when there
&
ssing top-level
files; however, you can still look at the individual files just fine.
On 11/02/14 23:50, David Holmes wrote:
> Hi Erik,
>
> webrevs still broken for some reason.
> On 1/11/2014 12:01 AM, Eric McCorkle wrote:
>> I went through and added comments in the binary data
Hello,
Please review this patch which fixes issues that arise with
getGenericParameterTypes() and getAnnotatedParameterTypes() when there
are generic signatures and synthetic parameters.
Please note that a complete fix is not possible for all cases. See
discussion on https://bugs.openjdk.java.ne
you get an unsigned/signed byte
conversion problem), then modify the data by hand. The intent is to
replace this with a better method at some point.
On 10/30/14 21:59, David Holmes wrote:
> Hi Erik,
>
> On 31/10/2014 9:41 AM, Eric McCorkle wrote:
>> Hello,
>>
>> Please
Hello,
Please review this patch which adds tests to the JDK test suite for two
reflection bugs that require hotspot changes (JDK-8058322 and JDK-8058313)
The webrev is here:
http://cr.openjdk.java.net/~emc/8062556/
Pushed. You may now resume normal integration activities.
On 05/09/14 15:12, Staffan Larsen wrote:
> Looks good.
>
> Many apologies.
>
> /Staffan
>
>
> On 9 maj 2014, at 21:08, Eric McCorkle wrote:
>
>> The following patch will fix it:
>>
>> d
t (c) 2005, 2014, Oracle and/or its affiliates. All rights
reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
On 05/09/14 15:01, Eric McCorkle wrote:
> The problem is a stray '' as the first two characters in the file.
>
> I have posted a patch that rem
.
> This is also blocking this week hotspot snapshot:
> http://prt-web.us.oracle.com//archive/2014/05/2014-05-09-174238.amurillo.jdk9-hs-2014-05-09-jdk9-dev-control/logs/solaris_sparcv9_5.10-fastdebug.log.FAILED.log
>
> please fix ASAP
>
> Thanks
> Alejandro
>
> On 5/9
attach framework
On 05/09/14 14:08, Eric McCorkle wrote:
> I am currently seeing build failures on solaris, coming from within the
> JDK repo:
>
> /opt/jprt/products/P1/SS12u1/SS12u1/prod/bin/cc -DTRACING
> -DMACRO_MEMSYS_OPS -DBREAKPTS -D_BIG_ENDIAN= -DSOLARIS
> -DARCH='"
I am currently seeing build failures on solaris, coming from within the
JDK repo:
/opt/jprt/products/P1/SS12u1/SS12u1/prod/bin/cc -DTRACING
-DMACRO_MEMSYS_OPS -DBREAKPTS -D_BIG_ENDIAN= -DSOLARIS
-DARCH='"sparcv9"' -Dsparcv9 -DNDEBUG -DTRIMMED
-DRELEASE='"1.9.0-internal"'
-I/tmp/jprt/P1/173806.emcc
I've been doing some upgrades on servers I run at home recently. One of
the upgrades I'd planned was to increase the key size of my internal CA
key and SSL keys to 8192 bits as a future-proofing measure (I use SSL
with client certificates for all service-to-service communication).
What I found wa
Changeset: f684c9773858
Author:vlivanov
Date: 2014-01-31 21:07 +0400
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/f684c9773858
8033278: Missed access checks for Lookup.unreflect* after 8032585
Reviewed-by: jrose, twisti
! src/share/classes/sun/invoke/util/VerifyAccess.java
! tes
Changeset: 11b83c913cca
Author:attila
Date: 2014-01-30 20:14 +0100
URL: http://hg.openjdk.java.net/jdk8/tl/nashorn/rev/11b83c913cca
8032681: Issues with Nashorn
Reviewed-by: ahgross, jlaskey, sundar
+ src/jdk/internal/dynalink/linker/GuardedTypeConversion.java
! src/jdk/internal/dy
Changeset: 56d05f260123
Author:vlivanov
Date: 2014-01-28 13:46 +0400
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/56d05f260123
8032585: JSR292: IllegalAccessError when attempting to invoke protected method
from different package
Reviewed-by: twisti, jrose
! src/share/classes/su
Changeset: 9e91793fd516
Author:vlivanov
Date: 2014-01-15 20:48 +0400
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/9e91793fd516
8031502: JSR292: IncompatibleClassChangeError in LambdaForm for
CharSequence.toString() method handle type converter
Reviewed-by: sundar, lagergren, drc
Changeset: 24eaf41a3974
Author:emc
Date: 2013-11-14 12:32 -0500
URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/24eaf41a3974
8028282: Remove @ignore from test langtools/test/tools/javac/T7042623.java
Summary: Remove @ignore from test
Reviewed-by: jjg
! test/tools/javac/T70426
Changeset: ea91826bc2e9
Author:emc
Date: 2013-11-13 15:48 -0500
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/ea91826bc2e9
8026884: test for fix of JDK-8021398 does not have @bug tag
8028021: @since 1.8 missing for certain methods in java.lang.reflect.Method in
generated api docs
Hello,
Please review this trivial fix that adds @since documentation tags to
changes added in support of parameter reflection.
Webrev here:
http://cr.openjdk.java.net/~emc/8028021/
Thanks,
Eric
Changeset: 4788eb38cac5
Author:emc
Date: 2013-11-11 09:47 -0500
URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/4788eb38cac5
8027439: Compile-time error in the case of ((Integer[] & Serializable)new
Integer[1]).getClass()
8027253: javac illegally accepts array as bound
Summar
Changeset: 119747cd9f25
Author:emc
Date: 2013-10-24 01:27 -0400
URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/119747cd9f25
8023682: Incorrect attributes emitted for anonymous class declaration
Summary: Cause javac to emit type annotations on new instruction as well as
anony
Changeset: d2fa3f7e964e
Author:emc
Date: 2013-10-23 23:20 -0400
URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/d2fa3f7e964e
8006732: support correct bytecode storage of type annotations in multicatch
Summary: Fix issue with annotations being added before attribution, which
c
Changeset: 23b124cbf242
Author:drchase
Date: 2013-10-22 12:57 -0400
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/23b124cbf242
8026818: Defmeth failures with -mode invoke
Summary: Added test for IllegalAccessException -> IllegalAccessError path to
check if root cause was Abstract
Changeset: 0a6730b5e192
Author:drchase
Date: 2013-10-16 17:55 -0400
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/0a6730b5e192
8022718: Runtime accessibility checking: protected class, if extended, should
be accessible from another package
Summary: Modify accessibility check; it
Changeset: 7f6481e5fe3a
Author:emc
Date: 2013-10-16 16:33 -0400
URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/7f6481e5fe3a
8026286: Improper locking of annotation queues causes assertion failures.
8026063: Calls to annotate.flush() cause incorrect type annotations to be
gen
Changeset: b1b4a6dcc282
Author:emc
Date: 2013-10-10 20:12 -0400
URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/b1b4a6dcc282
8008762: Type annotation on inner class in anonymous class show up as regular
type annotations
8015257: type annotation with TYPE_USE and FIELD attribu
Changeset: 4abfcbac1cb6
Author:emc
Date: 2013-10-10 18:56 -0400
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/4abfcbac1cb6
8026011: java.lang.reflect.MalformedParametersException introduces doclint
warnings
Summary: Add javadoc comments to members of MalformedParametersException
Hello,
Please review this simple patch that adds javadoc comments to
MalformedParametersException to quiet doclint warnings.
Note this patch involves no code changes.
The bug report is here:
https://bugs.openjdk.java.net/browse/JDK-8026011
The webrev is here:
http://cr.openjdk.java.net/~emc/802
Changeset: 6dcf94e32a3a
Author:emc
Date: 2013-10-10 13:55 -0400
URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/6dcf94e32a3a
8019461: Clean up javac diagnostics
7196553: Review error messages for repeating annotations
Summary: Changes to the diagnostic messages to improve clar
Changeset: cb8946eda85b
Author:emc
Date: 2013-10-02 19:13 -0400
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/cb8946eda85b
8020981: Update methods of java.lang.reflect.Parameter to throw correct
exceptions
Summary: Fix behavior of parameter reflection API for malformed class file
nt.
>
> Cheers,
>
> -Joe
>
> On 10/2/2013 11:55 AM, Eric McCorkle wrote:
>> I've updated the test, switched to an in-memory class loader, and added
>> a test case. Please review.
>>
>> On 10/01/13 16:27, Eric McCorkle wrote:
>>> On 10/0
I've updated the test, switched to an in-memory class loader, and added
a test case. Please review.
On 10/01/13 16:27, Eric McCorkle wrote:
> On 10/01/13 02:41, Joe Darcy wrote:
>
> (Suggested changes have been applied)
>
>>
>> I think the test is acceptable as-
Changeset: 5a7bd9825c01
Author:vlivanov
Date: 2013-09-23 19:51 +0400
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/5a7bd9825c01
8001107: @Stable annotation for constant folding of lazily evaluated variables
Reviewed-by: twisti, kvn, rbackman
Contributed-by: john.r.r...@oracle.com,
Changeset: de1c5dbe6c28
Author:emc
Date: 2013-10-01 17:41 -0400
URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/de1c5dbe6c28
8021339: Compile-time error during casting array to intersection
Summary: Add ability to have arrays in intersection types.
Reviewed-by: jjg, vromero
!
Changeset: bf1118ab775b
Author:emc
Date: 2013-10-01 17:35 -0400
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/bf1118ab775b
8021398: j.l.r.Parameter.getAnnotatedType().getType() for not annotated use of
type returns null
Summary: Fixed issue with type annotation reflection framewo
Changeset: 1ed675532589
Author:vlivanov
Date: 2013-09-18 20:12 +0400
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/1ed675532589
8024616: JSR292: lazily initialize core NamedFunctions used for bootstrapping
Reviewed-by: jrose
! src/share/classes/java/lang/invoke/DirectMethodHandle
> candidates.toArray(new TypeAnnotation[0]),
> 283
> (AnnotatedElement)decl);
> 284 }
>
> Paul.
>
> On Oct 1, 2013, at 8:18 PM, Eric McCorkle wrote:
>
file system).
>
Better yet: I've filed an RFE for a framework for generating bad class
files.
Webrev has been updated, please review:
http://cr.openjdk.java.net/~emc/8020981/
> Cheers,
>
> -Joe
>
> On 9/24/2013 12:15 PM, Eric McCorkle wrote:
>> Webrev updated to address th
Thanks all, committing after successful tests.
On 10/01/13 14:33, Joe Darcy wrote:
> Looks fine; cheers,
>
> -Joe
>
> On 10/1/2013 11:18 AM, Eric McCorkle wrote:
>> I forgot to hg add the test. I've addressed your and others' comments,
>> and refreshed th
I forgot to hg add the test. I've addressed your and others' comments,
and refreshed the webrev. Please review.
On 10/01/13 04:06, Joel Borggren-Franck wrote:
> Hi Eric,
>
> Thanks for fixing this.
>
> On 2013-10-01, Eric McCorkle wrote:
>> Hello, please review
Hello, please review this simple patch which fixes a problem in the type
annotations handling API. This manifests as a problem with both
j.l.r.Parameter.getAnnotatedType().getType() as well as
j.l.r.Executable.getAnnotatedParameterTypes().getType().
The webrev is here:
http://cr.openjdk.java.net/
Changeset: 84161510f257
Author:emc
Date: 2013-09-28 13:46 -0400
URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/84161510f257
8025413: NPE in Type.java due to recent change
Summary: isCompound throws a NPE for noType and other types. Made it return a
reasonable result instead
The enhanced metadata spec says nothing at all about an IAE. That is an
implementation detail, possibly subject to change at any point, and it
*should* not be leaked.
On 09/24/13 17:28, Paul Benedict wrote:
> Eric,
>
> Should MalformedParametersException save IAE as the root cause? Or is that
>
pot itself. There is a standard constant pool
index verifying function that throws IAE if given a bad index.
> On 2013-09-19, Eric McCorkle wrote:
>> The webrev has been updated with Joe's comments addressed.
>>
>> On 09/19/13 00:11, David Holmes wrote:
>>>
Updated webrev here:
http://cr.openjdk.java.net/~emc/8020981/
Are there any more comments, or is this good to go?
On 09/19/13 18:15, Eric McCorkle wrote:
> The webrev has been updated with Joe's comments addressed.
>
> On 09/19/13 00:11, David Holmes wrote:
>> On 19/0
Changeset: 09301757bb32
Author:emc
Date: 2013-09-23 15:37 -0400
URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/09301757bb32
6499673: Assertion check for TypeVariable.getUpperBound() fails.
Summary: Fix TypeVariable.getUpperBound to return results as specified
Reviewed-by: jjg
The webrev has been updated with Joe's comments addressed.
On 09/19/13 00:11, David Holmes wrote:
> On 19/09/2013 9:59 AM, Eric McCorkle wrote:
>> This still needs to be reviewed.
>
> You seem to have ignored Joe's comments regarding the change to Modifier
> being in
This still needs to be reviewed.
On 09/16/13 14:50, Eric McCorkle wrote:
> I pulled the class files into byte array constants, as a temporary
> measure until a viable method for testing bad class files is developed.
>
> The webrev has been refreshed. The class files will be taken ou
:
> To avoid storing binaries in Hg, you could try something like:
>
> * uuencode / ascii armor the class file
> * convert to byte array in the test
> * load classes from byte array
>
> -Joe
>
> On 09/13/2013 11:54 AM, Eric McCorkle wrote:
>> I did it by hand
On 09/13/13 09:53, Paul Benedict wrote:
> MalformedParametersException should receive a @since tag.
>
> Additionally, the javadoc doesn't describe what it means for a parameter to
> be malformed. The answer doesn't need to be exhaustive, but I think some
> examples would help developers if they ca
k that patching logic inside a custom
> ClassLoader...
>
> To hacky? Dependent on future changes of javac? At least the "bad name"
> patching could be performed trivially and reliably, I think: search and
> replace with same-length string.
>
> Regards, Peter
>
>
t;
> cheers
> /Joel
>
> On Sep 13, 2013, at 4:49 PM, Eric McCorkle wrote:
>
>> There is no simple means of generating bad class files for testing.
>> This is a huge deficiency in our testing abilities.
>>
>> If these class files shouldn't go in
ss file as an embedded byte array in
> the test.
>
> cheers
> /Joel
>
> On Sep 13, 2013, at 3:40 PM, Eric McCorkle wrote:
>
>> A new webrev is posted (and crucible updated), which actually validates
>> parameter names correctly. Apologies for the last one.
>>
A new webrev is posted (and crucible updated), which actually validates
parameter names correctly. Apologies for the last one.
On 09/12/13 16:02, Eric McCorkle wrote:
> Hello,
>
> Please review this patch, which implements correct behavior for the
> Parameter Reflection API in
Hello,
Please review this patch, which implements correct behavior for the
Parameter Reflection API in the case of malformed class files.
The bug report is here:
https://bugs.openjdk.java.net/browse/JDK-8020981
The webrev is here:
http://cr.openjdk.java.net/~emc/8020981/
This review is also on
Changeset: 5d2d484a1216
Author:emc
Date: 2013-09-12 14:52 -0400
URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/5d2d484a1216
8013846: javac fails to reject semantically equivalent generic method
declarations
Summary: Cause javac to consider intersection types with the same el
Changeset: c3ef78cd9081
Author:emc
Date: 2013-09-11 09:24 -0400
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/c3ef78cd9081
6962494: Update documentation on Executable.getParameterAnnotations()
Summary: Update javadoc comments on getParameterAnnotations to correctly
describe its b
d parameters) ..."
>
I worded this as "If the executable has no parameters (meaning no
formal, no synthetic, and no mandated...)"
> Thanks,
>
> -Joe
>
> On 9/10/2013 6:28 AM, Eric McCorkle wrote:
>> A new webrev has been posted, with some improvements t
Changeset: 65c218b25b61
Author:emc
Date: 2013-09-11 08:30 -0400
URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/65c218b25b61
8024510: lib/combo/tools/javac/combo/TemplateTest.java fails
Summary: Edit regex in Template to allow "MAJOR." pattern.
Reviewed-by: briangoetz
! test/
A new webrev has been posted, with some improvements to the comment.
On 09/09/13 17:41, Eric McCorkle wrote:
> Hello,
>
> Please review this patch which updates the javadoc comments for
> java.lang.reflect.Executable.getParameterAnnotations(). The patch
> corrects the javadocs
Hello,
Please review this patch which updates the javadoc comments for
java.lang.reflect.Executable.getParameterAnnotations(). The patch
corrects the javadocs to describe the actual behavior of this method.
It also refers users to the new java.lang.reflect.Parameter API.
See comments on the bug
Changeset: f4efd6ef6e80
Author:emc
Date: 2013-09-09 16:26 -0400
URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/f4efd6ef6e80
8022322: Reject default and static methods in annotation
Summary: Causes javac to reject static and default method declarations inside
an annotation
Re
Changeset: 67c5110c60fe
Author:emc
Date: 2013-09-09 17:11 -0400
URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/67c5110c60fe
8015322: Javac template test framework
Summary: Putback of the javac template test framework from the Lambda repository
Reviewed-by: jjg
Contributed-by:
Changeset: 1ab22e60a738
Author:emc
Date: 2013-08-22 12:47 -0400
URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/1ab22e60a738
8020745: Suspicious MethodParameters attribute generated for local classes
capturing local variables
Summary: Corrected an error in a previous patch th
Changeset: eebb29618f50
Author:emc
Date: 2013-08-21 20:41 -0400
URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/eebb29618f50
8023520: Add missing test for JDK-7118412
Summary: The test for JDK-7118412 was dropped from the changeset in a merging
accident.
Reviewed-by: jjg
+ t
Changeset: 2068190f8ac2
Author:emc
Date: 2013-08-21 20:23 -0400
URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/2068190f8ac2
7118412: Shadowing of type-variables vs. member types
4987840: What is the scope of an annotation?
Summary: Fixed issue with shadowing of type names.
Re
Changeset: 558fe98d1ac0
Author:emc
Date: 2013-07-23 20:42 -0400
URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/558fe98d1ac0
8016880: 42 tests in annot102* fail with compile-time errors.
Summary: Fixes error in type equality when bounds of type variables have
annotations.
Rev
Changeset: a8f51c3341a5
Author:emc
Date: 2013-07-03 19:47 -0400
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/a8f51c3341a5
8016285: Add java.lang.reflect.Parameter.isNamePresent()
Summary: Add isNamePresent method to parameter reflection library, which
indicates whether or real p
Pinging this one again...
On 06/24/13 15:20, Eric McCorkle wrote:
> Pinging this RFR. It still needs a capital R reviewer.
> http://cr.openjdk.java.net/~emc/8016285/
>
> On 06/21/13 19:21, Eric McCorkle wrote:
>> On 06/21/13 16:15, Aleksey Shipilev wrote:
>>> O
Changeset: 6101e52ce9e3
Author:emc
Date: 2013-06-28 06:54 -0400
URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/6101e52ce9e3
8016760: Failure of regression test langtools/tools/javac/T6725036.java
Summary: Marking the failing test @ignore; the proposed change for 8015666
addr
Changeset: 5c548a8542b8
Author:emc
Date: 2013-06-27 17:45 -0400
URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/5c548a8542b8
8013357: javac accepts erroneous binary comparison operations
Summary: javac does not report type errors on illegal Object == primitive
comparisons
Rev
ing ignored.
>
> -- Jon
>
> On 06/27/2013 10:31 AM, Eric McCorkle wrote:
>> I could easily replace the patch I have now with one that just marks the
>> test @ignore and commit it. Do we want to go ahead and approve that?
>>
>> On 06/27/13 13:12, Jonathan Gibbo
Accidentally dropped core-libs-dev from the reply.
On 06/27/13 13:31, Eric McCorkle wrote:
> I could easily replace the patch I have now with one that just marks the
> test @ignore and commit it. Do we want to go ahead and approve that?
>
> On 06/27/13 13:12, Jonathan Gibbons wrot
Changeset: c674b396827c
Author:emc
Date: 2013-06-27 00:37 -0400
URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/c674b396827c
8014230: Compilation incorrectly succeeds with inner class constructor with 254
parameters
Summary: The compiler does not account fr extra parameters d
Can I get a capital-R review on this so I can put it through?
On 06/24/13 17:28, Aleksey Shipilev wrote:
> Forgot to reply: I'm ok with webrev.01.
>
> -Aleksey
> (rural r reviewer)
>
> On 06/24/2013 11:20 PM, Eric McCorkle wrote:
>> Pinging this RFR. It sti
ill resolve the test failure, then
> we can simply use 8016760 to add the test to the ProblemList.txt. It can
> then be removed along with the other changes for 8015666.
>
> -Chris.
>
>
>>
>> -Sherman
>>
>> On 06/25/2013 12:27 PM, Eric McCorkle wrote:
ss
> shortly, if there is no objection.
>
> -Sherman
>
> On 06/25/2013 12:27 PM, Eric McCorkle wrote:
>> Does the fix for 8015666 stop the error from happening? If so, then
>> I'll withdraw this RFR.
>>
>> On 06/25/13 13:50, Xueming Shen wrote:
>>&
ing the "default" timezone. A new pair
> ZipEntry.get/setLastModifiedTime()
> will be added to access the "real" UTC time stored in the zip entry, if
> presents.
>
> The API doc will be updated accordingly as well to explicitly explain
> the source of the
> date/t
tModifiedTime()
> will be added to access the "real" UTC time stored in the zip entry, if
> presents.
>
> The API doc will be updated accordingly as well to explicitly explain
> the source of the
> date/time and the its timezone sensitive conversion.
>
> -Sherman
&g
Hello,
Please review this simple patch which updates regression test
langtools/tools/javac/T6725036.java to offset the time returned by
JavaFileObject.getLastModified() with the local time to UTC delta.
Please note that this patch is intended to address the test failures,
and that I will be immed
Changeset: bf020de5a6db
Author:emc
Date: 2013-06-24 22:03 -0400
URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/bf020de5a6db
8012722: Single comma in array initializer should parse
Summary: Annotations of the form @Foo({,}) should parse
Reviewed-by: jjg
! src/share/classes/co
Pinging this RFR. It still needs a capital R reviewer.
http://cr.openjdk.java.net/~emc/8016285/
On 06/21/13 19:21, Eric McCorkle wrote:
> On 06/21/13 16:15, Aleksey Shipilev wrote:
>> On 06/21/2013 11:57 PM, Eric McCorkle wrote:
>>> The webrev is here:
>>> http://cr.op
On 06/21/13 16:15, Aleksey Shipilev wrote:
> On 06/21/2013 11:57 PM, Eric McCorkle wrote:
>> The webrev is here:
>> http://cr.openjdk.java.net/~emc/8016285/
>
> Looks generally good (but not a Reviewer).
>
> A few questions though:
> a) Are we em-bracing the
Hello, please review this patch which adds isNamePresent to the
java.lang.reflect.Parameter API.
The webrev is here:
http://cr.openjdk.java.net/~emc/8016285/
The change request is here:
http://bugs.sun.com/view_bug.do?bug_id=8016285
The updated spec is here:
http://cr.openjdk.java.net/~abuckley/
Changeset: e9ebff1840e5
Author:emc
Date: 2013-06-20 19:01 -0400
URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/e9ebff1840e5
8007546: ClassCastException on JSR308 tests
8015993: jck-compiler tests are failed with java.lang.ClassCastException
Summary: Fix ClassCastExceptions ar
Thanks, Alan.
On 06/06/13 09:22, Alan Bateman wrote:
> On 06/06/2013 14:07, Eric McCorkle wrote:
>> Hello,
>>
>> Please review this simple patch that removes setProtectionDomain0 from
>> java.lang.Class. This is part of a two-stage process to remove the
>&
Changeset: 37aa82c52317
Author:emc
Date: 2013-06-06 09:51 -0400
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/37aa82c52317
8016019: Remove setProtectionDomain0 and JVM_SetProtectionDomain in JDK
Summary: setProtectionDomain0 and JVM_SetProtectionDomain are unused since at
least 1
Hello,
Please review this simple patch that removes setProtectionDomain0 from
java.lang.Class. This is part of a two-stage process to remove the
function from the JVM as well.
This method is not used, and has not been used in some time (at least
before 1.5)
The webrev is here:
http://cr.openjdk
Changeset: 8717586f7b05
Author:emc
Date: 2013-06-06 08:48 -0400
URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/8717586f7b05
8015701: MethodParameters are not filled in for synthetic captured local
variables
Summary: Synthetic parameters for captured local variables in an ano
Thanks all, putting the changes in.
On 06/03/13 06:07, Alan Bateman wrote:
> On 01/06/2013 13:32, Eric McCorkle wrote:
>> Thanks, Kumar. Do I have a second?
>>
> For the jdk repository then only one reviewer is required although in
> specialized or complicated areas then h
Changeset: 3d4d7ed93731
Author:emc
Date: 2013-06-03 10:44 -0400
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/3d4d7ed93731
8014834: shell tests don't begin with #!/bin/sh
Summary: Some shell tests don't begin with the command interpreter line
Reviewed-by: alanb, ksrini
! test/jav
Thanks, Kumar. Do I have a second?
Note: JPRT has been unreliable lately, so it may be a while before I can
put this in.
On 05/30/13 17:45, Kumar Srinivasan wrote:
> Hi Erik,
>
> This looks good, historically the #!shell was inserted after the
> CopyRight,
> because it was thought that the au
Hello,
Please review this simple patch which fixes an issue with several of the
shell script tests, namely that they do not start with #!/bin/sh
The webrev is here:
http://cr.openjdk.java.net/~emc/8014834/
The bug is here:
http://bugs.sun.com/view_bug.do?bug_id=8014834
Thanks,
Eric
Changeset: ca0957f0d408
Author:emc
Date: 2013-04-25 14:23 -0400
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/ca0957f0d408
8012937: Correct errors in javadoc comments.
Summary: Correct some errors in the javadoc comments for parameter reflection.
Reviewed-by: darcy
! src/share/cl
Any further comments, or is this one good to go?
On 04/23/13 19:54, Joseph Darcy wrote:
> Acknowledged; thanks for checking,
>
> -Joe
>
> On 4/23/2013 7:46 AM, Eric McCorkle wrote:
>> I believe so. Alex Buckley recommended the exact wording.
>>
>> On 0
1 - 100 of 140 matches
Mail list logo
