[8] Review request for 8005607:
Recursion in J2DXErrHandler() Causes a Stack Overflow on Linux
In-Reply-To: <51221D2D.9070907@oracle.com>
References: <50DEF3AC.2070600@oracle.com> <50ED9609.1070002@oracle.com>
<50EE1CEB.5080500@oracle.com> <50EFED12.6070700@oracle.com>
<50EFFB4F.4090405@oracle.com> <510946F7.3090709@oracle.com>
<510A6A54.1080705@oracle.com> <510A90DF.2060501@oracle.com>
<511BC5E4.9090302@oracle.com> <51220DA4.7040608@oracle.com>
<512218E7.9090808@oracle.com> <51221D2D.9070907@oracle.com>
Message-ID: <512251F0.7020204@oracle.com>
Hello Artem,
Could you please review a new version of the fix. The method
"XErrorHandlerUtil.getDisplay()" was removed and
"XErrorHandlerUtil.XSync()" method refers to the field "display"
directly now.
Webrev: http://cr.openjdk.java.net/~alitvinov/8005607/webrev.03
Thank you,
Anton
On 2/18/2013 4:23 PM, Artem Ananiev wrote:
>
> On 2/18/2013 4:04 PM, Anton Litvinov wrote:
>> Hello Artem,
>>
>> Thank you very much for the review of this fix. My responses to your
>> questions are provided below in the same order, which you defined.
>>
>> 1. I think that "XErrorHandlerUtil.saved_error" field can surely be
>> marked as private, but in this case the corresponding
>> "XErrorHandlerUtil.getSavedError" method will be necessary, because
>> this field is actively accessed from other classes which set a
>> certain instance of XErrorHandler. For example
>> "MotifDnDDropTargetProtocol.java", "XDragSourceProtocol.java" and a
>> few other classes edited in this fix.
>
> OK, I missed that usages when looking at the webrev. Let it stay
> unchanged now.
>
>> 2. Yes, I completely agree that "XErrorHandlerUtil.getDisplay()" is
>> reduntant. This method will be eliminated.
>
> Thanks,
>
> Artem
>
>> Thank you,
>> Anton
>>
>> On 2/18/2013 3:16 PM, Artem Ananiev wrote:
>>> Hi, Anton,
>>>
>>> a few minor comments:
>>>
>>> 1. XErrorHandlerUtil: can saved_error be private instead of package
>>> protected?
>>>
>>> 2. XErrorHandlerUtil.getDisplay() seems to be redundant.
>>>
>>> In general, the fix looks perfectly fine to me. Please, wait for
>>> comments from Java2D team, though.
>>>
>>> Thanks,
>>>
>>> Artem
>>>
>>> On 2/13/2013 8:57 PM, Anton Litvinov wrote:
>>>> Hello Anthony,
>>>>
>>>> Could you please review the third version of the fix containing
>>>> modifications discussed with you in the previous letter.
>>>>
>>>> Webrev: http://cr.openjdk.java.net/~alitvinov/8005607/webrev.02
>>>>
>>>> This version of the fix differs from the previous in the following
>>>> places:
>>>>
>>>> 1. A comment about the place of invocation of the method
>>>> "XErrorHandlerUtil.init" was added to a documentation block of the
>>>> method.
>>>> 2. A code related to XShmAttach function common to the files
>>>> "src/solaris/native/sun/awt/awt_GraphicsEnv.c" and
>>>> "src/solaris/native/sun/java2d/x11/X11SurfaceData.c" was extracted
>>>> into a separate function "TryXShmAttach" declared in
>>>> "src/solaris/native/sun/awt/awt_GraphicsEnv.h" file.
>>>> 3. All JNI code related to X error handling was implemented as
>>>> corresponding macros defined in
>>>> "src/solaris/native/sun/awt/awt_util.h" file.
>>>>
>>>> Thank you,
>>>> Anton
>>>>
>>>> On 1/31/2013 7:42 PM, Anton Litvinov wrote:
>>>>> Hello Anthony,
>>>>>
>>>>> Thank you for the review and these remarks. Surely, the comment will
>>>>> be added. I think that all JNI code related to XShmAttach can be
>>>>> definitely transferred into a separate dedicated function, which will
>>>>> be declared in "src/solaris/native/sun/awt/awt_GraphicsEnv.h" file. I
>>>>> will try to wrap all JNU calls connected with XErrorHandler into the
>>>>> particular "WITH_XERROR_HANDLER", "RESTORE_XERROR_HANDLER" functions
>>>>> or macros.
>>>>>
>>>>> Thank you,
>>>>> Anton
>>>>>
>>>>> On 1/31/2013 4:57 PM, Anthony Petrov wrote:
>>>>>> Hi Anton,
>>>>>>
>>>>>> A couple comments:
>>>>>>
>>>>>> 1. src/solaris/classes/sun/awt/X11/XErrorHandlerUtil.java
>>>>>>> 80 private static void init(long display) {
>>>>>>
>>>>>> This method is private and isn't called from anywhere in this class
>>>>>> itself. This looks confusing. Please add a comment stating that this
>>>>>> method is invoked from native code, and from where exactly.
>>>>>>
>>>>>>
>>>>>> 2. Interesting that we use this machinery to call the XShmAttach()
>>>>>> from native code twice, and the code looks quite similar in each
>>>>>> case. Would it be possible to extract the common code in a separate
>>>>>> function (a-la BOOL TryXShmAttach(...)) to avoid code replication?
>>>>>> There are other usages as well, so we could also introduce a macro
>>>>>> (such as the old EXEC_WITH_XERROR_HANDLER but now with other
>>>>>> arguments) that would minimize all the JNU_ calls required to use
>>>>>> this machinery.
>>>>>>
>>>>>>
>>>>>> Otherwise the fix looks great.
>>>>>>
>>>>>> --
>>>>>> best regards,
>>>>>> Anthony
>>>>>>
>>>>>> On 1/30/2013 20:14, Anton Litvinov wrote:
>>>>>>> Hello Anthony,
>>>>>>>
>>>>>>> Could you, please, review a second version of the fix, which is
>>>>>>> based on an idea of reusing the existing AWT native global error
>>>>>>> handler from Java 2D native code.
>>>>>>>
>>>>>>> Webrev: http://cr.openjdk.java.net/~alitvinov/8005607/webrev.01
>>>>>>>
>>>>>>> The fix consists of the following parts:
>>>>>>>
>>>>>>> 1. Migration of all X error handling code from XToolkit to a new
>>>>>>> XErrorHandlerUtil class for resolution of interdependency
>>>>>>> between
>>>>>>> a static initialization block of XToolkit and a block
>>>>>>> initializing
>>>>>>> java.awt.GraphicsEnvironment singleton. Such dependency is
>>>>>>> created
>>>>>>> by new calls to XToolkit static methods from
>>>>>>> "src/solaris/native/sun/awt/awt_GraphicsEnv.c",
>>>>>>> "src/solaris/native/sun/java2d/x11/X11SurfaceData.c" files.
>>>>>>> 2. Substitution of XToolkit.WITH_XERROR_HANDLER,
>>>>>>> XToolkit.RESTORE_XERROR_HANDLER ... for corresponding
>>>>>>> methods,
>>>>>>> fields of XErrorHandlerUtil class in all places of JDK source
>>>>>>> code, where they were used.
>>>>>>> 3. Substitution of all found native X error handlers which are
>>>>>>> set in
>>>>>>> native code (awt_GraphicsEnv.c, X11SurfaceData.c,
>>>>>>> GLXSurfaceData.c) for new synthetic Java error handlers.
>>>>>>> 4. Removal of X error handling code used by the native error
>>>>>>> handlers
>>>>>>> from "solaris/native/sun/awt/awt_util.c"
>>>>>>> "solaris/native/sun/awt/awt_util.h" files.
>>>>>>>
>>>>>>> Thank you,
>>>>>>> Anton
>>>>>>>
>>>>>>> On 1/11/2013 3:45 PM, Anthony Petrov wrote:
>>>>>>>> I'm not Jim, but as I indicated earlier my opinion is that the
>>>>>>>> easiest way to fix this is to install the existing
>>>>>>>> J2DXErrHandler()
>>>>>>>> only once. That is, it is the second option listed by you. Of
>>>>>>>> course, the J2DXErrHandler needs to be updated as well to detect
>>>>>>>> whether 2D code wants to use it at the moment or it must simply
>>>>>>>> delegate to the previous handler (i.e. where the code currently
>>>>>>>> installs/uninstalls the handler, it must instead set a global
>>>>>>>> boolean flag or something.)
>>>>>>>>
>>>>>>>> While the first option (reusing the existing AWT machinery) is an
>>>>>>>> interesting idea in general, I think it is complex and would
>>>>>>>> require too much additional testing and bring an unjustified risk
>>>>>>>> to the solution for such a basic problem.
>>>>>>>>
>>>>>>>> --
>>>>>>>> best regards,
>>>>>>>> Anthony
>>>>>>>>
>>>>>>>> On 1/11/2013 14:44, Anton Litvinov wrote:
>>>>>>>>> Hello Jim,
>>>>>>>>>
>>>>>>>>> Thank you very much for the review and provision of a new idea of
>>>>>>>>> a solution. Elimination of the logic, which sets/unsets
>>>>>>>>> J2DXErrHandler() for each call "XShmAttach(awt_display,
>>>>>>>>> &shminfo))" should effectively resolve the issue, but only in
>>>>>>>>> case
>>>>>>>>> if all other native error handlers, which were set by the system
>>>>>>>>> function "XSetErrorHandler()" in JDK or in any external library,
>>>>>>>>> observe the rule of relaying of all events, which are not
>>>>>>>>> relative
>>>>>>>>> to them, to the previously saved error handlers. Otherwise an
>>>>>>>>> error generated during "XShmAttach" function call will not be
>>>>>>>>> handled by J2DXErrHandler().
>>>>>>>>>
>>>>>>>>> Could you answer the following question. By setting
>>>>>>>>> J2DXErrHandler() only once and forever do you mean usage of AWT
>>>>>>>>> global event handler "static int ToolkitErrorHandler(Display *
>>>>>>>>> dpy, XErrorEvent * event)" from
>>>>>>>>> "src/solaris/native/sun/xawt/XlibWrapper.c" with Java synthetic
>>>>>>>>> handlers or creation of another global native error handler with
>>>>>>>>> J2DXErrHandler as native synthetic handler?
>>>>>>>>>
>>>>>>>>> Thank you,
>>>>>>>>> Anton
>>>>>>>>>
>>>>>>>>> On 1/10/2013 5:44 AM, Jim Graham wrote:
>>>>>>>>>> I think I'd rather see some way to prevent double-adding the
>>>>>>>>>> handler in the first place as well. Since it is only ever used
>>>>>>>>>> on errors I also think it is OK to set it once and leave it
>>>>>>>>>> there
>>>>>>>>>> forever...
>>>>>>>>>>
>>>>>>>>>> ...jim
>>>>>>>>>>
>>>>>>>>>> On 1/9/13 8:08 AM, Anthony Petrov wrote:
>>>>>>>>>>> Hi Anton et al.,
>>>>>>>>>>>
>>>>>>>>>>> If I read the description of the bug correctly, specifically
>>>>>>>>>>> this part:
>>>>>>>>>>>
>>>>>>>>>>>> The problem occurs, if another thread (for example, GTK
>>>>>>>>>>>> thread) is
>>>>>>>>>>>> doing the same sort of thing concurrently. This can lead to
>>>>>>>>>>>> the
>>>>>>>>>>>> following situation.
>>>>>>>>>>>> JVM thread: Sets J2DXErrHandler(), saves
>>>>>>>>>>>> ANY_PREVIOUS_HANDLER as
>>>>>>>>>>>> previous GTK thread: Sets some GTK_HANDLER, saves
>>>>>>>>>>>> J2DXErrHandler() as previous JVM thread: Restores
>>>>>>>>>>>> ANY_PREVIOUS_HANDLER GTK thread: Restores
>>>>>>>>>>>> J2DXErrHandler() JVM
>>>>>>>>>>>> thread: Sets J2DXErrHandler(), saves J2DXErrHandler() as
>>>>>>>>>>>> previous
>>>>>>>>>>>
>>>>>>>>>>> It is obvious that at this final step 2D is in an inconsistent
>>>>>>>>>>> state. We
>>>>>>>>>>> don't expect to replace our own error handler (and it shouldn't
>>>>>>>>>>> have
>>>>>>>>>>> been there in the first place).
>>>>>>>>>>>
>>>>>>>>>>> I realize that the fix you propose works around this problem.
>>>>>>>>>>> But this
>>>>>>>>>>> doesn't look like an ideal solution to me.
>>>>>>>>>>>
>>>>>>>>>>> BTW, IIRC, in JDK7 (and 6?) we decided to set the actual X11
>>>>>>>>>>> error
>>>>>>>>>>> handler only once and never replace it. All the rest of the
>>>>>>>>>>> push_handler/pop_handler logic is now located in Java code (see
>>>>>>>>>>> XToolkit.SAVED_ERROR_HANDLER() and the surrounding logic). I
>>>>>>>>>>> believe
>>>>>>>>>>> that we should somehow share this machinery with the 2D code to
>>>>>>>>>>> avoid
>>>>>>>>>>> this sort of problems. Though I'm not sure if this will
>>>>>>>>>>> eliminate this
>>>>>>>>>>> exact issue.
>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>> 2D/AWT folks: any other thoughts?
>>>>>>>>>>>
>>>>>>>>>>> --
>>>>>>>>>>> best regards,
>>>>>>>>>>> Anthony
>>>>>>>>>>>
>>>>>>>>>>> On 12/29/2012 17:44, Anton Litvinov wrote:
>>>>>>>>>>>> Hello,
>>>>>>>>>>>>
>>>>>>>>>>>> Please review the following fix for a bug.
>>>>>>>>>>>>
>>>>>>>>>>>> Bug:
>>>>>>>>>>>> http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8005607
>>>>>>>>>>>> https://jbs.oracle.com/bugs/browse/JDK-8005607
>>>>>>>>>>>> Webrev:
>>>>>>>>>>>> http://cr.openjdk.java.net/~alitvinov/8005607/webrev.00
>>>>>>>>>>>>
>>>>>>>>>>>> The bug consists in a crash which is caused by a stack
>>>>>>>>>>>> overflow
>>>>>>>>>>>> for
>>>>>>>>>>>> the reason of an infinite recursion in AWT native function
>>>>>>>>>>>> J2DXErrHandler() under certain circumstances on 32-bit Linux
>>>>>>>>>>>> OS. The
>>>>>>>>>>>> fix is based on introduction of the logic, which detects
>>>>>>>>>>>> indirect
>>>>>>>>>>>> recursive calls to J2DXErrHandler() by means of a simple
>>>>>>>>>>>> counter, to
>>>>>>>>>>>> J2DXErrHandler() native function. Such a solution requires
>>>>>>>>>>>> minimum
>>>>>>>>>>>> code changes, does not alter the handler's code significantly
>>>>>>>>>>>> and
>>>>>>>>>>>> eliminates this bug.
>>>>>>>>>>>>
>>>>>>>>>>>> Adding 2d-dev at openjdk.java.net e-mail alias to the list of
>>>>>>>>>>>> recipients
>>>>>>>>>>>> of this letter, because the edited function's name is related
>>>>>>>>>>>> to Java
>>>>>>>>>>>> 2D area of JDK, despite of the fact that the edited file is
>>>>>>>>>>>> located in
>>>>>>>>>>>> AWT directory.
>>>>>>>>>>>>
>>>>>>>>>>>> Thank you,
>>>>>>>>>>>> Anton
>>>>
>>
From jennifer.godinez at oracle.com Mon Feb 18 22:04:32 2013
From: jennifer.godinez at oracle.com (jennifer.godinez at oracle.com)
Date: Mon, 18 Feb 2013 22:04:32 +0000
Subject: [OpenJDK 2D-Dev] hg: jdk8/2d/jdk: 8005191: [parfait]
#384 sun/font/layout/LookupProcessor.cpp Null pointer dereference
Message-ID: <20130218220534.BD96547B75@hg.openjdk.java.net>
Changeset: 4b11045a9c4c
Author: jgodinez
Date: 2013-02-18 14:04 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/4b11045a9c4c
8005191: [parfait] #384 sun/font/layout/LookupProcessor.cpp Null pointer dereference
Reviewed-by: prr, vadim
Contributed-by: jia-hong.chen at oracle.com
! src/share/native/sun/font/layout/LookupProcessor.cpp
From lana.steuck at oracle.com Wed Feb 20 07:23:05 2013
From: lana.steuck at oracle.com (lana.steuck at oracle.com)
Date: Wed, 20 Feb 2013 07:23:05 +0000
Subject: [OpenJDK 2D-Dev] hg: jdk8/2d: 7 new changesets
Message-ID: <20130220072306.C3C4047BDB@hg.openjdk.java.net>
Changeset: 45dcccc6d221
Author: katleman
Date: 2013-02-14 11:43 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/rev/45dcccc6d221
Added tag jdk8-b77 for changeset 3933eebc659d
! .hgtags
Changeset: 8dd61906da5f
Author: chegar
Date: 2013-02-06 11:36 +0000
URL: http://hg.openjdk.java.net/jdk8/2d/rev/8dd61906da5f
8007625: race with nested repos in /common/bin/hgforest.sh
Reviewed-by: dholmes, ohair, ohrstrom
! common/bin/hgforest.sh
! get_source.sh
Changeset: 168dd033604a
Author: mduigou
Date: 2013-02-06 11:09 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/rev/168dd033604a
8004726: Link bug ids to jbs rather than monaco.
Reviewed-by: ohair, chegar, katleman
! make/scripts/webrev.ksh
Changeset: 7817368287cd
Author: mduigou
Date: 2013-02-06 11:12 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/rev/7817368287cd
8006595: Use jdk/test/Makefile targets in preference to local definitions
Reviewed-by: alanb
! common/makefiles/Main.gmk
! test/Makefile
Changeset: fdb1e09519ed
Author: sherman
Date: 2013-02-12 09:27 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/rev/fdb1e09519ed
8007392: JSR 310: DateTime API Updates
Summary: Integration of JSR310 Date/Time API for M7
Reviewed-by: darcy, alanb, naoto
Contributed-by: scolebourne at joda.org, roger.riggs at oracle.com, masayoshi.okutsu at oracle.com, patrick.zhang at oracle.com
! common/makefiles/javadoc/CORE_PKGS.gmk
Changeset: 76808fb4194a
Author: lana
Date: 2013-02-13 11:21 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/rev/76808fb4194a
Merge
! common/makefiles/Main.gmk
Changeset: bbb7548d45c7
Author: lana
Date: 2013-02-14 22:11 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/rev/bbb7548d45c7
Merge
From lana.steuck at oracle.com Wed Feb 20 07:23:05 2013
From: lana.steuck at oracle.com (lana.steuck at oracle.com)
Date: Wed, 20 Feb 2013 07:23:05 +0000
Subject: [OpenJDK 2D-Dev] hg: jdk8/2d/corba: Added tag jdk8-b77 for
changeset 35684a40c584
Message-ID: <20130220072312.23BB047BDC@hg.openjdk.java.net>
Changeset: 27d6368ae8ba
Author: katleman
Date: 2013-02-14 11:43 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/corba/rev/27d6368ae8ba
Added tag jdk8-b77 for changeset 35684a40c584
! .hgtags
From lana.steuck at oracle.com Wed Feb 20 07:23:08 2013
From: lana.steuck at oracle.com (lana.steuck at oracle.com)
Date: Wed, 20 Feb 2013 07:23:08 +0000
Subject: [OpenJDK 2D-Dev] hg: jdk8/2d/jaxp: Added tag jdk8-b77 for changeset
573e789c187a
Message-ID: <20130220072325.B1E7847BDD@hg.openjdk.java.net>
Changeset: 00958c5a7070
Author: katleman
Date: 2013-02-14 11:43 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/jaxp/rev/00958c5a7070
Added tag jdk8-b77 for changeset 573e789c187a
! .hgtags
From lana.steuck at oracle.com Wed Feb 20 07:23:18 2013
From: lana.steuck at oracle.com (lana.steuck at oracle.com)
Date: Wed, 20 Feb 2013 07:23:18 +0000
Subject: [OpenJDK 2D-Dev] hg: jdk8/2d/jaxws: Added tag jdk8-b77 for
changeset 64dfba1bad16
Message-ID: <20130220072328.05F2847BDE@hg.openjdk.java.net>
Changeset: 391de4c992d1
Author: katleman
Date: 2013-02-14 11:43 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/jaxws/rev/391de4c992d1
Added tag jdk8-b77 for changeset 64dfba1bad16
! .hgtags
From lana.steuck at oracle.com Wed Feb 20 07:23:26 2013
From: lana.steuck at oracle.com (lana.steuck at oracle.com)
Date: Wed, 20 Feb 2013 07:23:26 +0000
Subject: [OpenJDK 2D-Dev] hg: jdk8/2d/langtools: 20 new changesets
Message-ID: <20130220072459.C8D4247BDF@hg.openjdk.java.net>
Changeset: bc24411bcc37
Author: katleman
Date: 2013-02-14 11:44 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/langtools/rev/bc24411bcc37
Added tag jdk8-b77 for changeset 89c664151689
! .hgtags
Changeset: de932285124c
Author: jjg
Date: 2013-02-05 21:55 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/langtools/rev/de932285124c
8007485: test creates .class files in the test/ directory
Reviewed-by: mcimadamore
! test/tools/javac/api/8007344/Test.java
Changeset: 1df20330f6bd
Author: mcimadamore
Date: 2013-02-06 14:03 +0000
URL: http://hg.openjdk.java.net/jdk8/2d/langtools/rev/1df20330f6bd
8007463: Cleanup inference related classes
Summary: Make Infer.InferenceContext an inner class; adjust bound replacement logic in Type.UndetVar
Reviewed-by: jjg
! src/share/classes/com/sun/tools/javac/code/Type.java
! src/share/classes/com/sun/tools/javac/code/Types.java
! src/share/classes/com/sun/tools/javac/comp/Attr.java
! src/share/classes/com/sun/tools/javac/comp/Check.java
! src/share/classes/com/sun/tools/javac/comp/DeferredAttr.java
! src/share/classes/com/sun/tools/javac/comp/Infer.java
! src/share/classes/com/sun/tools/javac/comp/Resolve.java
! src/share/classes/com/sun/tools/javac/util/List.java
! test/tools/javac/generics/inference/7154127/T7154127.out
! test/tools/javac/lib/DPrinter.java
Changeset: 8cdd96f2fdb9
Author: mcimadamore
Date: 2013-02-06 14:04 +0000
URL: http://hg.openjdk.java.net/jdk8/2d/langtools/rev/8cdd96f2fdb9
8007479: Refactor DeferredAttrContext so that it points to parent context
Summary: Move DeferredAttrNode out of DeferredAttrContext; add support for nested deferred contexts
Reviewed-by: jjg
! src/share/classes/com/sun/tools/javac/comp/DeferredAttr.java
! src/share/classes/com/sun/tools/javac/comp/Infer.java
! src/share/classes/com/sun/tools/javac/comp/Resolve.java
Changeset: 153d20d0cac5
Author: jjg
Date: 2013-02-06 07:49 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/langtools/rev/153d20d0cac5
8007566: DocLint too aggressive with not allowed here:
Reviewed-by: mcimadamore
! src/share/classes/com/sun/tools/doclint/Checker.java
+ test/tools/doclint/ParaTagTest.java
Changeset: b386b8c45387
Author: jjh
Date: 2013-02-06 23:10 +0000
URL: http://hg.openjdk.java.net/jdk8/2d/langtools/rev/b386b8c45387
8007698: jtreg test T6306137.java won't compile with ASCII encoding
Reviewed-by: ksrini
! test/tools/javac/api/T6306137.java
Changeset: 5125b9854d07
Author: darcy
Date: 2013-02-07 20:47 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/langtools/rev/5125b9854d07
7195131: Update 2 compiler combo tests for repeating annotations to include package and default use cases
Reviewed-by: darcy
Contributed-by: sonali.goel at oracle.com
! test/tools/javac/annotations/repeatingAnnotations/combo/Helper.java
+ test/tools/javac/annotations/repeatingAnnotations/combo/TargetAnnoCombo.java
+ test/tools/javac/annotations/repeatingAnnotations/combo/TestCaseGenerator.java
Changeset: 762d0af062f5
Author: vromero
Date: 2013-02-08 09:12 +0000
URL: http://hg.openjdk.java.net/jdk8/2d/langtools/rev/762d0af062f5
7166455: javac doesn't set ACC_STRICT bit on for strictfp class
Reviewed-by: mcimadamore
! src/share/classes/com/sun/tools/javac/comp/Attr.java
! src/share/classes/com/sun/tools/javac/comp/Check.java
! src/share/classes/com/sun/tools/javac/jvm/Gen.java
+ test/tools/javac/7166455/CheckACC_STRICTFlagOnclinitTest.java
Changeset: b1deb90d2e37
Author: vromero
Date: 2013-02-08 09:15 +0000
URL: http://hg.openjdk.java.net/jdk8/2d/langtools/rev/b1deb90d2e37
8005931: javac doesn't set ACC_STRICT for classes with package access
Reviewed-by: mcimadamore
! src/share/classes/com/sun/tools/javac/parser/JavacParser.java
+ test/tools/javac/8005931/CheckACC_STRICTFlagOnPkgAccessClassTest.java
Changeset: 017e8bdd440f
Author: vromero
Date: 2013-02-08 09:21 +0000
URL: http://hg.openjdk.java.net/jdk8/2d/langtools/rev/017e8bdd440f
7167125: Two variables after the same operation in a inner class return different results
Reviewed-by: jjg, mcimadamore
! src/share/classes/com/sun/tools/javac/comp/Lower.java
+ test/tools/javac/7167125/DiffResultAfterSameOperationInnerClasses.java
Changeset: 60caf53b98e2
Author: jjg
Date: 2013-02-08 17:35 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/langtools/rev/60caf53b98e2
8007610: javadoc doclint does not work with -private
Reviewed-by: darcy
! src/share/classes/com/sun/tools/javadoc/DocEnv.java
! test/com/sun/javadoc/T6735320/T6735320.java
! test/tools/javadoc/doclint/DocLintTest.java
Changeset: 01af1b5c631d
Author: darcy
Date: 2013-02-11 13:37 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/langtools/rev/01af1b5c631d
8007574: Provide isFunctionalInterface in javax.lang.model
Reviewed-by: jjg
! src/share/classes/com/sun/tools/javac/model/JavacElements.java
! src/share/classes/javax/lang/model/element/TypeElement.java
! src/share/classes/javax/lang/model/util/Elements.java
+ test/tools/javac/processing/model/util/elements/TestIsFunctionalInterface.java
Changeset: 973646bf043a
Author: jfranck
Date: 2013-02-12 11:28 +0100
URL: http://hg.openjdk.java.net/jdk8/2d/langtools/rev/973646bf043a
8004822: RFE to write language model API tests for repeating annotations based on the spec updates
Reviewed-by: jjg, abuckley
Contributed-by: Matherey Nunez
+ test/tools/javac/processing/model/element/repeatingAnnotations/ElementRepAnnoTester.java
+ test/tools/javac/processing/model/element/repeatingAnnotations/MixRepeatableAndOfficialContainerBasicTest.java
+ test/tools/javac/processing/model/element/repeatingAnnotations/MixRepeatableAndOfficialContainerInheritedA1Test.java
+ test/tools/javac/processing/model/element/repeatingAnnotations/MixRepeatableAndOfficialContainerInheritedA2Test.java
+ test/tools/javac/processing/model/element/repeatingAnnotations/MixRepeatableAndOfficialContainerInheritedB1Test.java
+ test/tools/javac/processing/model/element/repeatingAnnotations/MixRepeatableAndOfficialContainerInheritedB2Test.java
+ test/tools/javac/processing/model/element/repeatingAnnotations/MixSingularAndUnofficialContainerBasicTest.java
+ test/tools/javac/processing/model/element/repeatingAnnotations/MixSingularAndUnofficialContainerInheritedA1Test.java
+ test/tools/javac/processing/model/element/repeatingAnnotations/MixSingularAndUnofficialContainerInheritedA2Test.java
+ test/tools/javac/processing/model/element/repeatingAnnotations/MixSingularAndUnofficialContainerInheritedB1Test.java
+ test/tools/javac/processing/model/element/repeatingAnnotations/MixSingularAndUnofficialContainerInheritedB2Test.java
+ test/tools/javac/processing/model/element/repeatingAnnotations/OfficialContainerBasicTest.java
+ test/tools/javac/processing/model/element/repeatingAnnotations/OfficialContainerInheritedTest.java
+ test/tools/javac/processing/model/element/repeatingAnnotations/RepeatableBasicTest.java
+ test/tools/javac/processing/model/element/repeatingAnnotations/RepeatableInheritedTest.java
+ test/tools/javac/processing/model/element/repeatingAnnotations/RepeatableOfficialContainerBasicTest.java
+ test/tools/javac/processing/model/element/repeatingAnnotations/RepeatableOfficialContainerInheritedTest.java
+ test/tools/javac/processing/model/element/repeatingAnnotations/RepeatableOverrideATest.java
+ test/tools/javac/processing/model/element/repeatingAnnotations/RepeatableOverrideBTest.java
+ test/tools/javac/processing/model/element/repeatingAnnotations/SingularBasicTest.java
+ test/tools/javac/processing/model/element/repeatingAnnotations/SingularInheritedATest.java
+ test/tools/javac/processing/model/element/repeatingAnnotations/SingularInheritedBTest.java
+ test/tools/javac/processing/model/element/repeatingAnnotations/UnofficialContainerBasicTest.java
+ test/tools/javac/processing/model/element/repeatingAnnotations/UnofficialContainerInheritedTest.java
+ test/tools/javac/processing/model/element/repeatingAnnotations/supportingAnnotations/Bar.java
+ test/tools/javac/processing/model/element/repeatingAnnotations/supportingAnnotations/BarContainer.java
+ test/tools/javac/processing/model/element/repeatingAnnotations/supportingAnnotations/BarContainerContainer.java
+ test/tools/javac/processing/model/element/repeatingAnnotations/supportingAnnotations/BarInherited.java
+ test/tools/javac/processing/model/element/repeatingAnnotations/supportingAnnotations/BarInheritedContainer.java
+ test/tools/javac/processing/model/element/repeatingAnnotations/supportingAnnotations/BarInheritedContainerContainer.java
+ test/tools/javac/processing/model/element/repeatingAnnotations/supportingAnnotations/ExpectedBase.java
+ test/tools/javac/processing/model/element/repeatingAnnotations/supportingAnnotations/ExpectedContainer.java
+ test/tools/javac/processing/model/element/repeatingAnnotations/supportingAnnotations/Foo.java
+ test/tools/javac/processing/model/element/repeatingAnnotations/supportingAnnotations/FooInherited.java
+ test/tools/javac/processing/model/element/repeatingAnnotations/supportingAnnotations/UnofficialContainer.java
+ test/tools/javac/processing/model/element/repeatingAnnotations/supportingAnnotations/UnofficialInheritedContainer.java
Changeset: 073696f59241
Author: vromero
Date: 2013-02-12 13:36 +0000
URL: http://hg.openjdk.java.net/jdk8/2d/langtools/rev/073696f59241
8006334: javap, JavapTask constructor breaks with null pointer exception if parameter options is null
Reviewed-by: jjg
! src/share/classes/com/sun/tools/javap/JavapTask.java
+ test/tools/javap/8006334/JavapTaskCtorFailWithNPE.java
Changeset: 2154ed9ff6c8
Author: mcimadamore
Date: 2013-02-12 19:25 +0000
URL: http://hg.openjdk.java.net/jdk8/2d/langtools/rev/2154ed9ff6c8
8007464: Add graph inference support
Summary: Add support for more aggressive type-inference scheme
Reviewed-by: jjg
! src/share/classes/com/sun/tools/javac/code/Source.java
! src/share/classes/com/sun/tools/javac/comp/Attr.java
! src/share/classes/com/sun/tools/javac/comp/DeferredAttr.java
! src/share/classes/com/sun/tools/javac/comp/Infer.java
+ src/share/classes/com/sun/tools/javac/util/GraphUtils.java
! test/tools/javac/6758789/T6758789b.out
! test/tools/javac/Diagnostics/6799605/T6799605.out
! test/tools/javac/diags/examples/CantApplyDiamond1.java
! test/tools/javac/diags/examples/InferredDoNotConformToEq.java
! test/tools/javac/diags/examples/InferredDoNotConformToUpper.java
! test/tools/javac/diags/examples/WhereFreshTvar.java
! test/tools/javac/generics/7015430/T7015430.out
! test/tools/javac/generics/7151802/T7151802.out
! test/tools/javac/generics/diamond/neg/Neg06.out
! test/tools/javac/generics/inference/6278587/T6278587Neg.java
! test/tools/javac/generics/inference/6638712/T6638712d.out
! test/tools/javac/generics/inference/6638712/T6638712e.out
! test/tools/javac/generics/inference/7154127/T7154127.java
! test/tools/javac/generics/inference/7154127/T7154127.out
! test/tools/javac/generics/inference/7177306/T7177306a.out
! test/tools/javac/generics/inference/7177306/T7177306e.java
! test/tools/javac/generics/inference/7177306/T7177306e.out
! test/tools/javac/generics/odersky/BadTest4.java
! test/tools/javac/lambda/TargetType14.out
! test/tools/javac/lambda/TargetType20.java
- test/tools/javac/lambda/TargetType20.out
! test/tools/javac/lambda/TargetType28.out
! test/tools/javac/lambda/TargetType50.java
- test/tools/javac/lambda/TargetType50.out
! test/tools/javac/lambda/TargetType51.java
! test/tools/javac/lambda/TargetType52.java
! test/tools/javac/lambda/TargetType52.out
+ test/tools/javac/lambda/TargetType53.java
+ test/tools/javac/lambda/TargetType54.java
+ test/tools/javac/lambda/TargetType55.java
+ test/tools/javac/lambda/TargetType56.java
+ test/tools/javac/lambda/TargetType57.java
+ test/tools/javac/lambda/TargetType57.out
+ test/tools/javac/lambda/TargetType58.java
+ test/tools/javac/lambda/TargetType59.java
+ test/tools/javac/lambda/TargetType61.java
+ test/tools/javac/lambda/TargetType62.java
Changeset: bc456436c613
Author: jjg
Date: 2013-02-12 17:15 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/langtools/rev/bc456436c613
8008077: update reference impl for type-annotations
Reviewed-by: jjg
Contributed-by: wmdietl at cs.washington.edu
! src/share/classes/com/sun/tools/classfile/ClassWriter.java
! src/share/classes/com/sun/tools/classfile/TypeAnnotation.java
! src/share/classes/com/sun/tools/javac/code/TargetType.java
! src/share/classes/com/sun/tools/javac/code/TypeAnnotationPosition.java
! src/share/classes/com/sun/tools/javac/code/TypeAnnotations.java
! src/share/classes/com/sun/tools/javac/code/Types.java
! src/share/classes/com/sun/tools/javac/comp/Attr.java
! src/share/classes/com/sun/tools/javac/jvm/ClassReader.java
! src/share/classes/com/sun/tools/javac/jvm/ClassWriter.java
! src/share/classes/com/sun/tools/javac/parser/JavacParser.java
! src/share/classes/com/sun/tools/javac/tree/TreeInfo.java
! src/share/classes/com/sun/tools/javap/AnnotationWriter.java
+ test/tools/javac/annotations/typeAnnotations/failures/LazyConstantValue.java
+ test/tools/javac/annotations/typeAnnotations/failures/TypeVariable.java
! test/tools/javac/annotations/typeAnnotations/failures/VoidGenericMethod.java
+ test/tools/javac/annotations/typeAnnotations/newlocations/Lambda.java
+ test/tools/javac/annotations/typeAnnotations/referenceinfos/Lambda.java
! test/tools/javac/annotations/typeAnnotations/referenceinfos/MethodParameters.java
! test/tools/javac/annotations/typeAnnotations/referenceinfos/TypeCasts.java
Changeset: aeadaf905d78
Author: jfranck
Date: 2013-02-13 10:33 +0100
URL: http://hg.openjdk.java.net/jdk8/2d/langtools/rev/aeadaf905d78
8007279: Rename javax.l.model.element.Element.getAnnotations(Class) to getAnnotationsByType(Class)
Reviewed-by: darcy, abuckley
! src/share/classes/com/sun/tools/javac/code/Symbol.java
! src/share/classes/javax/lang/model/element/Element.java
! test/tools/javac/processing/model/element/repeatingAnnotations/ElementRepAnnoTester.java
Changeset: d04960f05593
Author: mcimadamore
Date: 2013-02-13 17:04 +0000
URL: http://hg.openjdk.java.net/jdk8/2d/langtools/rev/d04960f05593
8006345: Report Synthesized Parameters in java.lang.reflect.Parameter API
8006896: ClassReader doesn't see MethodParameters attr for method of anon inner class
8007098: Output Synthesized Parameters to MethodParameters Attributes
Summary: Correctly report synthesized and mandated parameters
Reviewed-by: mcimadamore, jjg
Contributed-by: eric.mccorkle at oracle.com
! src/share/classes/com/sun/tools/classfile/AccessFlags.java
! src/share/classes/com/sun/tools/javac/code/Flags.java
! src/share/classes/com/sun/tools/javac/code/Symbol.java
! src/share/classes/com/sun/tools/javac/comp/Lower.java
! src/share/classes/com/sun/tools/javac/comp/MemberEnter.java
! src/share/classes/com/sun/tools/javac/jvm/ClassWriter.java
! src/share/classes/com/sun/tools/javap/AttributeWriter.java
Changeset: 3f9875aa5d67
Author: lana
Date: 2013-02-13 11:25 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/langtools/rev/3f9875aa5d67
Merge
Changeset: a3aa32fe4536
Author: lana
Date: 2013-02-14 22:11 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/langtools/rev/a3aa32fe4536
Merge
From lana.steuck at oracle.com Wed Feb 20 07:23:42 2013
From: lana.steuck at oracle.com (lana.steuck at oracle.com)
Date: Wed, 20 Feb 2013 07:23:42 +0000
Subject: [OpenJDK 2D-Dev] hg: jdk8/2d/hotspot: 36 new changesets
Message-ID: <20130220072520.9689147BE0@hg.openjdk.java.net>
Changeset: 1f84c84f8e1a
Author: katleman
Date: 2013-02-14 11:43 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/hotspot/rev/1f84c84f8e1a
Added tag jdk8-b77 for changeset cdb46031e718
! .hgtags
Changeset: 1a0174612b49
Author: amurillo
Date: 2013-02-08 08:16 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/hotspot/rev/1a0174612b49
8007801: new hotspot build - hs25-b19
Reviewed-by: jcoomes
! make/hotspot_version
Changeset: 8d9fc28831cc
Author: dcubed
Date: 2013-02-06 14:31 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/hotspot/rev/8d9fc28831cc
7182152: Instrumentation hot swap test incorrect monitor count
Summary: Add/refine new tracing support using -XX:TraceRedefineClasses=16384.
Reviewed-by: coleenp, acorn, sspitsyn
! src/share/vm/oops/cpCache.cpp
! src/share/vm/oops/cpCache.hpp
! src/share/vm/oops/klassVtable.cpp
! src/share/vm/oops/klassVtable.hpp
! src/share/vm/oops/method.cpp
! src/share/vm/oops/method.hpp
! src/share/vm/prims/jvmtiRedefineClasses.cpp
! src/share/vm/prims/jvmtiRedefineClasses.hpp
! src/share/vm/prims/jvmtiRedefineClassesTrace.hpp
! src/share/vm/utilities/accessFlags.cpp
! src/share/vm/utilities/accessFlags.hpp
Changeset: 3a88007634b0
Author: ctornqvi
Date: 2013-02-08 10:42 +0100
URL: http://hg.openjdk.java.net/jdk8/2d/hotspot/rev/3a88007634b0
8007434: Write tests for 8006298
Summary: Four tests written for 8006298
Reviewed-by: mgerdin, coleenp
+ test/runtime/CommandLine/BooleanFlagWithInvalidValue.java
+ test/runtime/CommandLine/FlagWithInvalidValue.java
+ test/runtime/CommandLine/NonBooleanFlagWithInvalidBooleanPrefix.java
+ test/runtime/CommandLine/UnrecognizedVMOption.java
Changeset: 758935f7c23f
Author: sla
Date: 2013-02-08 12:48 +0100
URL: http://hg.openjdk.java.net/jdk8/2d/hotspot/rev/758935f7c23f
8006423: SA: NullPointerException in sun.jvm.hotspot.debugger.bsd.BsdThread.getContext(BsdThread.java:67)
Summary: Do not rely on mach thread port names to identify threads from SA
Reviewed-by: dholmes, minqi, rbackman
! agent/src/os/bsd/MacosxDebuggerLocal.m
! agent/src/share/classes/sun/jvm/hotspot/debugger/bsd/BsdDebugger.java
! agent/src/share/classes/sun/jvm/hotspot/debugger/bsd/BsdDebuggerLocal.java
! agent/src/share/classes/sun/jvm/hotspot/debugger/bsd/BsdThread.java
! agent/src/share/classes/sun/jvm/hotspot/runtime/bsd_amd64/BsdAMD64JavaThreadPDAccess.java
! src/os/bsd/vm/osThread_bsd.hpp
! src/os/bsd/vm/os_bsd.cpp
! src/os_cpu/bsd_x86/vm/vmStructs_bsd_x86.hpp
Changeset: 7194f764221c
Author: sla
Date: 2013-02-08 14:05 +0100
URL: http://hg.openjdk.java.net/jdk8/2d/hotspot/rev/7194f764221c
Merge
Changeset: 461a3adac4d1
Author: sspitsyn
Date: 2013-02-08 09:14 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/hotspot/rev/461a3adac4d1
Merge
! src/share/vm/oops/cpCache.cpp
! src/share/vm/oops/method.cpp
! src/share/vm/oops/method.hpp
Changeset: 8bf62bd86a4e
Author: zgu
Date: 2013-02-08 14:49 -0500
URL: http://hg.openjdk.java.net/jdk8/2d/hotspot/rev/8bf62bd86a4e
8007791: More Restricted hs_err file permission
Summary: Enforce more restricted hs_file permission
Reviewed-by: acorn, dcubed, dsamersoff
! src/share/vm/utilities/vmError.cpp
Changeset: 1ba5b18088a8
Author: zgu
Date: 2013-02-08 14:32 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/hotspot/rev/1ba5b18088a8
Merge
Changeset: 41d73c9b30a8
Author: zgu
Date: 2013-02-08 16:31 -0500
URL: http://hg.openjdk.java.net/jdk8/2d/hotspot/rev/41d73c9b30a8
8006691: Remove jvm_version_info.is_kernel_jvm field
Summary: Removed is_kernel_jvm from jvm_version_info as Kernel VM has been deprecated
Reviewed-by: mchung, coleenp
! src/share/vm/prims/jvm.cpp
! src/share/vm/prims/jvm.h
Changeset: 3f11b37f047c
Author: zgu
Date: 2013-02-08 13:55 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/hotspot/rev/3f11b37f047c
Merge
Changeset: f989aff6946f
Author: zgu
Date: 2013-02-08 16:56 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/hotspot/rev/f989aff6946f
Merge
Changeset: 927a311d00f9
Author: coleenp
Date: 2013-02-11 14:06 -0500
URL: http://hg.openjdk.java.net/jdk8/2d/hotspot/rev/927a311d00f9
8007320: NPG: move method annotations
Summary: allocate method annotations and attach to ConstMethod if present
Reviewed-by: dcubed, jiangli, sspitsyn, iklam
! agent/src/share/classes/sun/jvm/hotspot/oops/ConstMethod.java
! src/share/vm/classfile/classFileParser.cpp
! src/share/vm/classfile/classFileParser.hpp
! src/share/vm/classfile/defaultMethods.cpp
! src/share/vm/memory/heapInspection.hpp
! src/share/vm/oops/annotations.cpp
! src/share/vm/oops/annotations.hpp
! src/share/vm/oops/constMethod.cpp
! src/share/vm/oops/constMethod.hpp
! src/share/vm/oops/instanceKlass.cpp
! src/share/vm/oops/instanceKlass.hpp
! src/share/vm/oops/method.cpp
! src/share/vm/oops/method.hpp
! src/share/vm/prims/jvm.cpp
! src/share/vm/prims/jvmtiRedefineClasses.cpp
! src/share/vm/prims/jvmtiRedefineClasses.hpp
! src/share/vm/runtime/fieldDescriptor.cpp
! src/share/vm/runtime/vmStructs.cpp
+ test/runtime/8007320/ConstMethodTest.java
Changeset: 5ee2b330eacd
Author: zgu
Date: 2013-02-12 12:19 -0500
URL: http://hg.openjdk.java.net/jdk8/2d/hotspot/rev/5ee2b330eacd
8007950: Undo hs_file permission change
Summary: Reverse hs_err file permission back to 0666, as early push was premature
Reviewed-by: dsamersoff, dcubed, acorn
! src/share/vm/utilities/vmError.cpp
Changeset: deb43b8a436e
Author: sspitsyn
Date: 2013-02-13 08:42 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/hotspot/rev/deb43b8a436e
Merge
Changeset: bce1ac447f6b
Author: johnc
Date: 2013-02-06 14:50 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/hotspot/rev/bce1ac447f6b
7052429: G1: Avoid unnecessary scanning of humongous regions during concurrent marking
Summary: Skip unnecessary scanning of bitmap for unmarked humongous objects/regions.
Reviewed-by: jwilhelm, johnc
Contributed-by: Tao Mao
! src/share/vm/gc_implementation/g1/concurrentMark.cpp
! src/share/vm/runtime/globals.hpp
Changeset: f64ffbf81af5
Author: jwilhelm
Date: 2013-02-07 15:51 +0100
URL: http://hg.openjdk.java.net/jdk8/2d/hotspot/rev/f64ffbf81af5
8006432: Ratio flags should be unsigned
Summary: Flags changed to be of uintx type
Reviewed-by: johnc, tamao
! src/cpu/zero/vm/shark_globals_zero.hpp
! src/os_cpu/bsd_x86/vm/globals_bsd_x86.hpp
! src/share/vm/gc_implementation/concurrentMarkSweep/concurrentMarkSweepGeneration.cpp
! src/share/vm/gc_implementation/concurrentMarkSweep/concurrentMarkSweepGeneration.hpp
! src/share/vm/gc_implementation/g1/g1_globals.hpp
! src/share/vm/runtime/arguments.cpp
! src/share/vm/runtime/globals.hpp
Changeset: 5d8325eb8240
Author: brutisso
Date: 2013-02-07 22:04 +0100
URL: http://hg.openjdk.java.net/jdk8/2d/hotspot/rev/5d8325eb8240
Merge
! src/share/vm/runtime/thread.cpp
Changeset: 9425ba04792d
Author: brutisso
Date: 2013-02-07 18:40 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/hotspot/rev/9425ba04792d
Merge
- agent/src/share/classes/sun/jvm/hotspot/memory/BinaryTreeDictionary.java
- make/solaris/makefiles/kernel.make
! src/share/vm/runtime/arguments.cpp
- test/runtime/7158988/TestFieldMonitor.sh
Changeset: ad747ee9d0b1
Author: brutisso
Date: 2013-02-10 21:15 +0100
URL: http://hg.openjdk.java.net/jdk8/2d/hotspot/rev/ad747ee9d0b1
8002144: G1: large number of evacuation failures may lead to large c heap memory usage
Summary: Use Stack<> instead of GrowableArray to keep track of preserved marks. Also reviewed by vitalyd at gmail.com.
Reviewed-by: johnc, jcoomes
! src/share/vm/gc_implementation/g1/g1CollectedHeap.cpp
! src/share/vm/gc_implementation/g1/g1CollectedHeap.hpp
Changeset: 5e401ef52ec0
Author: johnc
Date: 2013-02-11 15:24 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/hotspot/rev/5e401ef52ec0
8007772: G1: assert(!hr->isHumongous() || mr.start() == hr->bottom()) failed: the start of HeapRegion and MemRegion should be consistent for humongous regions
Summary: In do_marking_step(), we should always give up current region after scanning the object, if the region is humongous.
Reviewed-by: brutisso, jwilhelm, tamao
! src/share/vm/gc_implementation/g1/concurrentMark.cpp
Changeset: a83cd101fd62
Author: jmasa
Date: 2013-01-23 19:08 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/hotspot/rev/a83cd101fd62
8005452: NPG: Create new flags for Metaspace resizing policy
Reviewed-by: johnc, jwilhelm, coleenp, stefank
! src/share/vm/memory/metaspace.cpp
! src/share/vm/runtime/arguments.cpp
! src/share/vm/runtime/globals.hpp
Changeset: b8d5d7a6c94c
Author: brutisso
Date: 2013-02-14 11:01 +0100
URL: http://hg.openjdk.java.net/jdk8/2d/hotspot/rev/b8d5d7a6c94c
Merge
! src/share/vm/runtime/arguments.cpp
! src/share/vm/runtime/thread.cpp
Changeset: 91a23b11d8dc
Author: kvn
Date: 2013-02-08 15:07 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/hotspot/rev/91a23b11d8dc
8007708: compiler/6855215 assert(VM_Version::supports_sse4_2())
Summary: Added missing UseSSE42 check. Also added missing avx2 assert for vpermq instruction.
Reviewed-by: roland, twisti
! src/cpu/x86/vm/assembler_x86.cpp
! src/cpu/x86/vm/macroAssembler_x86.cpp
Changeset: 309460dcedf7
Author: morris
Date: 2013-02-08 15:39 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/hotspot/rev/309460dcedf7
8006851: When TieredCompilation is set, max code cache should be bumped to 256mb
Summary: Set ReservedCodeCacheSize to (default value)*5 when TieredCompilation is on.
Reviewed-by: kvn, twisti
! src/share/vm/runtime/arguments.cpp
Changeset: 2c673161698a
Author: drchase
Date: 2013-02-09 12:55 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/hotspot/rev/2c673161698a
8007402: Code cleanup to remove Parfait false positive
Summary: add array access range check
Reviewed-by: kvn
! src/share/vm/opto/regmask.cpp
! src/share/vm/opto/regmask.hpp
Changeset: 64d2a0a39954
Author: kmo
Date: 2013-02-10 22:35 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/hotspot/rev/64d2a0a39954
8006430: TraceTypeProfile is a product flag while it should be a diagnostic flag
Summary: make sure all diagnostic and experimental flag kinds are checked in Flag::is_unlocked()
Reviewed-by: kvn
! src/share/vm/runtime/globals.cpp
Changeset: a9c29dfc7d73
Author: morris
Date: 2013-02-11 10:38 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/hotspot/rev/a9c29dfc7d73
8003251: ARM: move MacroAssembler into separate file
Summary: moved MacroAssembler into separate file
Reviewed-by: twisti, kvn, dlong
! src/share/vm/asm/macroAssembler.hpp
! src/share/vm/asm/macroAssembler.inline.hpp
Changeset: 1e5e28bac299
Author: morris
Date: 2013-02-11 14:47 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/hotspot/rev/1e5e28bac299
8003252: PPC: move MacroAssembler into separate file
Summary: moved MacroAssembler into separate file
Reviewed-by: twisti, kvn, dlong
! src/share/vm/asm/macroAssembler.hpp
! src/share/vm/asm/macroAssembler.inline.hpp
Changeset: 8b3da8d14c93
Author: roland
Date: 2013-02-12 12:56 +0100
URL: http://hg.openjdk.java.net/jdk8/2d/hotspot/rev/8b3da8d14c93
7197327: 40% regression on 8 b41 comp 8 b40 on specjvm2008.mpegaudio on oob
Summary: Add support for expensive nodes.
Reviewed-by: kvn
! src/share/vm/opto/c2_globals.hpp
! src/share/vm/opto/compile.cpp
! src/share/vm/opto/compile.hpp
! src/share/vm/opto/library_call.cpp
! src/share/vm/opto/loopnode.cpp
! src/share/vm/opto/loopnode.hpp
! src/share/vm/opto/node.cpp
! src/share/vm/opto/node.hpp
! src/share/vm/opto/phaseX.cpp
! src/share/vm/opto/subnode.hpp
Changeset: c703f9c4b025
Author: kmo
Date: 2013-02-12 07:39 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/hotspot/rev/c703f9c4b025
8002169: TEST_BUG: compiler/7009359/Test7009359.java sometimes times out
Summary: make the test less prone to timeout by reducing the amount of iteration and allowing main to be compiled
Reviewed-by: jrose
! test/compiler/7009359/Test7009359.java
Changeset: aaad39923cdb
Author: kmo
Date: 2013-02-12 14:33 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/hotspot/rev/aaad39923cdb
Merge
Changeset: 12e01444ca2d
Author: iignatyev
Date: 2013-02-13 08:29 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/hotspot/rev/12e01444ca2d
8006683: Add WhiteBox API to testing of compiler
Reviewed-by: kvn, vlivanov
! src/share/tools/whitebox/sun/hotspot/WhiteBox.java
! src/share/vm/prims/wbtestmethods/parserTests.hpp
! src/share/vm/prims/whitebox.cpp
! src/share/vm/prims/whitebox.hpp
+ test/compiler/whitebox/CompilerWhiteBoxTest.java
+ test/compiler/whitebox/DeoptimizeAllTest.java
+ test/compiler/whitebox/DeoptimizeMethodTest.java
+ test/compiler/whitebox/IsMethodCompilableTest.java
+ test/compiler/whitebox/MakeMethodNotCompilableTest.java
+ test/compiler/whitebox/SetDontInlineMethodTest.java
Changeset: 1cdf241a4b26
Author: vlivanov
Date: 2013-02-14 05:36 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/hotspot/rev/1cdf241a4b26
Merge
! src/share/vm/runtime/arguments.cpp
Changeset: 9f19f4a7d48a
Author: amurillo
Date: 2013-02-15 13:27 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/hotspot/rev/9f19f4a7d48a
Merge
Changeset: d5e12e7d2f71
Author: amurillo
Date: 2013-02-15 13:27 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/hotspot/rev/d5e12e7d2f71
Added tag hs25-b19 for changeset 9f19f4a7d48a
! .hgtags
From lana.steuck at oracle.com Wed Feb 20 07:24:39 2013
From: lana.steuck at oracle.com (lana.steuck at oracle.com)
Date: Wed, 20 Feb 2013 07:24:39 +0000
Subject: [OpenJDK 2D-Dev] hg: jdk8/2d/jdk: 43 new changesets
Message-ID: <20130220073313.E4B3347BE2@hg.openjdk.java.net>
Changeset: c1304eb051f6
Author: katleman
Date: 2013-02-14 11:44 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/c1304eb051f6
Added tag jdk8-b77 for changeset b2fc8e31cecc
! .hgtags
Changeset: ac89a5d71466
Author: alexsch
Date: 2013-02-06 18:25 +0400
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/ac89a5d71466
8000326: Focus unable to traverse in the menubar
Reviewed-by: alexsch, malenkov
! src/share/classes/javax/swing/JMenuBar.java
Changeset: 6e17465f4a1a
Author: mcherkas
Date: 2013-02-08 22:08 +0400
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/6e17465f4a1a
8005932: Java 7 on mac os x only provides text clipboard formats
Reviewed-by: alexp, denis
! src/macosx/lib/flavormap.properties
+ test/java/awt/DataFlavor/MissedHtmlAndRtfBug/AbsoluteComponentCenterCalculator.java
+ test/java/awt/DataFlavor/MissedHtmlAndRtfBug/DataFlavorSearcher.java
+ test/java/awt/DataFlavor/MissedHtmlAndRtfBug/InterprocessMessages.java
+ test/java/awt/DataFlavor/MissedHtmlAndRtfBug/MissedHtmlAndRtfBug.html
+ test/java/awt/DataFlavor/MissedHtmlAndRtfBug/MissedHtmlAndRtfBug.java
+ test/java/awt/DataFlavor/MissedHtmlAndRtfBug/MyTransferable.java
+ test/java/awt/DataFlavor/MissedHtmlAndRtfBug/NextFramePositionCalculator.java
+ test/java/awt/DataFlavor/MissedHtmlAndRtfBug/SourcePanel.java
+ test/java/awt/DataFlavor/MissedHtmlAndRtfBug/TargetPanel.java
Changeset: 5406c4e381e2
Author: kshefov
Date: 2013-02-13 18:01 +0400
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/5406c4e381e2
7161759: TEST_BUG: java/awt/Frame/WindowDragTest/WindowDragTest.java fails to compile, should be modified
Summary: Added @build Util jtreg tag
Reviewed-by: serb, alexsch
Contributed-by: Vera Akulova
! test/java/awt/Frame/WindowDragTest/WindowDragTest.java
Changeset: dd6cf41a6953
Author: kshefov
Date: 2013-02-13 19:06 +0400
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/dd6cf41a6953
7132383: [macosx] bug6596966.java should be adapted for Mac
Reviewed-by: serb, alexsch
Contributed-by: Vera Akulova
! test/javax/swing/JLabel/6596966/bug6596966.java
! test/javax/swing/regtesthelpers/Util.java
Changeset: caec64340f42
Author: vkarnauk
Date: 2013-02-13 19:23 +0400
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/caec64340f42
4199622: RFE: JComboBox shouldn't sending ActionEvents for keyboard navigation
Reviewed-by: alexp, alexsch
! src/share/classes/javax/swing/plaf/basic/BasicComboBoxUI.java
! src/share/classes/javax/swing/plaf/basic/BasicLookAndFeel.java
+ test/javax/swing/JComboBox/4199622/bug4199622.java
Changeset: 4d9691e95e05
Author: pchelko
Date: 2013-02-13 15:27 +0000
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/4d9691e95e05
7079260: InputContext leaks memory
Summary: Replaced strong refs with weak refs
Reviewed-by: art, serb
! src/share/classes/sun/awt/im/CompositionAreaHandler.java
! src/solaris/classes/sun/awt/X11InputMethod.java
+ test/java/awt/im/memoryleak/InputContextMemoryLeakTest.java
! test/java/awt/regtesthelpers/Util.java
Changeset: c552cde0e3f9
Author: pchelko
Date: 2013-02-13 15:32 +0000
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/c552cde0e3f9
8005629: javac warnings compiling java.awt.EventDispatchThread and sun.awt.X11.XIconWindow
Summary: Removed macosx specific workaround from shared code and made macosx use public API
Reviewed-by: art, serb
! src/macosx/classes/sun/lwawt/macosx/CPrinterJob.java
- src/macosx/classes/sun/lwawt/macosx/EventDispatchAccess.java
! src/macosx/native/sun/awt/CPrinterJob.m
! src/share/classes/java/awt/EventDispatchThread.java
! src/solaris/classes/sun/awt/X11/XIconWindow.java
Changeset: c95dc15ac183
Author: lana
Date: 2013-02-13 12:38 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/c95dc15ac183
Merge
- src/share/classes/java/lang/annotation/ContainedBy.java
- src/share/classes/java/lang/annotation/ContainerFor.java
- test/java/net/URL/abnormal_http_urls
- test/java/net/URL/ftp_urls
- test/java/net/URL/jar_urls
- test/java/net/URL/normal_http_urls
- test/java/net/URL/runconstructor.sh
- test/java/net/URL/share_file_urls
- test/java/net/URL/win32_file_urls
- test/sun/net/www/EncDec.doc
- test/sun/net/www/MarkResetTest.java
- test/sun/net/www/MarkResetTest.sh
- test/sun/security/util/Oid/S11N.sh
- test/sun/security/util/Oid/SerialTest.java
Changeset: c9efb349b391
Author: lana
Date: 2013-02-13 17:55 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/c9efb349b391
Merge
- src/macosx/classes/sun/lwawt/macosx/EventDispatchAccess.java
Changeset: 0e7d5dd84fdf
Author: dsamersoff
Date: 2013-02-06 16:53 +0400
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/0e7d5dd84fdf
8007277: JDK-8002048 testcase fails to compile
Summary: sun.* classes is not included to ct.sym file and symbol file have to be ignored
Reviewed-by: alanb
! test/sun/management/jdp/JdpTest.sh
Changeset: 1574fa3df1c0
Author: lancea
Date: 2013-02-06 14:15 -0500
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/1574fa3df1c0
8006505: additional changes for JSR 310 support
Reviewed-by: naoto, ulfzibis
! src/share/classes/java/sql/JDBCType.java
! src/share/classes/java/sql/SQLInput.java
! src/share/classes/java/sql/SQLOutput.java
! src/share/classes/java/sql/Types.java
Changeset: 2f1505c49e79
Author: martin
Date: 2013-02-06 17:59 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/2f1505c49e79
8006995: java launcher fails to open executable JAR > 2GB
Summary: Use O_LARGEFILE consistently when opening jar files
Reviewed-by: alanb, sherman
! src/share/bin/parse_manifest.c
Changeset: 2de8c6c2d652
Author: ykantser
Date: 2013-02-07 11:22 +0100
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/2de8c6c2d652
8007142: Add utility classes for writing better multiprocess tests in jtreg
Reviewed-by: alanb, rbackman
+ test/lib/testlibrary/OutputAnalyzerTest.java
+ test/lib/testlibrary/jdk/testlibrary/JcmdBase.java
+ test/lib/testlibrary/jdk/testlibrary/JdkFinder.java
+ test/lib/testlibrary/jdk/testlibrary/OutputAnalyzer.java
+ test/lib/testlibrary/jdk/testlibrary/OutputBuffer.java
+ test/lib/testlibrary/jdk/testlibrary/ProcessTools.java
+ test/lib/testlibrary/jdk/testlibrary/StreamPumper.java
Changeset: 79d7595abe95
Author: naoto
Date: 2013-02-08 09:35 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/79d7595abe95
8007038: ArrayIndexOutOfBoundsException on calling localizedDateTime().print() with JapaneseChrono
Reviewed-by: okutsu
! src/share/classes/sun/util/locale/provider/CalendarNameProviderImpl.java
+ test/java/util/Calendar/Bug8007038.java
Changeset: 522fb3867a3a
Author: darcy
Date: 2013-02-08 16:00 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/522fb3867a3a
8005623: Retrofit FunctionalInterface annotations to core platform interfaces
Reviewed-by: mduigou, chegar, alanb
! src/share/classes/java/io/Closeable.java
! src/share/classes/java/io/FileFilter.java
! src/share/classes/java/io/FilenameFilter.java
! src/share/classes/java/io/Flushable.java
! src/share/classes/java/lang/AutoCloseable.java
! src/share/classes/java/lang/Comparable.java
! src/share/classes/java/lang/Iterable.java
! src/share/classes/java/lang/Readable.java
! src/share/classes/java/lang/Runnable.java
! src/share/classes/java/lang/Thread.java
! src/share/classes/java/nio/file/DirectoryStream.java
! src/share/classes/java/nio/file/PathMatcher.java
! src/share/classes/java/util/Comparator.java
! src/share/classes/java/util/function/BinaryOperator.java
! src/share/classes/java/util/function/Block.java
! src/share/classes/java/util/function/DoubleBinaryOperator.java
! src/share/classes/java/util/function/DoubleBlock.java
! src/share/classes/java/util/function/DoubleFunction.java
! src/share/classes/java/util/function/DoubleSupplier.java
! src/share/classes/java/util/function/DoubleUnaryOperator.java
! src/share/classes/java/util/function/Function.java
! src/share/classes/java/util/function/IntBinaryOperator.java
! src/share/classes/java/util/function/IntBlock.java
! src/share/classes/java/util/function/IntFunction.java
! src/share/classes/java/util/function/IntSupplier.java
! src/share/classes/java/util/function/IntUnaryOperator.java
! src/share/classes/java/util/function/LongBinaryOperator.java
! src/share/classes/java/util/function/LongBlock.java
! src/share/classes/java/util/function/LongFunction.java
! src/share/classes/java/util/function/LongSupplier.java
! src/share/classes/java/util/function/LongUnaryOperator.java
! src/share/classes/java/util/function/Predicate.java
! src/share/classes/java/util/function/Supplier.java
! src/share/classes/java/util/function/UnaryOperator.java
! src/share/classes/java/util/logging/Filter.java
! src/share/classes/java/util/prefs/PreferenceChangeListener.java
Changeset: 36d25dc2b8f0
Author: dl
Date: 2013-02-09 08:35 +0000
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/36d25dc2b8f0
8005697: Add StampedLock
Reviewed-by: chegar, alanb, dice, martin
! make/java/java/FILES_java.gmk
! src/share/classes/java/util/concurrent/locks/LockSupport.java
+ src/share/classes/java/util/concurrent/locks/StampedLock.java
+ test/java/util/concurrent/locks/StampedLock/Basic.java
Changeset: d14cd2272b2d
Author: weijun
Date: 2013-02-09 16:43 +0800
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/d14cd2272b2d
8001104: Unbound SASL service: the GSSAPI/krb5 mech
Reviewed-by: valeriep
! src/share/classes/com/sun/security/auth/module/Krb5LoginModule.java
! src/share/classes/javax/security/auth/kerberos/JavaxSecurityAuthKerberosAccessImpl.java
! src/share/classes/javax/security/auth/kerberos/KeyTab.java
! src/share/classes/sun/security/jgss/LoginConfigImpl.java
! src/share/classes/sun/security/jgss/krb5/Krb5Util.java
! src/share/classes/sun/security/jgss/krb5/ServiceCreds.java
! src/share/classes/sun/security/jgss/krb5/SubjectComber.java
! src/share/classes/sun/security/krb5/JavaxSecurityAuthKerberosAccess.java
! src/share/classes/sun/security/krb5/internal/ktab/KeyTab.java
! src/share/classes/sun/security/provider/ConfigSpiFile.java
! test/sun/security/krb5/ServiceCredsCombination.java
! test/sun/security/krb5/auto/AcceptPermissions.java
+ test/sun/security/krb5/auto/GSSUnbound.java
! test/sun/security/krb5/auto/OneKDC.java
+ test/sun/security/krb5/auto/SaslUnbound.java
+ test/sun/security/krb5/auto/UnboundService.java
Changeset: 57cb988c811e
Author: weijun
Date: 2013-02-09 16:43 +0800
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/57cb988c811e
8007761: NTLM coding errors
Reviewed-by: chegar
! src/share/classes/com/sun/security/ntlm/Client.java
! src/share/classes/com/sun/security/ntlm/NTLM.java
Changeset: 58c95d0b6b1a
Author: ksrini
Date: 2013-02-10 08:07 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/58c95d0b6b1a
8007519: [unpack200] produces bad class files when producing BootstrapMethods attribute
Reviewed-by: alanb
! test/ProblemList.txt
Changeset: 520a3433883d
Author: ksrini
Date: 2013-02-10 08:49 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/520a3433883d
8007902: [unpack200] incorrect BootstrapMethods attribute
Reviewed-by: jjh
! src/share/native/com/sun/java/util/jar/pack/unpack.cpp
! test/tools/pack200/Pack200Test.java
! test/tools/pack200/pack200-verifier/data/golden.jar
Changeset: 1df991184045
Author: dsamersoff
Date: 2013-02-11 18:44 +0400
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/1df991184045
8007536: Incorrect copyright header in JDP files
Summary: Copyright header in JDP files missed the "classpath exception" rule.
Reviewed-by: mikael
! src/share/classes/sun/management/jdp/JdpBroadcaster.java
! src/share/classes/sun/management/jdp/JdpController.java
! src/share/classes/sun/management/jdp/JdpException.java
! src/share/classes/sun/management/jdp/JdpGenericPacket.java
! src/share/classes/sun/management/jdp/JdpJmxPacket.java
! src/share/classes/sun/management/jdp/JdpPacket.java
! src/share/classes/sun/management/jdp/JdpPacketReader.java
! src/share/classes/sun/management/jdp/JdpPacketWriter.java
! src/share/classes/sun/management/jdp/package-info.java
Changeset: abd530253f01
Author: dcubed
Date: 2013-02-11 10:07 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/abd530253f01
8007420: add test for 6805864 to com/sun/jdi, add test for 7182152 to java/lang/instrument
Reviewed-by: coleenp, sspitsyn
+ test/com/sun/jdi/RedefineAbstractClass.sh
+ test/java/lang/instrument/RedefineSubclassWithTwoInterfaces.sh
+ test/java/lang/instrument/RedefineSubclassWithTwoInterfacesAgent.java
+ test/java/lang/instrument/RedefineSubclassWithTwoInterfacesApp.java
+ test/java/lang/instrument/RedefineSubclassWithTwoInterfacesImpl.java
+ test/java/lang/instrument/RedefineSubclassWithTwoInterfacesImpl_1.java
+ test/java/lang/instrument/RedefineSubclassWithTwoInterfacesIntf1.java
+ test/java/lang/instrument/RedefineSubclassWithTwoInterfacesIntf2.java
+ test/java/lang/instrument/RedefineSubclassWithTwoInterfacesRemote.java
+ test/java/lang/instrument/RedefineSubclassWithTwoInterfacesTarget.java
+ test/java/lang/instrument/RedefineSubclassWithTwoInterfacesTarget_1.java
Changeset: f21a4b761424
Author: alanb
Date: 2013-02-11 20:16 +0000
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/f21a4b761424
8007405: Update java.lang.reflect API to replace SYNTHESIZED with MANDATED
Reviewed-by: darcy
! src/share/classes/java/lang/reflect/Executable.java
! src/share/classes/java/lang/reflect/Modifier.java
! src/share/classes/java/lang/reflect/Parameter.java
Changeset: 465cce29a9ed
Author: mduigou
Date: 2013-02-06 11:28 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/465cce29a9ed
8006594: Add jdk_core target to jdk/test/Makefile
Reviewed-by: alanb
! make/jprt.properties
! test/Makefile
! test/ProblemList.txt
Changeset: f7fb173ac833
Author: dsamersoff
Date: 2013-02-12 16:02 +0400
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/f7fb173ac833
8007786: JDK-8002048 testcase doesn't work on Solaris
Summary: test built in into Solaris shell doesn't have -e operator
Reviewed-by: sla, sspitsyn
! test/sun/management/jdp/JdpTest.sh
Changeset: 7dcb74c3ffba
Author: sherman
Date: 2013-02-12 09:25 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/7dcb74c3ffba
8007392: JSR 310: DateTime API Updates
8007520: Update date/time classes in j.util and j.sql packages
8007572: Replace existing jdk timezone data at /lib/zi with JSR310's tzdb
Summary: Integration of JSR310 Date/Time API for M7
Reviewed-by: darcy, alanb, naoto
Contributed-by: scolebourne at joda.org, roger.riggs at oracle.com, masayoshi.okutsu at oracle.com, patrick.zhang at oracle.com
! make/docs/CORE_PKGS.gmk
! make/java/java/FILES_java.gmk
! make/sun/Makefile
! make/sun/javazic/Makefile
+ make/sun/javazic/tzdata/gmt
+ make/sun/javazic/tzdata/jdk11_backward
! make/sun/tzdb/Makefile
! make/tools/Makefile
! make/tools/src/build/tools/javazic/Zoneinfo.java
! make/tools/src/build/tools/tzdb/TzdbZoneRulesCompiler.java
! makefiles/GendataTZDB.gmk
! makefiles/GendataTimeZone.gmk
! makefiles/GenerateData.gmk
! makefiles/Tools.gmk
! src/share/classes/java/sql/Date.java
! src/share/classes/java/sql/Time.java
! src/share/classes/java/sql/Timestamp.java
! src/share/classes/java/time/Clock.java
! src/share/classes/java/time/DayOfWeek.java
! src/share/classes/java/time/Duration.java
! src/share/classes/java/time/Instant.java
! src/share/classes/java/time/LocalDate.java
! src/share/classes/java/time/LocalDateTime.java
! src/share/classes/java/time/LocalTime.java
! src/share/classes/java/time/Month.java
+ src/share/classes/java/time/MonthDay.java
+ src/share/classes/java/time/OffsetDateTime.java
+ src/share/classes/java/time/OffsetTime.java
! src/share/classes/java/time/Period.java
- src/share/classes/java/time/PeriodParser.java
! src/share/classes/java/time/Ser.java
+ src/share/classes/java/time/Year.java
+ src/share/classes/java/time/YearMonth.java
! src/share/classes/java/time/ZoneId.java
! src/share/classes/java/time/ZoneOffset.java
! src/share/classes/java/time/ZoneRegion.java
! src/share/classes/java/time/ZonedDateTime.java
- src/share/classes/java/time/calendar/ChronoDateImpl.java
- src/share/classes/java/time/calendar/HijrahChrono.java
- src/share/classes/java/time/calendar/HijrahDate.java
- src/share/classes/java/time/calendar/HijrahDeviationReader.java
- src/share/classes/java/time/calendar/HijrahEra.java
- src/share/classes/java/time/calendar/JapaneseChrono.java
- src/share/classes/java/time/calendar/JapaneseDate.java
- src/share/classes/java/time/calendar/JapaneseEra.java
- src/share/classes/java/time/calendar/MinguoChrono.java
- src/share/classes/java/time/calendar/MinguoDate.java
- src/share/classes/java/time/calendar/MinguoEra.java
- src/share/classes/java/time/calendar/Ser.java
- src/share/classes/java/time/calendar/ThaiBuddhistChrono.java
- src/share/classes/java/time/calendar/ThaiBuddhistDate.java
- src/share/classes/java/time/calendar/ThaiBuddhistEra.java
- src/share/classes/java/time/calendar/package-info.java
+ src/share/classes/java/time/chrono/ChronoDateImpl.java
+ src/share/classes/java/time/chrono/ChronoLocalDate.java
+ src/share/classes/java/time/chrono/ChronoLocalDateTime.java
+ src/share/classes/java/time/chrono/ChronoLocalDateTimeImpl.java
+ src/share/classes/java/time/chrono/ChronoZonedDateTime.java
+ src/share/classes/java/time/chrono/ChronoZonedDateTimeImpl.java
+ src/share/classes/java/time/chrono/Chronology.java
+ src/share/classes/java/time/chrono/Era.java
+ src/share/classes/java/time/chrono/HijrahChronology.java
+ src/share/classes/java/time/chrono/HijrahDate.java
+ src/share/classes/java/time/chrono/HijrahDeviationReader.java
+ src/share/classes/java/time/chrono/HijrahEra.java
+ src/share/classes/java/time/chrono/IsoChronology.java
+ src/share/classes/java/time/chrono/IsoEra.java
+ src/share/classes/java/time/chrono/JapaneseChronology.java
+ src/share/classes/java/time/chrono/JapaneseDate.java
+ src/share/classes/java/time/chrono/JapaneseEra.java
+ src/share/classes/java/time/chrono/MinguoChronology.java
+ src/share/classes/java/time/chrono/MinguoDate.java
+ src/share/classes/java/time/chrono/MinguoEra.java
+ src/share/classes/java/time/chrono/Ser.java
+ src/share/classes/java/time/chrono/ThaiBuddhistChronology.java
+ src/share/classes/java/time/chrono/ThaiBuddhistDate.java
+ src/share/classes/java/time/chrono/ThaiBuddhistEra.java
+ src/share/classes/java/time/chrono/package-info.java
! src/share/classes/java/time/format/DateTimeBuilder.java
! src/share/classes/java/time/format/DateTimeFormatStyleProvider.java
! src/share/classes/java/time/format/DateTimeFormatter.java
! src/share/classes/java/time/format/DateTimeFormatterBuilder.java
- src/share/classes/java/time/format/DateTimeFormatters.java
! src/share/classes/java/time/format/DateTimeParseContext.java
! src/share/classes/java/time/format/DateTimePrintContext.java
- src/share/classes/java/time/format/DateTimePrintException.java
! src/share/classes/java/time/format/DateTimeTextProvider.java
! src/share/classes/java/time/format/FormatStyle.java
+ src/share/classes/java/time/format/ZoneName.java
! src/share/classes/java/time/format/package-info.java
! src/share/classes/java/time/overview.html
! src/share/classes/java/time/package-info.java
- src/share/classes/java/time/temporal/Chrono.java
! src/share/classes/java/time/temporal/ChronoField.java
- src/share/classes/java/time/temporal/ChronoLocalDate.java
- src/share/classes/java/time/temporal/ChronoLocalDateTime.java
- src/share/classes/java/time/temporal/ChronoLocalDateTimeImpl.java
! src/share/classes/java/time/temporal/ChronoUnit.java
- src/share/classes/java/time/temporal/ChronoZonedDateTime.java
- src/share/classes/java/time/temporal/ChronoZonedDateTimeImpl.java
- src/share/classes/java/time/temporal/Era.java
- src/share/classes/java/time/temporal/ISOChrono.java
- src/share/classes/java/time/temporal/ISOEra.java
- src/share/classes/java/time/temporal/ISOFields.java
+ src/share/classes/java/time/temporal/IsoFields.java
! src/share/classes/java/time/temporal/JulianFields.java
- src/share/classes/java/time/temporal/MonthDay.java
- src/share/classes/java/time/temporal/OffsetDate.java
- src/share/classes/java/time/temporal/OffsetDateTime.java
- src/share/classes/java/time/temporal/OffsetTime.java
! src/share/classes/java/time/temporal/Queries.java
- src/share/classes/java/time/temporal/Ser.java
- src/share/classes/java/time/temporal/SimplePeriod.java
! src/share/classes/java/time/temporal/Temporal.java
! src/share/classes/java/time/temporal/TemporalAccessor.java
- src/share/classes/java/time/temporal/TemporalAdder.java
! src/share/classes/java/time/temporal/TemporalAdjuster.java
+ src/share/classes/java/time/temporal/TemporalAmount.java
! src/share/classes/java/time/temporal/TemporalField.java
! src/share/classes/java/time/temporal/TemporalQuery.java
- src/share/classes/java/time/temporal/TemporalSubtractor.java
! src/share/classes/java/time/temporal/TemporalUnit.java
! src/share/classes/java/time/temporal/WeekFields.java
- src/share/classes/java/time/temporal/Year.java
- src/share/classes/java/time/temporal/YearMonth.java
! src/share/classes/java/time/temporal/package-info.java
! src/share/classes/java/time/zone/TzdbZoneRulesProvider.java
! src/share/classes/java/time/zone/ZoneOffsetTransitionRule.java
! src/share/classes/java/time/zone/ZoneRules.java
! src/share/classes/java/time/zone/ZoneRulesProvider.java
! src/share/classes/java/util/Calendar.java
! src/share/classes/java/util/Date.java
! src/share/classes/java/util/Formatter.java
! src/share/classes/java/util/GregorianCalendar.java
! src/share/classes/java/util/TimeZone.java
! src/share/classes/sun/text/resources/FormatData.java
! src/share/classes/sun/text/resources/ar/FormatData_ar.java
! src/share/classes/sun/text/resources/el/FormatData_el.java
! src/share/classes/sun/text/resources/hr/FormatData_hr.java
! src/share/classes/sun/text/resources/ja/FormatData_ja.java
! src/share/classes/sun/text/resources/ko/FormatData_ko.java
! src/share/classes/sun/text/resources/sr/FormatData_sr.java
! src/share/classes/sun/text/resources/sv/FormatData_sv.java
! src/share/classes/sun/text/resources/zh/FormatData_zh.java
! src/share/classes/sun/text/resources/zh/FormatData_zh_TW.java
! src/share/classes/sun/util/calendar/CalendarSystem.java
! src/share/classes/sun/util/calendar/LocalGregorianCalendar.java
- src/share/classes/sun/util/calendar/TzIDOldMapping.java
! src/share/classes/sun/util/calendar/ZoneInfo.java
! src/share/classes/sun/util/calendar/ZoneInfoFile.java
! src/share/classes/sun/util/locale/provider/CalendarDataUtility.java
! src/share/classes/sun/util/locale/provider/CalendarNameProviderImpl.java
! src/share/classes/sun/util/locale/provider/LocaleResources.java
+ test/java/sql/JavatimeTest.java
+ test/java/time/META-INF/services/java.time.chrono.Chronology
- test/java/time/META-INF/services/java.time.temporal.Chrono
! test/java/time/tck/java/time/AbstractTCKTest.java
+ test/java/time/tck/java/time/MockSimplePeriod.java
! test/java/time/tck/java/time/TCKClock.java
! test/java/time/tck/java/time/TCKClock_Fixed.java
! test/java/time/tck/java/time/TCKClock_Offset.java
! test/java/time/tck/java/time/TCKClock_System.java
! test/java/time/tck/java/time/TCKClock_Tick.java
! test/java/time/tck/java/time/TCKDayOfWeek.java
! test/java/time/tck/java/time/TCKDuration.java
! test/java/time/tck/java/time/TCKInstant.java
! test/java/time/tck/java/time/TCKLocalDate.java
! test/java/time/tck/java/time/TCKLocalDateTime.java
! test/java/time/tck/java/time/TCKLocalTime.java
! test/java/time/tck/java/time/TCKMonth.java
+ test/java/time/tck/java/time/TCKMonthDay.java
+ test/java/time/tck/java/time/TCKOffsetDateTime.java
+ test/java/time/tck/java/time/TCKOffsetTime.java
+ test/java/time/tck/java/time/TCKPeriod.java
+ test/java/time/tck/java/time/TCKYear.java
+ test/java/time/tck/java/time/TCKYearMonth.java
! test/java/time/tck/java/time/TCKZoneId.java
! test/java/time/tck/java/time/TCKZoneOffset.java
! test/java/time/tck/java/time/TCKZonedDateTime.java
+ test/java/time/tck/java/time/TestChronology.java
+ test/java/time/tck/java/time/TestIsoChronology.java
- test/java/time/tck/java/time/calendar/CopticChrono.java
- test/java/time/tck/java/time/calendar/CopticDate.java
- test/java/time/tck/java/time/calendar/CopticEra.java
- test/java/time/tck/java/time/calendar/TestChronoLocalDate.java
- test/java/time/tck/java/time/calendar/TestChronoLocalDateTime.java
- test/java/time/tck/java/time/calendar/TestHijrahChrono.java
- test/java/time/tck/java/time/calendar/TestJapaneseChrono.java
- test/java/time/tck/java/time/calendar/TestMinguoChrono.java
- test/java/time/tck/java/time/calendar/TestServiceLoader.java
- test/java/time/tck/java/time/calendar/TestThaiBuddhistChrono.java
+ test/java/time/tck/java/time/chrono/CopticChronology.java
+ test/java/time/tck/java/time/chrono/CopticDate.java
+ test/java/time/tck/java/time/chrono/CopticEra.java
+ test/java/time/tck/java/time/chrono/TCKChronology.java
+ test/java/time/tck/java/time/chrono/TCKTestServiceLoader.java
+ test/java/time/tck/java/time/chrono/TestChronoLocalDate.java
+ test/java/time/tck/java/time/chrono/TestChronoLocalDateTime.java
+ test/java/time/tck/java/time/chrono/TestHijrahChronology.java
+ test/java/time/tck/java/time/chrono/TestJapaneseChronology.java
+ test/java/time/tck/java/time/chrono/TestMinguoChronology.java
+ test/java/time/tck/java/time/chrono/TestThaiBuddhistChronology.java
+ test/java/time/tck/java/time/format/TCKChronoPrinterParser.java
! test/java/time/tck/java/time/format/TCKDateTimeFormatter.java
! test/java/time/tck/java/time/format/TCKDateTimeFormatterBuilder.java
! test/java/time/tck/java/time/format/TCKDateTimeFormatters.java
- test/java/time/tck/java/time/format/TCKDateTimePrintException.java
! test/java/time/tck/java/time/format/TCKDateTimeTextPrinting.java
! test/java/time/tck/java/time/format/TCKLocalizedFieldParser.java
! test/java/time/tck/java/time/format/TCKLocalizedFieldPrinter.java
+ test/java/time/tck/java/time/format/TCKLocalizedPrinterParser.java
+ test/java/time/tck/java/time/format/TCKOffsetPrinterParser.java
+ test/java/time/tck/java/time/format/TCKPadPrinterParser.java
+ test/java/time/tck/java/time/format/TCKZoneIdPrinterParser.java
- test/java/time/tck/java/time/temporal/TCKISOFields.java
+ test/java/time/tck/java/time/temporal/TCKIsoFields.java
! test/java/time/tck/java/time/temporal/TCKJulianFields.java
- test/java/time/tck/java/time/temporal/TCKMonthDay.java
- test/java/time/tck/java/time/temporal/TCKOffsetDate.java
- test/java/time/tck/java/time/temporal/TCKOffsetDateTime.java
- test/java/time/tck/java/time/temporal/TCKOffsetTime.java
- test/java/time/tck/java/time/temporal/TCKSimplePeriod.java
! test/java/time/tck/java/time/temporal/TCKWeekFields.java
- test/java/time/tck/java/time/temporal/TCKYear.java
- test/java/time/tck/java/time/temporal/TCKYearMonth.java
- test/java/time/tck/java/time/temporal/TestChrono.java
! test/java/time/tck/java/time/temporal/TestChronoLocalDate.java
! test/java/time/tck/java/time/temporal/TestChronoLocalDateTime.java
! test/java/time/tck/java/time/temporal/TestChronoZonedDateTime.java
- test/java/time/tck/java/time/temporal/TestISOChrono.java
! test/java/time/tck/java/time/zone/TCKFixedZoneRules.java
! test/java/time/tck/java/time/zone/TCKZoneOffsetTransition.java
! test/java/time/tck/java/time/zone/TCKZoneOffsetTransitionRule.java
! test/java/time/tck/java/time/zone/TCKZoneRules.java
! test/java/time/tck/java/time/zone/TCKZoneRulesProvider.java
! test/java/time/test/java/time/MockSimplePeriod.java
! test/java/time/test/java/time/TestDuration.java
! test/java/time/test/java/time/TestLocalDateTime.java
! test/java/time/test/java/time/TestLocalTime.java
+ test/java/time/test/java/time/TestMonthDay.java
+ test/java/time/test/java/time/TestOffsetDateTime.java
+ test/java/time/test/java/time/TestOffsetDateTime_instants.java
+ test/java/time/test/java/time/TestOffsetTime.java
! test/java/time/test/java/time/TestPeriod.java
- test/java/time/test/java/time/TestPeriodParser.java
+ test/java/time/test/java/time/TestYear.java
+ test/java/time/test/java/time/TestYearMonth.java
! test/java/time/test/java/time/TestZoneId.java
+ test/java/time/test/java/time/chrono/TestExampleCode.java
+ test/java/time/test/java/time/chrono/TestIsoChronoImpl.java
+ test/java/time/test/java/time/chrono/TestServiceLoader.java
! test/java/time/test/java/time/format/TestCharLiteralParser.java
! test/java/time/test/java/time/format/TestCharLiteralPrinter.java
+ test/java/time/test/java/time/format/TestDateTimeFormatterBuilder.java
- test/java/time/test/java/time/format/TestDateTimeFormatters.java
- test/java/time/test/java/time/format/TestDateTimePrintException.java
! test/java/time/test/java/time/format/TestDateTimeTextProvider.java
! test/java/time/test/java/time/format/TestFractionPrinterParser.java
+ test/java/time/test/java/time/format/TestNonIsoFormatter.java
! test/java/time/test/java/time/format/TestNumberParser.java
! test/java/time/test/java/time/format/TestNumberPrinter.java
- test/java/time/test/java/time/format/TestPadParserDecorator.java
! test/java/time/test/java/time/format/TestPadPrinterDecorator.java
! test/java/time/test/java/time/format/TestReducedParser.java
! test/java/time/test/java/time/format/TestReducedPrinter.java
! test/java/time/test/java/time/format/TestSettingsParser.java
! test/java/time/test/java/time/format/TestStringLiteralParser.java
! test/java/time/test/java/time/format/TestStringLiteralPrinter.java
! test/java/time/test/java/time/format/TestTextParser.java
! test/java/time/test/java/time/format/TestTextPrinter.java
- test/java/time/test/java/time/format/TestZoneIdParser.java
! test/java/time/test/java/time/format/TestZoneOffsetParser.java
! test/java/time/test/java/time/format/TestZoneOffsetPrinter.java
! test/java/time/test/java/time/format/TestZoneTextPrinterParser.java
+ test/java/time/test/java/time/format/ZoneName.java
! test/java/time/test/java/time/temporal/MockFieldNoValue.java
! test/java/time/test/java/time/temporal/MockFieldValue.java
! test/java/time/test/java/time/temporal/TestChronoUnit.java
! test/java/time/test/java/time/temporal/TestDateTimeBuilderCombinations.java
- test/java/time/test/java/time/temporal/TestISOChronoImpl.java
! test/java/time/test/java/time/temporal/TestJapaneseChronoImpl.java
+ test/java/time/test/java/time/temporal/TestJulianFields.java
- test/java/time/test/java/time/temporal/TestMonthDay.java
- test/java/time/test/java/time/temporal/TestOffsetDate.java
- test/java/time/test/java/time/temporal/TestOffsetDateTime.java
- test/java/time/test/java/time/temporal/TestOffsetDateTime_instants.java
- test/java/time/test/java/time/temporal/TestOffsetTime.java
! test/java/time/test/java/time/temporal/TestThaiBuddhistChronoImpl.java
- test/java/time/test/java/time/temporal/TestYear.java
- test/java/time/test/java/time/temporal/TestYearMonth.java
! test/java/time/test/java/time/zone/TestFixedZoneRules.java
! test/java/time/test/java/util/TestFormatter.java
+ test/java/util/Calendar/JavatimeTest.java
! test/java/util/TimeZone/OldIDMappingTest.java
+ test/java/util/TimeZone/TzIDOldMapping.java
+ test/sun/util/calendar/zi/BackEnd.java
+ test/sun/util/calendar/zi/Checksum.java
+ test/sun/util/calendar/zi/DayOfWeek.java
+ test/sun/util/calendar/zi/Gen.java
+ test/sun/util/calendar/zi/GenDoc.java
+ test/sun/util/calendar/zi/Main.java
+ test/sun/util/calendar/zi/Mappings.java
+ test/sun/util/calendar/zi/Month.java
+ test/sun/util/calendar/zi/Rule.java
+ test/sun/util/calendar/zi/RuleDay.java
+ test/sun/util/calendar/zi/RuleRec.java
+ test/sun/util/calendar/zi/Simple.java
+ test/sun/util/calendar/zi/TestZoneInfo310.java
+ test/sun/util/calendar/zi/Time.java
+ test/sun/util/calendar/zi/Timezone.java
+ test/sun/util/calendar/zi/TzIDOldMapping.java
+ test/sun/util/calendar/zi/Zone.java
+ test/sun/util/calendar/zi/ZoneInfoFile.java
+ test/sun/util/calendar/zi/ZoneInfoOld.java
+ test/sun/util/calendar/zi/ZoneRec.java
+ test/sun/util/calendar/zi/Zoneinfo.java
+ test/sun/util/calendar/zi/tzdata/VERSION
+ test/sun/util/calendar/zi/tzdata/africa
+ test/sun/util/calendar/zi/tzdata/antarctica
+ test/sun/util/calendar/zi/tzdata/asia
+ test/sun/util/calendar/zi/tzdata/australasia
+ test/sun/util/calendar/zi/tzdata/backward
+ test/sun/util/calendar/zi/tzdata/etcetera
+ test/sun/util/calendar/zi/tzdata/europe
+ test/sun/util/calendar/zi/tzdata/factory
+ test/sun/util/calendar/zi/tzdata/gmt
+ test/sun/util/calendar/zi/tzdata/iso3166.tab
+ test/sun/util/calendar/zi/tzdata/jdk11_backward
+ test/sun/util/calendar/zi/tzdata/leapseconds
+ test/sun/util/calendar/zi/tzdata/northamerica
+ test/sun/util/calendar/zi/tzdata/pacificnew
+ test/sun/util/calendar/zi/tzdata/solar87
+ test/sun/util/calendar/zi/tzdata/solar88
+ test/sun/util/calendar/zi/tzdata/solar89
+ test/sun/util/calendar/zi/tzdata/southamerica
+ test/sun/util/calendar/zi/tzdata/systemv
+ test/sun/util/calendar/zi/tzdata/zone.tab
+ test/sun/util/calendar/zi/tzdata_jdk/gmt
+ test/sun/util/calendar/zi/tzdata_jdk/jdk11_backward
+ test/sun/util/calendar/zi/tzdata_jdk/jdk11_full_backward
Changeset: 2cd67a8c7abc
Author: jfranck
Date: 2013-02-13 10:36 +0100
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/2cd67a8c7abc
8007278: Rename j.l.r.AnnotatedElement.getAnnotations(Class) to getAnnotationsByType(Class)
Reviewed-by: darcy, abuckley
! src/share/classes/java/lang/Class.java
! src/share/classes/java/lang/Package.java
! src/share/classes/java/lang/reflect/AccessibleObject.java
! src/share/classes/java/lang/reflect/AnnotatedElement.java
! src/share/classes/java/lang/reflect/Executable.java
! src/share/classes/java/lang/reflect/Field.java
! src/share/classes/java/lang/reflect/Parameter.java
! src/share/classes/sun/reflect/annotation/AnnotatedTypeFactory.java
! src/share/classes/sun/reflect/generics/reflectiveObjects/TypeVariableImpl.java
! test/java/lang/annotation/TypeParamAnnotation.java
! test/java/lang/annotation/repeatingAnnotations/RepeatedUnitTest.java
Changeset: cd111064d4e9
Author: zgu
Date: 2013-02-12 14:47 -0500
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/cd111064d4e9
8006691: Remove jvm_version_info->is_kernel_jvm field
Summary: Remove is_kernel_jvm field in jvm_version_info structure, as kernel VM has been deprecated
Reviewed-by: mchung
! src/share/javavm/export/jvm.h
Changeset: bf64f83aa0cd
Author: vinnie
Date: 2013-02-13 16:01 +0000
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/bf64f83aa0cd
8007934: algorithm parameters for PBE Scheme 2 not decoded correctly in PKCS12 keystore
Reviewed-by: mullan
! src/share/classes/sun/security/pkcs12/PKCS12KeyStore.java
! test/java/security/KeyStore/PBETest.java
Changeset: ceb7c712c693
Author: vinnie
Date: 2013-02-13 16:03 +0000
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/ceb7c712c693
Merge
Changeset: 8181be9a3538
Author: dsamersoff
Date: 2013-02-13 21:06 +0400
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/8181be9a3538
8008095: TEST_BUG: JDK-8002048 one more testcase failure on Solaris
Summary: fixed couple of more Solaris shell incompatibilities
Reviewed-by: chegar
! test/sun/management/jdp/JdpTest.sh
Changeset: 11438befdd4c
Author: vinnie
Date: 2013-02-13 19:40 +0000
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/11438befdd4c
8007755: Support the logical grouping of keystores
Reviewed-by: mullan
! src/share/classes/java/security/KeyStore.java
+ src/share/classes/sun/security/provider/DomainKeyStore.java
! src/share/classes/sun/security/provider/PolicyParser.java
! src/share/classes/sun/security/provider/Sun.java
! src/share/classes/sun/security/provider/SunEntries.java
! src/share/classes/sun/security/util/Resources.java
+ test/sun/security/provider/KeyStore/DKSTest.java
+ test/sun/security/provider/KeyStore/DKSTest.sh
+ test/sun/security/provider/KeyStore/domains.cfg
! test/sun/security/tools/keytool/AltProviderPath.sh
! test/sun/security/tools/keytool/DummyProvider.java
Changeset: efc66fe16f91
Author: sherman
Date: 2013-02-13 11:49 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/efc66fe16f91
8008161: Regression: j.u.TimeZone.getAvailableIDs(rawOffset) returns non-sorted list
Summary: to return a sorted list
Reviewed-by: lancea, naoto
! src/share/classes/sun/util/calendar/ZoneInfoFile.java
! test/sun/util/calendar/zi/TestZoneInfo310.java
Changeset: ff80a6b2ae9b
Author: lana
Date: 2013-02-13 11:25 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/ff80a6b2ae9b
Merge
Changeset: a5aad284904e
Author: lana
Date: 2013-02-13 11:57 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/a5aad284904e
Merge
Changeset: 83c09292f5ad
Author: ksrini
Date: 2013-02-13 12:56 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/83c09292f5ad
8005750: [parfait] Memory leak at jdk/src/share/bin/parse_manifest.c
Reviewed-by: jjh
! src/share/bin/parse_manifest.c
Changeset: b13247d5408d
Author: dcubed
Date: 2013-02-13 13:22 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/b13247d5408d
8007935: java/lang/instrument/RedefineSubclassWithTwoInterfaces.sh should use $COMPILEJAVA for javac
Reviewed-by: sspitsyn, alanb
! test/java/lang/instrument/RedefineSubclassWithTwoInterfaces.sh
Changeset: 4f520ce7ba3f
Author: acorn
Date: 2013-02-13 16:09 -0500
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/4f520ce7ba3f
8007888: jdk fix default method: VerifyError: Illegal use of nonvirtual
Summary: Recognize VM generated method in old verifier. With 8004967
Reviewed-by: coleenp, acorn
Contributed-by: bharadwaj.yadavalli at oracle.com
! src/share/javavm/export/jvm.h
! src/share/native/common/check_code.c
Changeset: e6f34051c60c
Author: acorn
Date: 2013-02-13 16:15 -0500
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/e6f34051c60c
Merge
Changeset: dc3019a336c0
Author: lana
Date: 2013-02-13 17:57 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/dc3019a336c0
Merge
- src/share/classes/java/time/PeriodParser.java
- src/share/classes/java/time/calendar/ChronoDateImpl.java
- src/share/classes/java/time/calendar/HijrahChrono.java
- src/share/classes/java/time/calendar/HijrahDate.java
- src/share/classes/java/time/calendar/HijrahDeviationReader.java
- src/share/classes/java/time/calendar/HijrahEra.java
- src/share/classes/java/time/calendar/JapaneseChrono.java
- src/share/classes/java/time/calendar/JapaneseDate.java
- src/share/classes/java/time/calendar/JapaneseEra.java
- src/share/classes/java/time/calendar/MinguoChrono.java
- src/share/classes/java/time/calendar/MinguoDate.java
- src/share/classes/java/time/calendar/MinguoEra.java
- src/share/classes/java/time/calendar/Ser.java
- src/share/classes/java/time/calendar/ThaiBuddhistChrono.java
- src/share/classes/java/time/calendar/ThaiBuddhistDate.java
- src/share/classes/java/time/calendar/ThaiBuddhistEra.java
- src/share/classes/java/time/calendar/package-info.java
- src/share/classes/java/time/format/DateTimeFormatters.java
- src/share/classes/java/time/format/DateTimePrintException.java
- src/share/classes/java/time/temporal/Chrono.java
- src/share/classes/java/time/temporal/ChronoLocalDate.java
- src/share/classes/java/time/temporal/ChronoLocalDateTime.java
- src/share/classes/java/time/temporal/ChronoLocalDateTimeImpl.java
- src/share/classes/java/time/temporal/ChronoZonedDateTime.java
- src/share/classes/java/time/temporal/ChronoZonedDateTimeImpl.java
- src/share/classes/java/time/temporal/Era.java
- src/share/classes/java/time/temporal/ISOChrono.java
- src/share/classes/java/time/temporal/ISOEra.java
- src/share/classes/java/time/temporal/ISOFields.java
- src/share/classes/java/time/temporal/MonthDay.java
- src/share/classes/java/time/temporal/OffsetDate.java
- src/share/classes/java/time/temporal/OffsetDateTime.java
- src/share/classes/java/time/temporal/OffsetTime.java
- src/share/classes/java/time/temporal/Ser.java
- src/share/classes/java/time/temporal/SimplePeriod.java
- src/share/classes/java/time/temporal/TemporalAdder.java
- src/share/classes/java/time/temporal/TemporalSubtractor.java
- src/share/classes/java/time/temporal/Year.java
- src/share/classes/java/time/temporal/YearMonth.java
- src/share/classes/sun/util/calendar/TzIDOldMapping.java
- test/java/time/META-INF/services/java.time.temporal.Chrono
- test/java/time/tck/java/time/calendar/CopticChrono.java
- test/java/time/tck/java/time/calendar/CopticDate.java
- test/java/time/tck/java/time/calendar/CopticEra.java
- test/java/time/tck/java/time/calendar/TestChronoLocalDate.java
- test/java/time/tck/java/time/calendar/TestChronoLocalDateTime.java
- test/java/time/tck/java/time/calendar/TestHijrahChrono.java
- test/java/time/tck/java/time/calendar/TestJapaneseChrono.java
- test/java/time/tck/java/time/calendar/TestMinguoChrono.java
- test/java/time/tck/java/time/calendar/TestServiceLoader.java
- test/java/time/tck/java/time/calendar/TestThaiBuddhistChrono.java
- test/java/time/tck/java/time/format/TCKDateTimePrintException.java
- test/java/time/tck/java/time/temporal/TCKISOFields.java
- test/java/time/tck/java/time/temporal/TCKMonthDay.java
- test/java/time/tck/java/time/temporal/TCKOffsetDate.java
- test/java/time/tck/java/time/temporal/TCKOffsetDateTime.java
- test/java/time/tck/java/time/temporal/TCKOffsetTime.java
- test/java/time/tck/java/time/temporal/TCKSimplePeriod.java
- test/java/time/tck/java/time/temporal/TCKYear.java
- test/java/time/tck/java/time/temporal/TCKYearMonth.java
- test/java/time/tck/java/time/temporal/TestChrono.java
- test/java/time/tck/java/time/temporal/TestISOChrono.java
- test/java/time/test/java/time/TestPeriodParser.java
- test/java/time/test/java/time/format/TestDateTimeFormatters.java
- test/java/time/test/java/time/format/TestDateTimePrintException.java
- test/java/time/test/java/time/format/TestPadParserDecorator.java
- test/java/time/test/java/time/format/TestZoneIdParser.java
- test/java/time/test/java/time/temporal/TestISOChronoImpl.java
- test/java/time/test/java/time/temporal/TestMonthDay.java
- test/java/time/test/java/time/temporal/TestOffsetDate.java
- test/java/time/test/java/time/temporal/TestOffsetDateTime.java
- test/java/time/test/java/time/temporal/TestOffsetDateTime_instants.java
- test/java/time/test/java/time/temporal/TestOffsetTime.java
- test/java/time/test/java/time/temporal/TestYear.java
- test/java/time/test/java/time/temporal/TestYearMonth.java
Changeset: 5ea0024ba765
Author: lana
Date: 2013-02-14 22:12 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/5ea0024ba765
Merge
Changeset: 41008f5cef1a
Author: lana
Date: 2013-02-19 22:10 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/41008f5cef1a
Merge
- src/macosx/classes/sun/lwawt/macosx/EventDispatchAccess.java
- src/share/classes/java/time/PeriodParser.java
- src/share/classes/java/time/calendar/ChronoDateImpl.java
- src/share/classes/java/time/calendar/HijrahChrono.java
- src/share/classes/java/time/calendar/HijrahDate.java
- src/share/classes/java/time/calendar/HijrahDeviationReader.java
- src/share/classes/java/time/calendar/HijrahEra.java
- src/share/classes/java/time/calendar/JapaneseChrono.java
- src/share/classes/java/time/calendar/JapaneseDate.java
- src/share/classes/java/time/calendar/JapaneseEra.java
- src/share/classes/java/time/calendar/MinguoChrono.java
- src/share/classes/java/time/calendar/MinguoDate.java
- src/share/classes/java/time/calendar/MinguoEra.java
- src/share/classes/java/time/calendar/Ser.java
- src/share/classes/java/time/calendar/ThaiBuddhistChrono.java
- src/share/classes/java/time/calendar/ThaiBuddhistDate.java
- src/share/classes/java/time/calendar/ThaiBuddhistEra.java
- src/share/classes/java/time/calendar/package-info.java
- src/share/classes/java/time/format/DateTimeFormatters.java
- src/share/classes/java/time/format/DateTimePrintException.java
- src/share/classes/java/time/temporal/Chrono.java
- src/share/classes/java/time/temporal/ChronoLocalDate.java
- src/share/classes/java/time/temporal/ChronoLocalDateTime.java
- src/share/classes/java/time/temporal/ChronoLocalDateTimeImpl.java
- src/share/classes/java/time/temporal/ChronoZonedDateTime.java
- src/share/classes/java/time/temporal/ChronoZonedDateTimeImpl.java
- src/share/classes/java/time/temporal/Era.java
- src/share/classes/java/time/temporal/ISOChrono.java
- src/share/classes/java/time/temporal/ISOEra.java
- src/share/classes/java/time/temporal/ISOFields.java
- src/share/classes/java/time/temporal/MonthDay.java
- src/share/classes/java/time/temporal/OffsetDate.java
- src/share/classes/java/time/temporal/OffsetDateTime.java
- src/share/classes/java/time/temporal/OffsetTime.java
- src/share/classes/java/time/temporal/Ser.java
- src/share/classes/java/time/temporal/SimplePeriod.java
- src/share/classes/java/time/temporal/TemporalAdder.java
- src/share/classes/java/time/temporal/TemporalSubtractor.java
- src/share/classes/java/time/temporal/Year.java
- src/share/classes/java/time/temporal/YearMonth.java
- src/share/classes/sun/util/calendar/TzIDOldMapping.java
- test/java/time/META-INF/services/java.time.temporal.Chrono
- test/java/time/tck/java/time/calendar/CopticChrono.java
- test/java/time/tck/java/time/calendar/CopticDate.java
- test/java/time/tck/java/time/calendar/CopticEra.java
- test/java/time/tck/java/time/calendar/TestChronoLocalDate.java
- test/java/time/tck/java/time/calendar/TestChronoLocalDateTime.java
- test/java/time/tck/java/time/calendar/TestHijrahChrono.java
- test/java/time/tck/java/time/calendar/TestJapaneseChrono.java
- test/java/time/tck/java/time/calendar/TestMinguoChrono.java
- test/java/time/tck/java/time/calendar/TestServiceLoader.java
- test/java/time/tck/java/time/calendar/TestThaiBuddhistChrono.java
- test/java/time/tck/java/time/format/TCKDateTimePrintException.java
- test/java/time/tck/java/time/temporal/TCKISOFields.java
- test/java/time/tck/java/time/temporal/TCKMonthDay.java
- test/java/time/tck/java/time/temporal/TCKOffsetDate.java
- test/java/time/tck/java/time/temporal/TCKOffsetDateTime.java
- test/java/time/tck/java/time/temporal/TCKOffsetTime.java
- test/java/time/tck/java/time/temporal/TCKSimplePeriod.java
- test/java/time/tck/java/time/temporal/TCKYear.java
- test/java/time/tck/java/time/temporal/TCKYearMonth.java
- test/java/time/tck/java/time/temporal/TestChrono.java
- test/java/time/tck/java/time/temporal/TestISOChrono.java
- test/java/time/test/java/time/TestPeriodParser.java
- test/java/time/test/java/time/format/TestDateTimeFormatters.java
- test/java/time/test/java/time/format/TestDateTimePrintException.java
- test/java/time/test/java/time/format/TestPadParserDecorator.java
- test/java/time/test/java/time/format/TestZoneIdParser.java
- test/java/time/test/java/time/temporal/TestISOChronoImpl.java
- test/java/time/test/java/time/temporal/TestMonthDay.java
- test/java/time/test/java/time/temporal/TestOffsetDate.java
- test/java/time/test/java/time/temporal/TestOffsetDateTime.java
- test/java/time/test/java/time/temporal/TestOffsetDateTime_instants.java
- test/java/time/test/java/time/temporal/TestOffsetTime.java
- test/java/time/test/java/time/temporal/TestYear.java
- test/java/time/test/java/time/temporal/TestYearMonth.java
From artem.ananiev at oracle.com Wed Feb 20 13:43:49 2013
From: artem.ananiev at oracle.com (Artem Ananiev)
Date: Wed, 20 Feb 2013 17:43:49 +0400
Subject: [OpenJDK 2D-Dev] [8] Review request for 8005607:
Recursion in J2DXErrHandler() Causes a Stack Overflow on Linux
In-Reply-To: <512251F0.7020204@oracle.com>
References: <50DEF3AC.2070600@oracle.com> <50ED9609.1070002@oracle.com>
<50EE1CEB.5080500@oracle.com> <50EFED12.6070700@oracle.com>
<50EFFB4F.4090405@oracle.com> <510946F7.3090709@oracle.com>
<510A6A54.1080705@oracle.com> <510A90DF.2060501@oracle.com>
<511BC5E4.9090302@oracle.com> <51220DA4.7040608@oracle.com>
<512218E7.9090808@oracle.com> <51221D2D.9070907@oracle.com>
<512251F0.7020204@oracle.com>
Message-ID: <5124D315.8080809@oracle.com>
Looks fine.
Thanks,
Artem
On 2/18/2013 8:08 PM, Anton Litvinov wrote:
> Hello Artem,
>
> Could you please review a new version of the fix. The method
> "XErrorHandlerUtil.getDisplay()" was removed and
> "XErrorHandlerUtil.XSync()" method refers to the field "display"
> directly now.
>
> Webrev: http://cr.openjdk.java.net/~alitvinov/8005607/webrev.03
>
> Thank you,
> Anton
>
> On 2/18/2013 4:23 PM, Artem Ananiev wrote:
>>
>> On 2/18/2013 4:04 PM, Anton Litvinov wrote:
>>> Hello Artem,
>>>
>>> Thank you very much for the review of this fix. My responses to your
>>> questions are provided below in the same order, which you defined.
>>>
>>> 1. I think that "XErrorHandlerUtil.saved_error" field can surely be
>>> marked as private, but in this case the corresponding
>>> "XErrorHandlerUtil.getSavedError" method will be necessary, because
>>> this field is actively accessed from other classes which set a
>>> certain instance of XErrorHandler. For example
>>> "MotifDnDDropTargetProtocol.java", "XDragSourceProtocol.java" and a
>>> few other classes edited in this fix.
>>
>> OK, I missed that usages when looking at the webrev. Let it stay
>> unchanged now.
>>
>>> 2. Yes, I completely agree that "XErrorHandlerUtil.getDisplay()" is
>>> reduntant. This method will be eliminated.
>>
>> Thanks,
>>
>> Artem
>>
>>> Thank you,
>>> Anton
>>>
>>> On 2/18/2013 3:16 PM, Artem Ananiev wrote:
>>>> Hi, Anton,
>>>>
>>>> a few minor comments:
>>>>
>>>> 1. XErrorHandlerUtil: can saved_error be private instead of package
>>>> protected?
>>>>
>>>> 2. XErrorHandlerUtil.getDisplay() seems to be redundant.
>>>>
>>>> In general, the fix looks perfectly fine to me. Please, wait for
>>>> comments from Java2D team, though.
>>>>
>>>> Thanks,
>>>>
>>>> Artem
>>>>
>>>> On 2/13/2013 8:57 PM, Anton Litvinov wrote:
>>>>> Hello Anthony,
>>>>>
>>>>> Could you please review the third version of the fix containing
>>>>> modifications discussed with you in the previous letter.
>>>>>
>>>>> Webrev: http://cr.openjdk.java.net/~alitvinov/8005607/webrev.02
>>>>>
>>>>> This version of the fix differs from the previous in the following
>>>>> places:
>>>>>
>>>>> 1. A comment about the place of invocation of the method
>>>>> "XErrorHandlerUtil.init" was added to a documentation block of the
>>>>> method.
>>>>> 2. A code related to XShmAttach function common to the files
>>>>> "src/solaris/native/sun/awt/awt_GraphicsEnv.c" and
>>>>> "src/solaris/native/sun/java2d/x11/X11SurfaceData.c" was extracted
>>>>> into a separate function "TryXShmAttach" declared in
>>>>> "src/solaris/native/sun/awt/awt_GraphicsEnv.h" file.
>>>>> 3. All JNI code related to X error handling was implemented as
>>>>> corresponding macros defined in
>>>>> "src/solaris/native/sun/awt/awt_util.h" file.
>>>>>
>>>>> Thank you,
>>>>> Anton
>>>>>
>>>>> On 1/31/2013 7:42 PM, Anton Litvinov wrote:
>>>>>> Hello Anthony,
>>>>>>
>>>>>> Thank you for the review and these remarks. Surely, the comment will
>>>>>> be added. I think that all JNI code related to XShmAttach can be
>>>>>> definitely transferred into a separate dedicated function, which will
>>>>>> be declared in "src/solaris/native/sun/awt/awt_GraphicsEnv.h" file. I
>>>>>> will try to wrap all JNU calls connected with XErrorHandler into the
>>>>>> particular "WITH_XERROR_HANDLER", "RESTORE_XERROR_HANDLER" functions
>>>>>> or macros.
>>>>>>
>>>>>> Thank you,
>>>>>> Anton
>>>>>>
>>>>>> On 1/31/2013 4:57 PM, Anthony Petrov wrote:
>>>>>>> Hi Anton,
>>>>>>>
>>>>>>> A couple comments:
>>>>>>>
>>>>>>> 1. src/solaris/classes/sun/awt/X11/XErrorHandlerUtil.java
>>>>>>>> 80 private static void init(long display) {
>>>>>>>
>>>>>>> This method is private and isn't called from anywhere in this class
>>>>>>> itself. This looks confusing. Please add a comment stating that this
>>>>>>> method is invoked from native code, and from where exactly.
>>>>>>>
>>>>>>>
>>>>>>> 2. Interesting that we use this machinery to call the XShmAttach()
>>>>>>> from native code twice, and the code looks quite similar in each
>>>>>>> case. Would it be possible to extract the common code in a separate
>>>>>>> function (a-la BOOL TryXShmAttach(...)) to avoid code replication?
>>>>>>> There are other usages as well, so we could also introduce a macro
>>>>>>> (such as the old EXEC_WITH_XERROR_HANDLER but now with other
>>>>>>> arguments) that would minimize all the JNU_ calls required to use
>>>>>>> this machinery.
>>>>>>>
>>>>>>>
>>>>>>> Otherwise the fix looks great.
>>>>>>>
>>>>>>> --
>>>>>>> best regards,
>>>>>>> Anthony
>>>>>>>
>>>>>>> On 1/30/2013 20:14, Anton Litvinov wrote:
>>>>>>>> Hello Anthony,
>>>>>>>>
>>>>>>>> Could you, please, review a second version of the fix, which is
>>>>>>>> based on an idea of reusing the existing AWT native global error
>>>>>>>> handler from Java 2D native code.
>>>>>>>>
>>>>>>>> Webrev: http://cr.openjdk.java.net/~alitvinov/8005607/webrev.01
>>>>>>>>
>>>>>>>> The fix consists of the following parts:
>>>>>>>>
>>>>>>>> 1. Migration of all X error handling code from XToolkit to a new
>>>>>>>> XErrorHandlerUtil class for resolution of interdependency
>>>>>>>> between
>>>>>>>> a static initialization block of XToolkit and a block
>>>>>>>> initializing
>>>>>>>> java.awt.GraphicsEnvironment singleton. Such dependency is
>>>>>>>> created
>>>>>>>> by new calls to XToolkit static methods from
>>>>>>>> "src/solaris/native/sun/awt/awt_GraphicsEnv.c",
>>>>>>>> "src/solaris/native/sun/java2d/x11/X11SurfaceData.c" files.
>>>>>>>> 2. Substitution of XToolkit.WITH_XERROR_HANDLER,
>>>>>>>> XToolkit.RESTORE_XERROR_HANDLER ... for corresponding
>>>>>>>> methods,
>>>>>>>> fields of XErrorHandlerUtil class in all places of JDK source
>>>>>>>> code, where they were used.
>>>>>>>> 3. Substitution of all found native X error handlers which are
>>>>>>>> set in
>>>>>>>> native code (awt_GraphicsEnv.c, X11SurfaceData.c,
>>>>>>>> GLXSurfaceData.c) for new synthetic Java error handlers.
>>>>>>>> 4. Removal of X error handling code used by the native error
>>>>>>>> handlers
>>>>>>>> from "solaris/native/sun/awt/awt_util.c"
>>>>>>>> "solaris/native/sun/awt/awt_util.h" files.
>>>>>>>>
>>>>>>>> Thank you,
>>>>>>>> Anton
>>>>>>>>
>>>>>>>> On 1/11/2013 3:45 PM, Anthony Petrov wrote:
>>>>>>>>> I'm not Jim, but as I indicated earlier my opinion is that the
>>>>>>>>> easiest way to fix this is to install the existing
>>>>>>>>> J2DXErrHandler()
>>>>>>>>> only once. That is, it is the second option listed by you. Of
>>>>>>>>> course, the J2DXErrHandler needs to be updated as well to detect
>>>>>>>>> whether 2D code wants to use it at the moment or it must simply
>>>>>>>>> delegate to the previous handler (i.e. where the code currently
>>>>>>>>> installs/uninstalls the handler, it must instead set a global
>>>>>>>>> boolean flag or something.)
>>>>>>>>>
>>>>>>>>> While the first option (reusing the existing AWT machinery) is an
>>>>>>>>> interesting idea in general, I think it is complex and would
>>>>>>>>> require too much additional testing and bring an unjustified risk
>>>>>>>>> to the solution for such a basic problem.
>>>>>>>>>
>>>>>>>>> --
>>>>>>>>> best regards,
>>>>>>>>> Anthony
>>>>>>>>>
>>>>>>>>> On 1/11/2013 14:44, Anton Litvinov wrote:
>>>>>>>>>> Hello Jim,
>>>>>>>>>>
>>>>>>>>>> Thank you very much for the review and provision of a new idea of
>>>>>>>>>> a solution. Elimination of the logic, which sets/unsets
>>>>>>>>>> J2DXErrHandler() for each call "XShmAttach(awt_display,
>>>>>>>>>> &shminfo))" should effectively resolve the issue, but only in
>>>>>>>>>> case
>>>>>>>>>> if all other native error handlers, which were set by the system
>>>>>>>>>> function "XSetErrorHandler()" in JDK or in any external library,
>>>>>>>>>> observe the rule of relaying of all events, which are not
>>>>>>>>>> relative
>>>>>>>>>> to them, to the previously saved error handlers. Otherwise an
>>>>>>>>>> error generated during "XShmAttach" function call will not be
>>>>>>>>>> handled by J2DXErrHandler().
>>>>>>>>>>
>>>>>>>>>> Could you answer the following question. By setting
>>>>>>>>>> J2DXErrHandler() only once and forever do you mean usage of AWT
>>>>>>>>>> global event handler "static int ToolkitErrorHandler(Display *
>>>>>>>>>> dpy, XErrorEvent * event)" from
>>>>>>>>>> "src/solaris/native/sun/xawt/XlibWrapper.c" with Java synthetic
>>>>>>>>>> handlers or creation of another global native error handler with
>>>>>>>>>> J2DXErrHandler as native synthetic handler?
>>>>>>>>>>
>>>>>>>>>> Thank you,
>>>>>>>>>> Anton
>>>>>>>>>>
>>>>>>>>>> On 1/10/2013 5:44 AM, Jim Graham wrote:
>>>>>>>>>>> I think I'd rather see some way to prevent double-adding the
>>>>>>>>>>> handler in the first place as well. Since it is only ever used
>>>>>>>>>>> on errors I also think it is OK to set it once and leave it
>>>>>>>>>>> there
>>>>>>>>>>> forever...
>>>>>>>>>>>
>>>>>>>>>>> ...jim
>>>>>>>>>>>
>>>>>>>>>>> On 1/9/13 8:08 AM, Anthony Petrov wrote:
>>>>>>>>>>>> Hi Anton et al.,
>>>>>>>>>>>>
>>>>>>>>>>>> If I read the description of the bug correctly, specifically
>>>>>>>>>>>> this part:
>>>>>>>>>>>>
>>>>>>>>>>>>> The problem occurs, if another thread (for example, GTK
>>>>>>>>>>>>> thread) is
>>>>>>>>>>>>> doing the same sort of thing concurrently. This can lead to
>>>>>>>>>>>>> the
>>>>>>>>>>>>> following situation.
>>>>>>>>>>>>> JVM thread: Sets J2DXErrHandler(), saves
>>>>>>>>>>>>> ANY_PREVIOUS_HANDLER as
>>>>>>>>>>>>> previous GTK thread: Sets some GTK_HANDLER, saves
>>>>>>>>>>>>> J2DXErrHandler() as previous JVM thread: Restores
>>>>>>>>>>>>> ANY_PREVIOUS_HANDLER GTK thread: Restores
>>>>>>>>>>>>> J2DXErrHandler() JVM
>>>>>>>>>>>>> thread: Sets J2DXErrHandler(), saves J2DXErrHandler() as
>>>>>>>>>>>>> previous
>>>>>>>>>>>>
>>>>>>>>>>>> It is obvious that at this final step 2D is in an inconsistent
>>>>>>>>>>>> state. We
>>>>>>>>>>>> don't expect to replace our own error handler (and it shouldn't
>>>>>>>>>>>> have
>>>>>>>>>>>> been there in the first place).
>>>>>>>>>>>>
>>>>>>>>>>>> I realize that the fix you propose works around this problem.
>>>>>>>>>>>> But this
>>>>>>>>>>>> doesn't look like an ideal solution to me.
>>>>>>>>>>>>
>>>>>>>>>>>> BTW, IIRC, in JDK7 (and 6?) we decided to set the actual X11
>>>>>>>>>>>> error
>>>>>>>>>>>> handler only once and never replace it. All the rest of the
>>>>>>>>>>>> push_handler/pop_handler logic is now located in Java code (see
>>>>>>>>>>>> XToolkit.SAVED_ERROR_HANDLER() and the surrounding logic). I
>>>>>>>>>>>> believe
>>>>>>>>>>>> that we should somehow share this machinery with the 2D code to
>>>>>>>>>>>> avoid
>>>>>>>>>>>> this sort of problems. Though I'm not sure if this will
>>>>>>>>>>>> eliminate this
>>>>>>>>>>>> exact issue.
>>>>>>>>>>>>
>>>>>>>>>>>>
>>>>>>>>>>>> 2D/AWT folks: any other thoughts?
>>>>>>>>>>>>
>>>>>>>>>>>> --
>>>>>>>>>>>> best regards,
>>>>>>>>>>>> Anthony
>>>>>>>>>>>>
>>>>>>>>>>>> On 12/29/2012 17:44, Anton Litvinov wrote:
>>>>>>>>>>>>> Hello,
>>>>>>>>>>>>>
>>>>>>>>>>>>> Please review the following fix for a bug.
>>>>>>>>>>>>>
>>>>>>>>>>>>> Bug:
>>>>>>>>>>>>> http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8005607
>>>>>>>>>>>>> https://jbs.oracle.com/bugs/browse/JDK-8005607
>>>>>>>>>>>>> Webrev:
>>>>>>>>>>>>> http://cr.openjdk.java.net/~alitvinov/8005607/webrev.00
>>>>>>>>>>>>>
>>>>>>>>>>>>> The bug consists in a crash which is caused by a stack
>>>>>>>>>>>>> overflow
>>>>>>>>>>>>> for
>>>>>>>>>>>>> the reason of an infinite recursion in AWT native function
>>>>>>>>>>>>> J2DXErrHandler() under certain circumstances on 32-bit Linux
>>>>>>>>>>>>> OS. The
>>>>>>>>>>>>> fix is based on introduction of the logic, which detects
>>>>>>>>>>>>> indirect
>>>>>>>>>>>>> recursive calls to J2DXErrHandler() by means of a simple
>>>>>>>>>>>>> counter, to
>>>>>>>>>>>>> J2DXErrHandler() native function. Such a solution requires
>>>>>>>>>>>>> minimum
>>>>>>>>>>>>> code changes, does not alter the handler's code significantly
>>>>>>>>>>>>> and
>>>>>>>>>>>>> eliminates this bug.
>>>>>>>>>>>>>
>>>>>>>>>>>>> Adding 2d-dev at openjdk.java.net e-mail alias to the list of
>>>>>>>>>>>>> recipients
>>>>>>>>>>>>> of this letter, because the edited function's name is related
>>>>>>>>>>>>> to Java
>>>>>>>>>>>>> 2D area of JDK, despite of the fact that the edited file is
>>>>>>>>>>>>> located in
>>>>>>>>>>>>> AWT directory.
>>>>>>>>>>>>>
>>>>>>>>>>>>> Thank you,
>>>>>>>>>>>>> Anton
>>>>>
>>>
>
From anton.litvinov at oracle.com Thu Feb 21 14:53:48 2013
From: anton.litvinov at oracle.com (Anton Litvinov)
Date: Thu, 21 Feb 2013 18:53:48 +0400
Subject: [OpenJDK 2D-Dev] [8] Review request for 8007642: Media Names on
Java Print Do Not Match the Printer's and Confuse Users
In-Reply-To: <51152343.8060303@oracle.com>
References: <51152343.8060303@oracle.com>
Message-ID: <512634FC.8060504@oracle.com>
Hello,
I am sorry for inconvenience. This is a reminder message. I am still
interested in reception of the response to this review request and just
want to be sure that it is not lost on the mail alias's archive.
Thank you,
Anton
On 2/8/2013 8:09 PM, Anton Litvinov wrote:
> Hello,
>
> Please review the following fix for a bug.
>
> Bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8007642
> Webrev: http://cr.openjdk.java.net/~alitvinov/8007642/webrev.00
>
> The bug consists in the fact that Java cross-platform Page Setup and
> Print dialogs do not always list all media sizes supported by a
> printer. The fix is based on addition of dynamic creation of new media
> names of the type "sun.print.Win32MediaSize" based on paper names
> received from Windows API function with corresponding media sizes of
> the type "javax.print.attribute.standard.MediaSize" for the case, when
> the printer's media size name, which is being analyzed in
> "sun.print.Win32PrintService.initMedia" method, is not added to the
> final list of media sizes supported by the printer because of an
> already existing duplicate in that list. In such a case the printer's
> paper size matches with one of the media sizes registered in JDK,
> while the paper size ID does not match with any ID known to JDK.
>
> Also the code in "Win32PrintService.findWin32Media" method was altered
> to allow three cases from "switch" block to work as expected, because
> currently they never match with "dmIndex" value, since it is always
> less then "dmPaperToPrintService.length" under "if" statement.
>
> Thank you,
> Anton
From Sergey.Bylokhov at oracle.com Fri Feb 22 18:05:36 2013
From: Sergey.Bylokhov at oracle.com (Sergey Bylokhov)
Date: Fri, 22 Feb 2013 22:05:36 +0400
Subject: [OpenJDK 2D-Dev] [8] Request for review: 8008660 : Failure in 2D
Queue Flusher thread on Mac
Message-ID: <5127B370.4010203@oracle.com>
Hello,
Please review the fix for jdk 8. Fix will be ported to jdk7 as well.
There is a few problems:
1 To eliminate the crash we should check that the system return correct
CGLConfigInfofrom in CGLGraphicsConfig.getConfig(). (see
http://bugs.sun.com/view_bug.do?bug_id=6755274)
2 Requested opengl attributes are too strict, so we cannot fail-back to
the "software renderer".
Next attrs were removed:
NSOpenGLPFANoRecovery, -> if an accelerated renderer fails due to lack
of resources, OpenGL automatically switches to another renderer.
NSOpenGLPFAAccelerated, -> accelerated renderers are still preferred.
NSOpenGLPFAFullScreen, -> we don't use this functionality.
Also in this code we try to mix "CoreGraphics display id" and "Screen
index". Currently we never use "screen index" so all related code was
removed/renamed to be more obvious.
Bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8008660
Webrev can be found at: http://cr.openjdk.java.net/~serb/8008660/webrev.00
--
Best regards, Sergey.
From swingler at apple.com Fri Feb 22 18:34:52 2013
From: swingler at apple.com (Mike Swingler)
Date: Fri, 22 Feb 2013 10:34:52 -0800
Subject: [OpenJDK 2D-Dev] [8] Request for review: 8008660 : Failure in
2D Queue Flusher thread on Mac
In-Reply-To: <5127B370.4010203@oracle.com>
References: <5127B370.4010203@oracle.com>
Message-ID: <36824436-3E09-4540-8265-B0C5C53ACC55@apple.com>
On Feb 22, 2013, at 10:05 AM, Sergey Bylokhov wrote:
> Hello,
> Please review the fix for jdk 8. Fix will be ported to jdk7 as well.
> There is a few problems:
> 1 To eliminate the crash we should check that the system return correct CGLConfigInfofrom in CGLGraphicsConfig.getConfig(). (see http://bugs.sun.com/view_bug.do?bug_id=6755274)
> 2 Requested opengl attributes are too strict, so we cannot fail-back to the "software renderer".
> Next attrs were removed:
> NSOpenGLPFANoRecovery, -> if an accelerated renderer fails due to lack of resources, OpenGL automatically switches to another renderer.
> NSOpenGLPFAAccelerated, -> accelerated renderers are still preferred.
> NSOpenGLPFAFullScreen, -> we don't use this functionality.
>
> Also in this code we try to mix "CoreGraphics display id" and "Screen index". Currently we never use "screen index" so all related code was removed/renamed to be more obvious.
>
> Bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8008660
> Webrev can be found at: http://cr.openjdk.java.net/~serb/8008660/webrev.00
This looks good. I like the simplification.
Regards,
Mike Swingler
Apple Inc.
From jennifer.godinez at oracle.com Fri Feb 22 19:01:55 2013
From: jennifer.godinez at oracle.com (jennifer.godinez at oracle.com)
Date: Fri, 22 Feb 2013 19:01:55 +0000
Subject: [OpenJDK 2D-Dev] hg: jdk8/2d/jdk: 8006110: pageDialog is showing
the swing dialog with DialogTypeSelection.NATIVE
Message-ID: <20130222190303.1E72B47CCB@hg.openjdk.java.net>
Changeset: d2d7da120c37
Author: jgodinez
Date: 2013-02-22 11:01 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/d2d7da120c37
8006110: pageDialog is showing the swing dialog with DialogTypeSelection.NATIVE
Reviewed-by: bae, prr
! src/share/classes/sun/print/RasterPrinterJob.java
From jennifer.godinez at oracle.com Fri Feb 22 21:20:40 2013
From: jennifer.godinez at oracle.com (jennifer.godinez at oracle.com)
Date: Fri, 22 Feb 2013 21:20:40 +0000
Subject: [OpenJDK 2D-Dev] hg: jdk8/2d/jdk: 8005796: [parfait] Possible
uninitialised variable at
jdk/src/share/native/sun/java2d/loops/ByteBinary1Bit.c
Message-ID: <20130222212111.8A69247D0E@hg.openjdk.java.net>
Changeset: 99c1f910abcc
Author: jgodinez
Date: 2013-02-22 13:20 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/99c1f910abcc
8005796: [parfait] Possible uninitialised variable at jdk/src/share/native/sun/java2d/loops/ByteBinary1Bit.c
Reviewed-by: prr, vadim, flar
Contributed-by: jia-hong.chen at oracle.com
! src/share/native/sun/java2d/loops/AnyByteBinary.h
! src/share/native/sun/java2d/loops/ByteIndexed.h
! src/share/native/sun/java2d/loops/IntArgb.h
! src/share/native/sun/java2d/loops/IntArgbBm.h
! src/share/native/sun/java2d/loops/IntArgbPre.h
! src/share/native/sun/java2d/loops/Ushort4444Argb.h
! src/share/native/sun/java2d/loops/UshortIndexed.h
From lana.steuck at oracle.com Sat Feb 23 07:13:35 2013
From: lana.steuck at oracle.com (lana.steuck at oracle.com)
Date: Sat, 23 Feb 2013 07:13:35 +0000
Subject: [OpenJDK 2D-Dev] hg: jdk8/2d: 9 new changesets
Message-ID: <20130223071336.47DFD47D61@hg.openjdk.java.net>
Changeset: ffb4d2e95140
Author: erikj
Date: 2013-02-15 10:40 +0100
URL: http://hg.openjdk.java.net/jdk8/2d/rev/ffb4d2e95140
8005879: Add -DMAC_OS_X_VERSION_MAX_ALLOWED=1070 to builds on Mac
Reviewed-by: ohair
! common/autoconf/generated-configure.sh
! common/autoconf/spec.gmk.in
! common/autoconf/toolchain.m4
Changeset: b0642df54d63
Author: erikj
Date: 2013-02-18 10:46 +0100
URL: http://hg.openjdk.java.net/jdk8/2d/rev/b0642df54d63
Merge
Changeset: b80abec66e70
Author: bpatel
Date: 2013-01-21 00:29 -0500
URL: http://hg.openjdk.java.net/jdk8/2d/rev/b80abec66e70
8006124: javadoc/doclet should be updated to support profiles
Reviewed-by: jjg, dholmes
! common/makefiles/javadoc/Javadoc.gmk
Changeset: 7ed0c9db6943
Author: dholmes
Date: 2013-01-21 01:50 -0500
URL: http://hg.openjdk.java.net/jdk8/2d/rev/7ed0c9db6943
8004265: Add build support for Compact Profiles
Reviewed-by: erikj, ohair
! NewMakefile.gmk
! common/autoconf/generated-configure.sh
! common/makefiles/Main.gmk
Changeset: 2f8fd30f02e6
Author: dholmes
Date: 2013-01-22 19:30 -0500
URL: http://hg.openjdk.java.net/jdk8/2d/rev/2f8fd30f02e6
Merge
! common/autoconf/generated-configure.sh
Changeset: bebeaa04ab8e
Author: dholmes
Date: 2013-02-04 18:08 -0500
URL: http://hg.openjdk.java.net/jdk8/2d/rev/bebeaa04ab8e
Merge
! common/autoconf/generated-configure.sh
! common/makefiles/javadoc/Javadoc.gmk
Changeset: 28071e4ca1de
Author: dholmes
Date: 2013-02-17 16:44 -0500
URL: http://hg.openjdk.java.net/jdk8/2d/rev/28071e4ca1de
Merge
! common/autoconf/generated-configure.sh
! common/makefiles/Main.gmk
Changeset: fd1a5574cf68
Author: dholmes
Date: 2013-02-18 15:35 -0500
URL: http://hg.openjdk.java.net/jdk8/2d/rev/fd1a5574cf68
Merge
! common/autoconf/generated-configure.sh
Changeset: 91d35211e744
Author: katleman
Date: 2013-02-21 11:12 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/rev/91d35211e744
Added tag jdk8-b78 for changeset fd1a5574cf68
! .hgtags
From lana.steuck at oracle.com Sat Feb 23 07:13:34 2013
From: lana.steuck at oracle.com (lana.steuck at oracle.com)
Date: Sat, 23 Feb 2013 07:13:34 +0000
Subject: [OpenJDK 2D-Dev] hg: jdk8/2d/corba: Added tag jdk8-b78 for
changeset 27d6368ae8ba
Message-ID: <20130223071337.C522247D64@hg.openjdk.java.net>
Changeset: e41fb1aa0329
Author: katleman
Date: 2013-02-21 11:12 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/corba/rev/e41fb1aa0329
Added tag jdk8-b78 for changeset 27d6368ae8ba
! .hgtags
From lana.steuck at oracle.com Sat Feb 23 07:13:34 2013
From: lana.steuck at oracle.com (lana.steuck at oracle.com)
Date: Sat, 23 Feb 2013 07:13:34 +0000
Subject: [OpenJDK 2D-Dev] hg: jdk8/2d/jaxp: Added tag jdk8-b78 for changeset
00958c5a7070
Message-ID: <20130223071342.17BA647D6A@hg.openjdk.java.net>
Changeset: 58fa065dd5d6
Author: katleman
Date: 2013-02-21 11:13 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/jaxp/rev/58fa065dd5d6
Added tag jdk8-b78 for changeset 00958c5a7070
! .hgtags
From lana.steuck at oracle.com Sat Feb 23 07:13:36 2013
From: lana.steuck at oracle.com (lana.steuck at oracle.com)
Date: Sat, 23 Feb 2013 07:13:36 +0000
Subject: [OpenJDK 2D-Dev] hg: jdk8/2d/hotspot: Added tag jdk8-b78 for
changeset d5e12e7d2f71
Message-ID: <20130223071343.0957047D6D@hg.openjdk.java.net>
Changeset: db3359133cdd
Author: katleman
Date: 2013-02-21 11:12 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/hotspot/rev/db3359133cdd
Added tag jdk8-b78 for changeset d5e12e7d2f71
! .hgtags
From lana.steuck at oracle.com Sat Feb 23 07:13:36 2013
From: lana.steuck at oracle.com (lana.steuck at oracle.com)
Date: Sat, 23 Feb 2013 07:13:36 +0000
Subject: [OpenJDK 2D-Dev] hg: jdk8/2d/nashorn: Added tag jdk8-b78 for
changeset 000000000000
Message-ID: <20130223071336.2D62447D5E@hg.openjdk.java.net>
Changeset: 8f49d8121c7e
Author: katleman
Date: 2013-02-21 11:13 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/nashorn/rev/8f49d8121c7e
Added tag jdk8-b78 for changeset 000000000000
+ .hgtags
From lana.steuck at oracle.com Sat Feb 23 07:13:36 2013
From: lana.steuck at oracle.com (lana.steuck at oracle.com)
Date: Sat, 23 Feb 2013 07:13:36 +0000
Subject: [OpenJDK 2D-Dev] hg: jdk8/2d/jaxws: Added tag jdk8-b78 for
changeset 391de4c992d1
Message-ID: <20130223071341.1F88747D67@hg.openjdk.java.net>
Changeset: 70d8658d2a30
Author: katleman
Date: 2013-02-21 11:13 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/jaxws/rev/70d8658d2a30
Added tag jdk8-b78 for changeset 391de4c992d1
! .hgtags
From lana.steuck at oracle.com Sat Feb 23 07:13:53 2013
From: lana.steuck at oracle.com (lana.steuck at oracle.com)
Date: Sat, 23 Feb 2013 07:13:53 +0000
Subject: [OpenJDK 2D-Dev] hg: jdk8/2d/langtools: 5 new changesets
Message-ID: <20130223071412.628E547D70@hg.openjdk.java.net>
Changeset: 5f0731e4e5e6
Author: bpatel
Date: 2013-01-21 00:45 -0500
URL: http://hg.openjdk.java.net/jdk8/2d/langtools/rev/5f0731e4e5e6
8006124: javadoc/doclet should be updated to support profiles
Reviewed-by: jjg
! src/share/classes/com/sun/tools/doclets/formats/html/AbstractPackageIndexWriter.java
+ src/share/classes/com/sun/tools/doclets/formats/html/AbstractProfileIndexWriter.java
! src/share/classes/com/sun/tools/doclets/formats/html/ClassWriterImpl.java
! src/share/classes/com/sun/tools/doclets/formats/html/FrameOutputWriter.java
! src/share/classes/com/sun/tools/doclets/formats/html/HtmlDoclet.java
! src/share/classes/com/sun/tools/doclets/formats/html/HtmlDocletWriter.java
! src/share/classes/com/sun/tools/doclets/formats/html/PackageIndexFrameWriter.java
! src/share/classes/com/sun/tools/doclets/formats/html/PackageIndexWriter.java
+ src/share/classes/com/sun/tools/doclets/formats/html/ProfileIndexFrameWriter.java
+ src/share/classes/com/sun/tools/doclets/formats/html/ProfilePackageFrameWriter.java
+ src/share/classes/com/sun/tools/doclets/formats/html/ProfilePackageIndexFrameWriter.java
+ src/share/classes/com/sun/tools/doclets/formats/html/ProfilePackageWriterImpl.java
+ src/share/classes/com/sun/tools/doclets/formats/html/ProfileWriterImpl.java
! src/share/classes/com/sun/tools/doclets/formats/html/WriterFactoryImpl.java
! src/share/classes/com/sun/tools/doclets/formats/html/markup/HtmlConstants.java
! src/share/classes/com/sun/tools/doclets/formats/html/markup/HtmlWriter.java
! src/share/classes/com/sun/tools/doclets/formats/html/resources/standard.properties
! src/share/classes/com/sun/tools/doclets/internal/toolkit/AbstractDoclet.java
! src/share/classes/com/sun/tools/doclets/internal/toolkit/Configuration.java
+ src/share/classes/com/sun/tools/doclets/internal/toolkit/ProfilePackageSummaryWriter.java
+ src/share/classes/com/sun/tools/doclets/internal/toolkit/ProfileSummaryWriter.java
! src/share/classes/com/sun/tools/doclets/internal/toolkit/WriterFactory.java
! src/share/classes/com/sun/tools/doclets/internal/toolkit/builders/BuilderFactory.java
+ src/share/classes/com/sun/tools/doclets/internal/toolkit/builders/ProfilePackageSummaryBuilder.java
+ src/share/classes/com/sun/tools/doclets/internal/toolkit/builders/ProfileSummaryBuilder.java
! src/share/classes/com/sun/tools/doclets/internal/toolkit/resources/doclet.xml
! src/share/classes/com/sun/tools/doclets/internal/toolkit/resources/doclets.properties
! src/share/classes/com/sun/tools/doclets/internal/toolkit/resources/stylesheet.css
! src/share/classes/com/sun/tools/doclets/internal/toolkit/util/DocPaths.java
! src/share/classes/com/sun/tools/doclets/internal/toolkit/util/MetaKeywords.java
+ test/com/sun/javadoc/testProfiles/TestProfiles.java
+ test/com/sun/javadoc/testProfiles/pkg1/Class1Pkg1.java
+ test/com/sun/javadoc/testProfiles/pkg1/Class2Pkg1.java
+ test/com/sun/javadoc/testProfiles/pkg1/Class3Pkg1.java
+ test/com/sun/javadoc/testProfiles/pkg1/Interface1Pkg1.java
+ test/com/sun/javadoc/testProfiles/pkg2/Anno1Pkg2.java
+ test/com/sun/javadoc/testProfiles/pkg2/Anno2Pkg2.java
+ test/com/sun/javadoc/testProfiles/pkg2/Class1Pkg2.java
+ test/com/sun/javadoc/testProfiles/pkg3/Class1Pkg3.java
+ test/com/sun/javadoc/testProfiles/pkg3/Class2Pkg3.java
+ test/com/sun/javadoc/testProfiles/pkg3/Interface1Pkg3.java
+ test/com/sun/javadoc/testProfiles/pkg4/Anno1Pkg4.java
+ test/com/sun/javadoc/testProfiles/pkg4/Class1Pkg4.java
+ test/com/sun/javadoc/testProfiles/pkg5/Class1Pkg5.java
+ test/com/sun/javadoc/testProfiles/pkg5/Interface1Pkg5.java
+ test/com/sun/javadoc/testProfiles/profile-rtjar-includes.txt
Changeset: 475eb15dfdad
Author: jjg
Date: 2013-01-21 01:27 -0500
URL: http://hg.openjdk.java.net/jdk8/2d/langtools/rev/475eb15dfdad
8004182: Add support for profiles in javac
Reviewed-by: mcimadamore
! src/share/classes/com/sun/tools/javac/api/JavacTaskImpl.java
! src/share/classes/com/sun/tools/javac/code/Flags.java
! src/share/classes/com/sun/tools/javac/code/Symtab.java
! src/share/classes/com/sun/tools/javac/comp/Attr.java
! src/share/classes/com/sun/tools/javac/comp/Check.java
! src/share/classes/com/sun/tools/javac/jvm/ClassReader.java
+ src/share/classes/com/sun/tools/javac/jvm/Profile.java
! src/share/classes/com/sun/tools/javac/jvm/Target.java
! src/share/classes/com/sun/tools/javac/main/Main.java
! src/share/classes/com/sun/tools/javac/main/Option.java
! src/share/classes/com/sun/tools/javac/resources/compiler.properties
! src/share/classes/com/sun/tools/javac/resources/javac.properties
! src/share/classes/com/sun/tools/javac/sym/CreateSymbols.java
+ src/share/classes/com/sun/tools/javac/sym/Profiles.java
! src/share/classes/com/sun/tools/javac/util/AbstractDiagnosticFormatter.java
! src/share/classes/com/sun/tools/javac/util/RichDiagnosticFormatter.java
+ test/tools/javac/diags/examples/NotInProfile.java
+ test/tools/javac/profiles/ProfileOptionTest.java
Changeset: f91144b7da75
Author: dholmes
Date: 2013-02-04 18:08 -0500
URL: http://hg.openjdk.java.net/jdk8/2d/langtools/rev/f91144b7da75
Merge
! src/share/classes/com/sun/tools/doclets/formats/html/HtmlDocletWriter.java
! src/share/classes/com/sun/tools/doclets/internal/toolkit/Configuration.java
! src/share/classes/com/sun/tools/doclets/internal/toolkit/resources/doclets.properties
! src/share/classes/com/sun/tools/javac/code/Flags.java
! src/share/classes/com/sun/tools/javac/code/Symtab.java
! src/share/classes/com/sun/tools/javac/comp/Attr.java
! src/share/classes/com/sun/tools/javac/comp/Check.java
! src/share/classes/com/sun/tools/javac/jvm/ClassReader.java
! src/share/classes/com/sun/tools/javac/resources/compiler.properties
! src/share/classes/com/sun/tools/javac/util/RichDiagnosticFormatter.java
- test/tools/javac/annotations/repeatingAnnotations/MissingContainedBy.java
- test/tools/javac/annotations/repeatingAnnotations/MissingContainerFor.java
- test/tools/javac/annotations/repeatingAnnotations/UseWrongContainedBy.java
- test/tools/javac/annotations/repeatingAnnotations/UseWrongContainerFor.java
- test/tools/javac/annotations/repeatingAnnotations/WrongContainedBy.java
- test/tools/javac/annotations/repeatingAnnotations/WrongContainerFor.java
- test/tools/javac/diags/examples/ContainedByDocumentedMismatch.java
- test/tools/javac/diags/examples/ContainedByInheritedMismatch.java
- test/tools/javac/diags/examples/ContainedByNoValue.java
- test/tools/javac/diags/examples/ContainedByNonDefault.java
- test/tools/javac/diags/examples/ContainedByRetentionMismatch.java
- test/tools/javac/diags/examples/ContainedByTargetMismatch.java
- test/tools/javac/diags/examples/ContainedByWrongValueType.java
- test/tools/javac/diags/examples/InferredDoNotConformToLower.java
- test/tools/javac/diags/examples/NoUniqueMaximalInstance.java
- test/tools/javac/diags/examples/WrongContainedBy.java
- test/tools/javac/diags/examples/WrongContainerFor.java
- test/tools/javac/lambda/MethodReference26.out
- test/tools/javac/lambda/TargetType06.out
- test/tools/javac/lambda/TargetType11.out
- test/tools/javac/lambda/TargetType45.out
- test/tools/javac/lambda/VoidCompatibility.out
- test/tools/javac/typeAnnotations/newlocations/BasicTest.java
- test/tools/javac/typeAnnotations/newlocations/BasicTest.out
Changeset: af8417e590f4
Author: dholmes
Date: 2013-02-17 16:44 -0500
URL: http://hg.openjdk.java.net/jdk8/2d/langtools/rev/af8417e590f4
Merge
! src/share/classes/com/sun/tools/doclets/formats/html/PackageIndexWriter.java
! src/share/classes/com/sun/tools/javac/code/Flags.java
! src/share/classes/com/sun/tools/javac/comp/Attr.java
! src/share/classes/com/sun/tools/javac/comp/Check.java
! src/share/classes/com/sun/tools/javac/jvm/ClassReader.java
- test/tools/javac/lambda/TargetType20.out
- test/tools/javac/lambda/TargetType50.out
Changeset: 56dfafbb9e1a
Author: katleman
Date: 2013-02-21 11:13 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/langtools/rev/56dfafbb9e1a
Added tag jdk8-b78 for changeset af8417e590f4
! .hgtags
From lana.steuck at oracle.com Sat Feb 23 07:14:58 2013
From: lana.steuck at oracle.com (lana.steuck at oracle.com)
Date: Sat, 23 Feb 2013 07:14:58 +0000
Subject: [OpenJDK 2D-Dev] hg: jdk8/2d/jdk: 25 new changesets
Message-ID: <20130223071954.8896B47D75@hg.openjdk.java.net>
Changeset: 90707943f83c
Author: erikj
Date: 2013-02-15 10:41 +0100
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/90707943f83c
8005879: Add -DMAC_OS_X_VERSION_MAX_ALLOWED=1070 to builds on Mac
Reviewed-by: ohair
! make/common/Defs-macosx.gmk
Changeset: 9a693ebd5595
Author: erikj
Date: 2013-02-18 10:48 +0100
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/9a693ebd5595
Merge
- src/macosx/classes/sun/lwawt/macosx/EventDispatchAccess.java
- src/share/classes/java/time/PeriodParser.java
- src/share/classes/java/time/calendar/ChronoDateImpl.java
- src/share/classes/java/time/calendar/HijrahChrono.java
- src/share/classes/java/time/calendar/HijrahDate.java
- src/share/classes/java/time/calendar/HijrahDeviationReader.java
- src/share/classes/java/time/calendar/HijrahEra.java
- src/share/classes/java/time/calendar/JapaneseChrono.java
- src/share/classes/java/time/calendar/JapaneseDate.java
- src/share/classes/java/time/calendar/JapaneseEra.java
- src/share/classes/java/time/calendar/MinguoChrono.java
- src/share/classes/java/time/calendar/MinguoDate.java
- src/share/classes/java/time/calendar/MinguoEra.java
- src/share/classes/java/time/calendar/Ser.java
- src/share/classes/java/time/calendar/ThaiBuddhistChrono.java
- src/share/classes/java/time/calendar/ThaiBuddhistDate.java
- src/share/classes/java/time/calendar/ThaiBuddhistEra.java
- src/share/classes/java/time/calendar/package-info.java
- src/share/classes/java/time/format/DateTimeFormatters.java
- src/share/classes/java/time/format/DateTimePrintException.java
- src/share/classes/java/time/temporal/Chrono.java
- src/share/classes/java/time/temporal/ChronoLocalDate.java
- src/share/classes/java/time/temporal/ChronoLocalDateTime.java
- src/share/classes/java/time/temporal/ChronoLocalDateTimeImpl.java
- src/share/classes/java/time/temporal/ChronoZonedDateTime.java
- src/share/classes/java/time/temporal/ChronoZonedDateTimeImpl.java
- src/share/classes/java/time/temporal/Era.java
- src/share/classes/java/time/temporal/ISOChrono.java
- src/share/classes/java/time/temporal/ISOEra.java
- src/share/classes/java/time/temporal/ISOFields.java
- src/share/classes/java/time/temporal/MonthDay.java
- src/share/classes/java/time/temporal/OffsetDate.java
- src/share/classes/java/time/temporal/OffsetDateTime.java
- src/share/classes/java/time/temporal/OffsetTime.java
- src/share/classes/java/time/temporal/Ser.java
- src/share/classes/java/time/temporal/SimplePeriod.java
- src/share/classes/java/time/temporal/TemporalAdder.java
- src/share/classes/java/time/temporal/TemporalSubtractor.java
- src/share/classes/java/time/temporal/Year.java
- src/share/classes/java/time/temporal/YearMonth.java
- src/share/classes/sun/util/calendar/TzIDOldMapping.java
- test/java/time/META-INF/services/java.time.temporal.Chrono
- test/java/time/tck/java/time/calendar/CopticChrono.java
- test/java/time/tck/java/time/calendar/CopticDate.java
- test/java/time/tck/java/time/calendar/CopticEra.java
- test/java/time/tck/java/time/calendar/TestChronoLocalDate.java
- test/java/time/tck/java/time/calendar/TestChronoLocalDateTime.java
- test/java/time/tck/java/time/calendar/TestHijrahChrono.java
- test/java/time/tck/java/time/calendar/TestJapaneseChrono.java
- test/java/time/tck/java/time/calendar/TestMinguoChrono.java
- test/java/time/tck/java/time/calendar/TestServiceLoader.java
- test/java/time/tck/java/time/calendar/TestThaiBuddhistChrono.java
- test/java/time/tck/java/time/format/TCKDateTimePrintException.java
- test/java/time/tck/java/time/temporal/TCKISOFields.java
- test/java/time/tck/java/time/temporal/TCKMonthDay.java
- test/java/time/tck/java/time/temporal/TCKOffsetDate.java
- test/java/time/tck/java/time/temporal/TCKOffsetDateTime.java
- test/java/time/tck/java/time/temporal/TCKOffsetTime.java
- test/java/time/tck/java/time/temporal/TCKSimplePeriod.java
- test/java/time/tck/java/time/temporal/TCKYear.java
- test/java/time/tck/java/time/temporal/TCKYearMonth.java
- test/java/time/tck/java/time/temporal/TestChrono.java
- test/java/time/tck/java/time/temporal/TestISOChrono.java
- test/java/time/test/java/time/TestPeriodParser.java
- test/java/time/test/java/time/format/TestDateTimeFormatters.java
- test/java/time/test/java/time/format/TestDateTimePrintException.java
- test/java/time/test/java/time/format/TestPadParserDecorator.java
- test/java/time/test/java/time/format/TestZoneIdParser.java
- test/java/time/test/java/time/temporal/TestISOChronoImpl.java
- test/java/time/test/java/time/temporal/TestMonthDay.java
- test/java/time/test/java/time/temporal/TestOffsetDate.java
- test/java/time/test/java/time/temporal/TestOffsetDateTime.java
- test/java/time/test/java/time/temporal/TestOffsetDateTime_instants.java
- test/java/time/test/java/time/temporal/TestOffsetTime.java
- test/java/time/test/java/time/temporal/TestYear.java
- test/java/time/test/java/time/temporal/TestYearMonth.java
Changeset: fb7e3edf22b2
Author: erikj
Date: 2013-02-18 11:26 +0100
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/fb7e3edf22b2
8008294: build-infra: Build-infra closed fails on solaris 11.1
Reviewed-by: ohrstrom, dholmes, tbell
! makefiles/CompileDemos.gmk
! makefiles/CompileNativeLibraries.gmk
Changeset: a23b0df73324
Author: erikj
Date: 2013-02-18 11:27 +0100
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/a23b0df73324
8008295: build-infra: Cleanup in Import.gmk
Reviewed-by: ohrstrom, tbell
! makefiles/Import.gmk
Changeset: 32549d339437
Author: bpatel
Date: 2013-01-21 00:31 -0500
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/32549d339437
8006124: javadoc/doclet should be updated to support profiles
Reviewed-by: jjg, dholmes
! make/docs/Makefile
Changeset: 80afadbf967d
Author: alanb
Date: 2013-01-21 01:46 -0500
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/80afadbf967d
8004182: Add support for profiles in javac
Reviewed-by: dholmes
! make/common/Release.gmk
Changeset: 353b88963430
Author: dholmes
Date: 2013-01-21 21:54 -0500
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/353b88963430
8006651: build-infra: Import.gmk needs to add support for the minimal VM
Reviewed-by: erikj, ohair
! makefiles/Import.gmk
Changeset: 7096f51288ab
Author: dholmes
Date: 2013-01-21 23:17 -0500
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/7096f51288ab
8004265: Add build support for Compact Profiles
Reviewed-by: erikj, ohair
! make/tools/src/build/tools/jarreorder/JarReorder.java
! makefiles/BuildJdk.gmk
! makefiles/CreateJars.gmk
! makefiles/Images.gmk
+ makefiles/ProfileNames.gmk
+ makefiles/Profiles.gmk
Changeset: ccd0aceb1190
Author: alanb
Date: 2013-01-21 23:20 -0500
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/ccd0aceb1190
8003255: (profiles) Update JAR file specification to support profiles
Reviewed-by: dholmes, mchung, ksrini
! src/share/classes/java/net/URLClassLoader.java
! src/share/classes/java/util/jar/Attributes.java
+ src/share/classes/java/util/jar/UnsupportedProfileException.java
! src/share/classes/sun/launcher/LauncherHelper.java
! src/share/classes/sun/launcher/resources/launcher.properties
! src/share/classes/sun/misc/URLClassPath.java
! src/share/classes/sun/tools/jar/Main.java
! src/share/classes/sun/tools/jar/resources/jar.properties
+ test/java/net/URLClassLoader/profiles/Basic.java
+ test/java/net/URLClassLoader/profiles/Lib.java
+ test/java/net/URLClassLoader/profiles/basic.sh
+ test/tools/jar/AddAndUpdateProfile.java
+ test/tools/launcher/profiles/Basic.java
+ test/tools/launcher/profiles/Logging.java
+ test/tools/launcher/profiles/Main.java
Changeset: c024147205f6
Author: alanb
Date: 2013-01-21 23:21 -0500
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/c024147205f6
8003256: (profiles) Add support for profile identification
Reviewed-by: dholmes, mchung, ksrini
! make/java/version/Makefile
! makefiles/GensrcMisc.gmk
! src/share/classes/sun/misc/Version.java.template
+ test/tools/launcher/profiles/VersionCheck.java
Changeset: 4b3434f5f509
Author: alanb
Date: 2013-01-21 23:23 -0500
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/4b3434f5f509
8004931: add/removePropertyChangeListener should not exist in subset Profiles of Java SE
Reviewed-by: dholmes, mchung, ksrini
+ make/tools/src/build/tools/RemoveMethods.java
! makefiles/Tools.gmk
! src/share/classes/java/util/jar/Pack200.java
! src/share/classes/java/util/logging/LogManager.java
+ test/java/util/logging/Reflect.java
+ test/tools/pack200/NoBeans.java
+ test/tools/pack200/Reflect.java
Changeset: d9cfe581c8fe
Author: alanb
Date: 2013-01-21 23:35 -0500
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/d9cfe581c8fe
8004502: Compact Profiles contents
Reviewed-by: dholmes, mchung
+ makefiles/profile-includes.txt
+ makefiles/profile-rtjar-includes.txt
+ test/java/lang/SecurityManager/NoAWT.java
+ test/java/security/cert/CertStore/NoLDAP.java
+ test/javax/management/remote/mandatory/connection/NoIIOP.java
+ test/javax/naming/InitialContext/NoApplet.java
+ test/sun/net/www/protocol/http/NoNTLM.java
+ test/sun/security/ssl/sanity/ciphersuites/NoKerberos.java
Changeset: d1b29d290ebd
Author: dholmes
Date: 2013-01-22 19:31 -0500
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/d1b29d290ebd
Merge
Changeset: 0918d6d9c18b
Author: dholmes
Date: 2013-01-22 20:04 -0500
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/0918d6d9c18b
8006667: Merge issue: Profile attribute need to be examined before custom attributes
Summary: swap profile checking and FXHelper checking
Reviewed-by: alanb
! src/share/classes/sun/launcher/LauncherHelper.java
Changeset: 77668918a388
Author: dholmes
Date: 2013-02-04 18:08 -0500
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/77668918a388
Merge
! make/docs/Makefile
! makefiles/CreateJars.gmk
! makefiles/GensrcMisc.gmk
! makefiles/Tools.gmk
! src/share/classes/sun/misc/URLClassPath.java
- test/java/rmi/activation/ActivationSystem/unregisterGroup/CallbackInterface.java
- test/java/rmi/activation/ActivationSystem/unregisterGroup/Callback_Stub.java
- test/java/rmi/activation/ActivationSystem/unregisterGroup/UnregisterGroup_Stub.java
Changeset: f308a689c049
Author: dholmes
Date: 2013-02-17 16:44 -0500
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/f308a689c049
Merge
! makefiles/Tools.gmk
- src/macosx/classes/sun/lwawt/macosx/EventDispatchAccess.java
- src/share/classes/java/lang/annotation/ContainedBy.java
- src/share/classes/java/lang/annotation/ContainerFor.java
- src/share/classes/java/time/PeriodParser.java
- src/share/classes/java/time/calendar/ChronoDateImpl.java
- src/share/classes/java/time/calendar/HijrahChrono.java
- src/share/classes/java/time/calendar/HijrahDate.java
- src/share/classes/java/time/calendar/HijrahDeviationReader.java
- src/share/classes/java/time/calendar/HijrahEra.java
- src/share/classes/java/time/calendar/JapaneseChrono.java
- src/share/classes/java/time/calendar/JapaneseDate.java
- src/share/classes/java/time/calendar/JapaneseEra.java
- src/share/classes/java/time/calendar/MinguoChrono.java
- src/share/classes/java/time/calendar/MinguoDate.java
- src/share/classes/java/time/calendar/MinguoEra.java
- src/share/classes/java/time/calendar/Ser.java
- src/share/classes/java/time/calendar/ThaiBuddhistChrono.java
- src/share/classes/java/time/calendar/ThaiBuddhistDate.java
- src/share/classes/java/time/calendar/ThaiBuddhistEra.java
- src/share/classes/java/time/calendar/package-info.java
- src/share/classes/java/time/format/DateTimeFormatters.java
- src/share/classes/java/time/format/DateTimePrintException.java
- src/share/classes/java/time/temporal/Chrono.java
- src/share/classes/java/time/temporal/ChronoLocalDate.java
- src/share/classes/java/time/temporal/ChronoLocalDateTime.java
- src/share/classes/java/time/temporal/ChronoLocalDateTimeImpl.java
- src/share/classes/java/time/temporal/ChronoZonedDateTime.java
- src/share/classes/java/time/temporal/ChronoZonedDateTimeImpl.java
- src/share/classes/java/time/temporal/Era.java
- src/share/classes/java/time/temporal/ISOChrono.java
- src/share/classes/java/time/temporal/ISOEra.java
- src/share/classes/java/time/temporal/ISOFields.java
- src/share/classes/java/time/temporal/MonthDay.java
- src/share/classes/java/time/temporal/OffsetDate.java
- src/share/classes/java/time/temporal/OffsetDateTime.java
- src/share/classes/java/time/temporal/OffsetTime.java
- src/share/classes/java/time/temporal/Ser.java
- src/share/classes/java/time/temporal/SimplePeriod.java
- src/share/classes/java/time/temporal/TemporalAdder.java
- src/share/classes/java/time/temporal/TemporalSubtractor.java
- src/share/classes/java/time/temporal/Year.java
- src/share/classes/java/time/temporal/YearMonth.java
- src/share/classes/sun/util/calendar/TzIDOldMapping.java
- test/java/net/URL/abnormal_http_urls
- test/java/net/URL/ftp_urls
- test/java/net/URL/jar_urls
- test/java/net/URL/normal_http_urls
- test/java/net/URL/runconstructor.sh
- test/java/net/URL/share_file_urls
- test/java/net/URL/win32_file_urls
- test/java/time/META-INF/services/java.time.temporal.Chrono
- test/java/time/tck/java/time/calendar/CopticChrono.java
- test/java/time/tck/java/time/calendar/CopticDate.java
- test/java/time/tck/java/time/calendar/CopticEra.java
- test/java/time/tck/java/time/calendar/TestChronoLocalDate.java
- test/java/time/tck/java/time/calendar/TestChronoLocalDateTime.java
- test/java/time/tck/java/time/calendar/TestHijrahChrono.java
- test/java/time/tck/java/time/calendar/TestJapaneseChrono.java
- test/java/time/tck/java/time/calendar/TestMinguoChrono.java
- test/java/time/tck/java/time/calendar/TestServiceLoader.java
- test/java/time/tck/java/time/calendar/TestThaiBuddhistChrono.java
- test/java/time/tck/java/time/format/TCKDateTimePrintException.java
- test/java/time/tck/java/time/temporal/TCKISOFields.java
- test/java/time/tck/java/time/temporal/TCKMonthDay.java
- test/java/time/tck/java/time/temporal/TCKOffsetDate.java
- test/java/time/tck/java/time/temporal/TCKOffsetDateTime.java
- test/java/time/tck/java/time/temporal/TCKOffsetTime.java
- test/java/time/tck/java/time/temporal/TCKSimplePeriod.java
- test/java/time/tck/java/time/temporal/TCKYear.java
- test/java/time/tck/java/time/temporal/TCKYearMonth.java
- test/java/time/tck/java/time/temporal/TestChrono.java
- test/java/time/tck/java/time/temporal/TestISOChrono.java
- test/java/time/test/java/time/TestPeriodParser.java
- test/java/time/test/java/time/format/TestDateTimeFormatters.java
- test/java/time/test/java/time/format/TestDateTimePrintException.java
- test/java/time/test/java/time/format/TestPadParserDecorator.java
- test/java/time/test/java/time/format/TestZoneIdParser.java
- test/java/time/test/java/time/temporal/TestISOChronoImpl.java
- test/java/time/test/java/time/temporal/TestMonthDay.java
- test/java/time/test/java/time/temporal/TestOffsetDate.java
- test/java/time/test/java/time/temporal/TestOffsetDateTime.java
- test/java/time/test/java/time/temporal/TestOffsetDateTime_instants.java
- test/java/time/test/java/time/temporal/TestOffsetTime.java
- test/java/time/test/java/time/temporal/TestYear.java
- test/java/time/test/java/time/temporal/TestYearMonth.java
- test/sun/net/www/EncDec.doc
- test/sun/net/www/MarkResetTest.java
- test/sun/net/www/MarkResetTest.sh
- test/sun/security/util/Oid/S11N.sh
- test/sun/security/util/Oid/SerialTest.java
Changeset: 16c684b2ab82
Author: alanb
Date: 2013-02-18 08:57 +0000
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/16c684b2ab82
8007436: (profiles) Add JSR-310 to Compact Profiles contents
Reviewed-by: dholmes, erikj
! makefiles/profile-includes.txt
! makefiles/profile-rtjar-includes.txt
Changeset: c24bc91caa67
Author: dholmes
Date: 2013-02-18 15:35 -0500
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/c24bc91caa67
Merge
! makefiles/Import.gmk
Changeset: b46c75e221c7
Author: dholmes
Date: 2013-02-19 06:27 -0500
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/b46c75e221c7
8008424: Isolate PROFILE make variable from incidental setting in the environment
Reviewed-by: erikj, alanb
! makefiles/BuildJdk.gmk
Changeset: 6f4615fd32da
Author: alanb
Date: 2013-02-19 11:08 +0000
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/6f4615fd32da
8007097: (profiles) Build needs test to ensure that profile definitions are updated
Reviewed-by: dholmes, erikj
- make/tools/src/build/tools/RemoveMethods.java
+ make/tools/src/build/tools/classfile/RemoveMethods.java
+ make/tools/src/build/tools/deps/CheckDeps.java
+ make/tools/src/build/tools/deps/refs.allowed
! makefiles/Images.gmk
! makefiles/Tools.gmk
! makefiles/profile-rtjar-includes.txt
Changeset: 033f2707ef32
Author: alanb
Date: 2013-02-19 11:32 +0000
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/033f2707ef32
Merge
Changeset: 00b7535d743f
Author: dholmes
Date: 2013-02-19 17:32 -0500
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/00b7535d743f
8008481: Dependency analyzer needs exclusion for profile builds with JFR disabled
Reviewed-by: alanb
! make/tools/src/build/tools/deps/refs.allowed
Changeset: bb97c93e4fd7
Author: katleman
Date: 2013-02-21 11:13 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/bb97c93e4fd7
Added tag jdk8-b78 for changeset 00b7535d743f
! .hgtags
Changeset: 934f5f10107d
Author: lana
Date: 2013-02-22 11:37 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/934f5f10107d
Merge
Changeset: 4fd6048a78c0
Author: lana
Date: 2013-02-22 23:12 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/4fd6048a78c0
Merge
From andrew.brygin at oracle.com Wed Feb 27 09:45:44 2013
From: andrew.brygin at oracle.com (Andrew Brygin)
Date: Wed, 27 Feb 2013 13:45:44 +0400
Subject: [OpenJDK 2D-Dev] [8] Request for review: 8008660 : Failure in
2D Queue Flusher thread on Mac
In-Reply-To: <5127B370.4010203@oracle.com>
References: <5127B370.4010203@oracle.com>
Message-ID: <512DD5C8.4060302@oracle.com>
Hi Sergey,
the fix looks fine to me.
Thanks,
Andrew
On 2/22/2013 10:05 PM, Sergey Bylokhov wrote:
> Hello,
> Please review the fix for jdk 8. Fix will be ported to jdk7 as well.
> There is a few problems:
> 1 To eliminate the crash we should check that the system return
> correct CGLConfigInfofrom in CGLGraphicsConfig.getConfig(). (see
> http://bugs.sun.com/view_bug.do?bug_id=6755274)
> 2 Requested opengl attributes are too strict, so we cannot fail-back
> to the "software renderer".
> Next attrs were removed:
> NSOpenGLPFANoRecovery, -> if an accelerated renderer fails due to lack
> of resources, OpenGL automatically switches to another renderer.
> NSOpenGLPFAAccelerated, -> accelerated renderers are still preferred.
> NSOpenGLPFAFullScreen, -> we don't use this functionality.
>
> Also in this code we try to mix "CoreGraphics display id" and "Screen
> index". Currently we never use "screen index" so all related code was
> removed/renamed to be more obvious.
>
> Bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8008660
> Webrev can be found at:
> http://cr.openjdk.java.net/~serb/8008660/webrev.00
>