Re: [OpenJDK Rasterizer] Fwd: Re: Fwd: RFR: Marlin renderer #3
If you'll give me a few minutes, I'll synch the repo before this goes in... ...jim On 8/24/15 3:08 PM, Jim Graham wrote: Hi Laurent, I'm feeling much better this week (and can even start typing with my bad hand in short spurts now), so I'll hopefully be more on the ball now. Short version - looks good to go, consider the following input at your own discretion and go ahead and push. Reviewing s3.4: On 8/17/15 2:34 PM, Laurent Bourgès wrote: Changes: - Renderer: remove unused D_ERR_STEP_MAX and 0x1 replaced by 1 for readability Renderer, line 319 - it makes sense to me that the scale changes would be based off of the same metric. Basically the constraint is keeping the speed (dxy) between INC and DEC bounds. I'll admit, though, that I haven't been through the actual derivations of the math here, but I thought I'd give my 2 cents on your comment there. Renderer, line 479 - I often put extra parens in for cases like this because it isn't common knowledge whether cast or add is higher in precedence. I generally assume no better knowledge than the basic PEMDAS rules that we learn in grade school. But that's just my style (I also don't have the OOO table memorized, but maybe that's just my senility ;). 0x1 vs 1 - I usually use 0x1 when I'm using numbers for their bit patterns, so | and & typically use 0x constants in my style philosophy. In the case of "1" it's a degenerate case that doesn't seem to matter, but the added 0x tells me to think in terms of bits, not cardinal values. Renderer, note for future investigation - we store the direction flag in Y_MAX, but then we need to do manipulation to process Y_MAX and we also need more manipulation to transfer that bit to crossing values. What if instead we stored it in the lowest bit of curx? We'd then have 31.31 fixed point, we'd have to process slope so that its whole part was doubled (to skip affecting the LSbit of curx) and we'd have to do the carry using "((error >> 31) << 1)", but we'd get the transfer of the direction bit to the crossing for free and all in all, the total operations may be reduced (at the cost of 1 bit of X coordinate range). - MarlinRenderingEngine: optimized loop in pathTo() Looks good - Dasher: TODO remains related to dash length issue OK. Hope it is good now, I plan to work soon on the Array Cache & dispose code. Looks great! Consider the style issues I mentioned above if you wish and otherwise it is good to go. I don't need to see any further webrevs on those issues. One more suggestion for optimization below can be considered in subsequent work: For CHECK_NAN, how fast is Math.isNaN? Is it worth first comparing the intpart to the value that NaN converts to before calling isNaN? The majority case, by a wide margin, is that the value is not NaN. That's the aim: I test first the most probable case (a <= intpart) for ceil() and then check possible overflow and finally check for NaN values to avoid adding / substracting 1 ! I made benchmarks and this implementation optimized for the integer domain works very well: 2x faster. I guess I wasn't clear. I understand why the 3 parts of the test are in the order that they are in, but I was referring only to the "CHECK_NAN && Float.isNaN(a)" clause. Is that worth adding one more test for "CHECK_NAN && intpart == NAN_TO_INT_VAL && Float.isNaN(a)" in that one clause? For the ceil_int case, a very common case is that a is non-negative and non-integer. In that case, "a <= intpart" will fail and you will do the CHECK_NAN test before you can return "intpart+1". For the floor_int case, you would only get to the CHECK_NAN case on negative-non-integers which we don't expect a lot of, but in general, the added test might make it slightly faster. I was under the understanding that if a constant is declared final and static, then it is compiled directly into the code and there is no constant lookup. Check the bytecodes and you should see a hard-coded literal 0x7fff everywhere it is used... I agree but it seems slightly faster: probably loading the literal is a bit slower than loading from the stack ? Maybe some hotspot experts could give their opinion. As this is the renderer hot loop, I prefer making it as fast as possible. In the case of 0x7fff it may be because of the size of the constant. A smaller literal like "1" or "2" might be faster as an in-lined source literal. I think that at least for now they should either be sun.java2d.renderer.marlin.* or just sun.java2d.marlin.* Ok, I can rename them in the next webrev as it requires me to upgrade all my testing & benchmarking scripts... Sounds fine to me. This is more "something we need to finalize before integrating into the master workspace" than an issue for any given webrev. But the more important question may be around how much benefit these bring compared to the additional testing overhead of so many combin
Re: [OpenJDK Rasterizer] Fwd: Re: Fwd: RFR: Marlin renderer #3
Jim, 2015-08-25 22:39 GMT+02:00 Jim Graham : > If you'll give me a few minutes, I'll synch the repo before this goes in... > Ok, I was just typing 'hg push' ... PS: I will answer your email meanwhile... Laurent
Re: [OpenJDK Rasterizer] Fwd: Re: Fwd: RFR: Marlin renderer #3
Jim, 2015-08-25 0:08 GMT+02:00 Jim Graham : > Hi Laurent, > > I'm feeling much better this week (and can even start typing with my bad > hand in short spurts now), so I'll hopefully be more on the ball now. > Great ! Hope your hand is not too bad. > > Short version - looks good to go, consider the following input at your own > discretion and go ahead and push. > Excellent. Here are my comments: > > Reviewing s3.4: > > On 8/17/15 2:34 PM, Laurent Bourgès wrote: > >> Changes: >> - Renderer: remove unused D_ERR_STEP_MAX and 0x1 replaced by 1 for >> readability >> > > Renderer, line 319 - it makes sense to me that the scale changes would be > based off of the same metric. Basically the constraint is keeping the > speed (dxy) between INC and DEC bounds. I'll admit, though, that I haven't > been through the actual derivations of the math here, but I thought I'd > give my 2 cents on your comment there. > I left a comment as changing the metric will require adjusting the bounds + tests (postponed later). According to my understandings, the error between the linear segment and the curve is <= 8 x | acceleration(t) | (Graphics Gem I) so I would prefer using the ddx|y. Besides, I would use th squared norm = ddx*ddx + ddy*ddy instead of abs(ddx). I think the speed dx|y describes the curve slope so it does not correspond to the approximation error, but it seems you already use such constraints in another renderer (ProcessPath.c). Another point: filled curves (handled by Renderer directly) are not monotonic so this error approximation is then not valid ! Maybe we should split the curve as the Stroker does (at cups, inflexion points ...) > Renderer, line 479 - I often put extra parens in for cases like this > because it isn't common knowledge whether cast or add is higher in > precedence. I generally assume no better knowledge than the basic PEMDAS > rules that we learn in grade school. But that's just my style (I also don't > have the OOO table memorized, but maybe that's just my senility ;). > Done. > 0x1 vs 1 - I usually use 0x1 when I'm using numbers for their bit > patterns, so | and & typically use 0x constants in my style philosophy. In > the case of "1" it's a degenerate case that doesn't seem to matter, but the > added 0x tells me to think in terms of bits, not cardinal values. > Fixed > > Renderer, note for future investigation - we store the direction flag in > Y_MAX, but then we need to do manipulation to process Y_MAX and we also > need more manipulation to transfer that bit to crossing values. What if > instead we stored it in the lowest bit of curx? We'd then have 31.31 fixed > point, we'd have to process slope so that its whole part was doubled (to > skip affecting the LSbit of curx) and we'd have to do the carry using > "((error >> 31) << 1)", but we'd get the transfer of the direction bit to > the crossing for free and all in all, the total operations may be reduced > (at the cost of 1 bit of X coordinate range). > I seems a very good idea to evaluate soon: it will save several shift operations (+ 1 Unsafe access). I would also make error, bump_x, bump_error as 31.31 FP (LSB=0) so it would be consistent and may simplify even more computations. Hope it is good now, I plan to work soon on the Array Cache & dispose code. >> > > Looks great! Consider the style issues I mentioned above if you wish and > otherwise it is good to go. I don't need to see any further webrevs on > those issues. > Thanks. I started working on RLE / pixel coverage encoding but still expect to improve Array Cache next. > > One more suggestion for optimization below can be considered in subsequent > work: > > For CHECK_NAN, how fast is Math.isNaN? Is it worth first comparing >> the intpart to the value that NaN converts to before calling isNaN? >> The majority case, by a wide margin, is that the value is not NaN. >> >> >> That's the aim: I test first the most probable case (a <= intpart) for >> ceil() and then check possible overflow and finally check for NaN values >> to avoid adding / substracting 1 ! >> I made benchmarks and this implementation optimized for the integer >> domain works very well: 2x faster. >> > > I guess I wasn't clear. I understand why the 3 parts of the test are in > the order that they are in, but I was referring only to the "CHECK_NAN && > Float.isNaN(a)" clause. Is that worth adding one more test for "CHECK_NAN > && intpart == NAN_TO_INT_VAL && Float.isNaN(a)" in that one clause? > > For the ceil_int case, a very common case is that a is non-negative and > non-integer. In that case, "a <= intpart" will fail and you will do the > CHECK_NAN test before you can return "intpart+1". > > For the floor_int case, you would only get to the CHECK_NAN case on > negative-non-integers which we don't expect a lot of, but in general, the > added test might make it slightly faster. > I quickly tried but sorry it provides no gains: Float.isNaN() is just implemented as (a != a
[OpenJDK Rasterizer] hg: graphics-rasterizer/jdk9/corba: 4 new changesets
Changeset: 960b56805abd Author:katleman Date: 2015-07-23 11:54 -0700 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/corba/rev/960b56805abd Added tag jdk9-b74 for changeset 622fe934e351 ! .hgtags Changeset: d8126bc88fa5 Author:katleman Date: 2015-07-30 11:15 -0700 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/corba/rev/d8126bc88fa5 Added tag jdk9-b75 for changeset 960b56805abd ! .hgtags Changeset: 8bb2441c0fec Author:katleman Date: 2015-08-06 08:07 -0700 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/corba/rev/8bb2441c0fec Added tag jdk9-b76 for changeset d8126bc88fa5 ! .hgtags Changeset: 182bb7accc52 Author:katleman Date: 2015-08-13 12:20 -0700 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/corba/rev/182bb7accc52 Added tag jdk9-b77 for changeset 8bb2441c0fec ! .hgtags
[OpenJDK Rasterizer] hg: graphics-rasterizer/jdk9/jaxws: 4 new changesets
Changeset: 086bcd5e4a53 Author:katleman Date: 2015-07-23 11:54 -0700 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/jaxws/rev/086bcd5e4a53 Added tag jdk9-b74 for changeset 6232472e5141 ! .hgtags Changeset: 55bb88306dc5 Author:katleman Date: 2015-07-30 11:15 -0700 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/jaxws/rev/55bb88306dc5 Added tag jdk9-b75 for changeset 086bcd5e4a53 ! .hgtags Changeset: bd6ece68cf8a Author:katleman Date: 2015-08-06 08:07 -0700 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/jaxws/rev/bd6ece68cf8a Added tag jdk9-b76 for changeset 55bb88306dc5 ! .hgtags Changeset: ac1748bab074 Author:katleman Date: 2015-08-13 12:20 -0700 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/jaxws/rev/ac1748bab074 Added tag jdk9-b77 for changeset bd6ece68cf8a ! .hgtags
[OpenJDK Rasterizer] hg: graphics-rasterizer/jdk9/hotspot: 118 new changesets
Changeset: 36fd5d1982b0 Author:ascarpino Date: 2015-07-10 11:31 -0700 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/hotspot/rev/36fd5d1982b0 8130341: GHASH 32bit intrinsics has AEADBadTagException Reviewed-by: kvn, mcberg ! src/cpu/x86/vm/stubGenerator_x86_32.cpp ! test/compiler/codegen/7184394/TestAESBase.java ! test/compiler/codegen/7184394/TestAESDecode.java ! test/compiler/codegen/7184394/TestAESEncode.java Changeset: 94403236f303 Author:kvn Date: 2015-07-10 11:59 -0700 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/hotspot/rev/94403236f303 8129920: Vectorized loop unrolling Summary: optimize loop opts for vectorizible loops. Reviewed-by: kvn, roland ! src/share/vm/opto/loopTransform.cpp ! src/share/vm/opto/loopUnswitch.cpp ! src/share/vm/opto/loopnode.cpp ! src/share/vm/opto/loopnode.hpp ! src/share/vm/opto/superword.cpp ! src/share/vm/opto/superword.hpp Changeset: 91f45ea76992 Author:goetz Date: 2015-07-07 10:40 +0200 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/hotspot/rev/91f45ea76992 8130653: ppc: implement MultiplyToLen intrinsic Reviewed-by: simonis Contributed-by: peter.janusc...@sap.com ! src/cpu/ppc/vm/frame_ppc.inline.hpp ! src/cpu/ppc/vm/macroAssembler_ppc.cpp ! src/cpu/ppc/vm/macroAssembler_ppc.hpp ! src/cpu/ppc/vm/macroAssembler_ppc.inline.hpp ! src/cpu/ppc/vm/ppc.ad ! src/cpu/ppc/vm/stubGenerator_ppc.cpp ! src/cpu/ppc/vm/vm_version_ppc.cpp Changeset: 3e15bdb908cb Author:ascarpino Date: 2015-07-13 13:22 -0700 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/hotspot/rev/3e15bdb908cb 8131078: typos in ghash cpu message Reviewed-by: goetz, kvn ! src/cpu/sparc/vm/vm_version_sparc.cpp Changeset: 0fb7705845de Author:mhaupt Date: 2015-03-31 21:46 +0200 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/hotspot/rev/0fb7705845de 6900757: minor bug fixes to LogCompilation tool Summary: improve internal error reporting (point to XML element causing trouble); fix comparator for sorting by name and start; make tool more robust wrt. incorrect options and files not found; make inlining decision output more clear; adopt uncommon traps history printing; properly mention compiler in generated logs; add options for printing time stamps and omitting compilation IDs; add option for comparing compilation logs; overall code cleanup and API documentation Reviewed-by: kvn, vlivanov ! .hgignore ! src/share/tools/LogCompilation/Makefile ! src/share/tools/LogCompilation/src/com/sun/hotspot/tools/compiler/BasicLogEvent.java ! src/share/tools/LogCompilation/src/com/sun/hotspot/tools/compiler/CallSite.java ! src/share/tools/LogCompilation/src/com/sun/hotspot/tools/compiler/Compilation.java ! src/share/tools/LogCompilation/src/com/sun/hotspot/tools/compiler/LogCleanupReader.java ! src/share/tools/LogCompilation/src/com/sun/hotspot/tools/compiler/LogCompilation.java ! src/share/tools/LogCompilation/src/com/sun/hotspot/tools/compiler/LogEvent.java ! src/share/tools/LogCompilation/src/com/sun/hotspot/tools/compiler/LogParser.java ! src/share/tools/LogCompilation/src/com/sun/hotspot/tools/compiler/MakeNotEntrantEvent.java ! src/share/tools/LogCompilation/src/com/sun/hotspot/tools/compiler/Method.java ! src/share/tools/LogCompilation/src/com/sun/hotspot/tools/compiler/NMethod.java ! src/share/tools/LogCompilation/src/com/sun/hotspot/tools/compiler/Phase.java + src/share/tools/LogCompilation/src/com/sun/hotspot/tools/compiler/UncommonTrap.java ! src/share/tools/LogCompilation/src/com/sun/hotspot/tools/compiler/UncommonTrapEvent.java ! src/share/vm/compiler/compileBroker.cpp Changeset: 2963c44aa1f5 Author:hseigel Date: 2015-07-09 08:36 -0400 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/hotspot/rev/2963c44aa1f5 8130183: InnerClasses: VM permits wrong inner_class_info_index value of zero Summary: Throw ClassFormatError if InnerClasses attribute's inner_class_info_index is 0 Reviewed-by: acorn, lfoltan ! src/share/vm/classfile/classFileParser.cpp + test/runtime/classFileParserBug/EnclosingMethod.java + test/runtime/classFileParserBug/badEnclMthd.jcod Changeset: 07f48b118941 Author:hseigel Date: 2015-07-09 15:39 -0400 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/hotspot/rev/07f48b118941 8130669: VM prohibits methods with return values Summary: Ignore methods with return values instead of throwing ClassFormatError exceptions Reviewed-by: acorn, iklam ! src/share/vm/classfile/classFileParser.cpp ! src/share/vm/classfile/verifier.cpp + test/runtime/classFileParserBug/BadInitMethod.java + test/runtime/classFileParserBug/badInit.jasm + test/runtime/classFileParserBug/ignoredClinit.jasm Changeset: a87c296434eb Author:jbachorik Date: 2015-07-10 16:37 +0200 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/hotspot/rev/a87c296434eb Merge ! src/share/vm/classfile/classFileParser.cpp Changeset: b
[OpenJDK Rasterizer] hg: graphics-rasterizer/jdk9: 22 new changesets
Changeset: fa8b91ef00bf Author:weijun Date: 2015-07-20 20:45 +0800 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/rev/fa8b91ef00bf 8131350: policytool can directly reference permission classes Reviewed-by: xuelei ! modules.xml Changeset: bb77b3156c6e Author:ykantser Date: 2015-07-09 12:56 +0200 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/rev/bb77b3156c6e 8032763: Remove use of sun.misc.Ref from hprof parser in testlibrary Reviewed-by: jbachorik, alanb ! test/lib/share/classes/jdk/test/lib/hprof/model/Snapshot.java Changeset: b82c27650a7e Author:jbachorik Date: 2015-07-10 16:37 +0200 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/rev/b82c27650a7e Merge Changeset: 39b1000561b9 Author:dcubed Date: 2015-07-14 09:36 -0700 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/rev/39b1000561b9 8131128: Merge error in jprt.properties leads to missing devkit argument Summary: Add missing line break; fix backslash lineup. Reviewed-by: tbell, kvn ! make/jprt.properties Changeset: 8f72b3d8805e Author:amurillo Date: 2015-07-17 08:46 -0700 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/rev/8f72b3d8805e Merge Changeset: 571788f53574 Author:amurillo Date: 2015-07-21 09:19 -0700 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/rev/571788f53574 Merge Changeset: 887a2657adef Author:katleman Date: 2015-07-23 11:54 -0700 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/rev/887a2657adef Added tag jdk9-b74 for changeset 57f3134853ec ! .hgtags Changeset: 8fd6eeb87860 Author:lana Date: 2015-07-23 15:27 -0700 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/rev/8fd6eeb87860 Merge Changeset: b9aba99deb2d Author:jlahoda Date: 2015-07-24 08:37 +0200 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/rev/b9aba99deb2d 8086737: Add support for -release to Javadoc Summary: Exporting com.sun.tools.javac.platform to jdk.javadoc. Reviewed-by: jjg, ksrini, alanb ! modules.xml Changeset: 2de723be58cf Author:katleman Date: 2015-07-30 11:15 -0700 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/rev/2de723be58cf Added tag jdk9-b75 for changeset 8fd6eeb87860 ! .hgtags Changeset: d82072b699b8 Author:lana Date: 2015-07-30 15:21 -0700 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/rev/d82072b699b8 Merge Changeset: a74d1574da34 Author:naoto Date: 2015-08-03 21:49 -0700 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/rev/a74d1574da34 8129881: JDK-8008577 breaks Nashorn test 8130845: Change to CLDR Locale data in JDK 9 b71 causes SimpleDateFormat parsing errors 8132125: German (Switzerland) formatting broken if CLDR Locale Data is used Reviewed-by: tbell, okutsu ! make/Main.gmk Changeset: 2671addb3319 Author:zmajo Date: 2015-07-28 19:20 +0200 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/rev/2671addb3319 8130832: Extend the WhiteBox API to provide information about the availability of compiler intrinsics Summary: Add a new method, sun.hotspot.WhiteBox.isIntrinsicAvailable, that can be used to determine if an intrinsic is available. Reviewed-by: kvn, jrose ! test/lib/sun/hotspot/WhiteBox.java Changeset: f47ccd58ac92 Author:amurillo Date: 2015-07-31 10:15 -0700 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/rev/f47ccd58ac92 Merge Changeset: cf76386db5c7 Author:amurillo Date: 2015-08-04 10:59 -0700 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/rev/cf76386db5c7 Merge Changeset: 3bf4d869d8f0 Author:katleman Date: 2015-08-06 08:07 -0700 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/rev/3bf4d869d8f0 Added tag jdk9-b76 for changeset d82072b699b8 ! .hgtags Changeset: 7972dc8f2a47 Author:lana Date: 2015-08-06 11:17 -0700 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/rev/7972dc8f2a47 Merge Changeset: 4c85a31c02e8 Author:jlahoda Date: 2015-08-10 09:47 +0200 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/rev/4c85a31c02e8 8129562: JDK 9 build using boot-jdk classes instead of newly compiled classes Summary: Need to specify empty -extdirs and -endorseddirs to javac to avoid loading of boot JDK classes during build. Reviewed-by: tbell, coffeys, jjg, henryjen ! make/CompileJavaModules.gmk Changeset: 59d19d081bbc Author:sundar Date: 2015-08-13 19:09 +0530 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/rev/59d19d081bbc 8133347: Add makefiles support and basic session, persistence history navigation with jline Reviewed-by: erikj, jlahoda, jlaskey ! make/CompileJavaModules.gmk ! make/Images.gmk ! modules.xml Changeset: 675991eb9dd2 Author:katleman Date: 2015-08-13 12:20 -0700 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/rev/675991eb9
[OpenJDK Rasterizer] hg: graphics-rasterizer/jdk9/nashorn: 19 new changesets
Changeset: 348ce347ba14 Author:hannesw Date: 2015-07-20 13:11 +0200 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/nashorn/rev/348ce347ba14 8131340: Varargs function is recompiled each time it is linked Reviewed-by: mhaupt, sundar ! src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/runtime/FinalScriptFunctionData.java ! src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/runtime/RecompilableScriptFunctionData.java ! src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/runtime/ScriptFunctionData.java + test/script/basic/JDK-8131340.js Changeset: b983e998f528 Author:hannesw Date: 2015-07-22 10:18 +0200 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/nashorn/rev/b983e998f528 8131683: Delete fails over multiple scopes Reviewed-by: mhaupt, sundar ! src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/codegen/AssignSymbols.java ! src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/ir/RuntimeNode.java ! src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/runtime/ScriptRuntime.java + test/script/basic/JDK-8131683.js + test/script/basic/JDK-8131683.js.EXPECTED Changeset: b27730a502c3 Author:mhaupt Date: 2015-07-22 09:28 +0200 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/nashorn/rev/b27730a502c3 8131142: late-bind check for testng.jar presence in Nashorn test execution Reviewed-by: hannesw, sundar ! make/build.xml Changeset: 4193f8c6706a Author:katleman Date: 2015-07-23 11:54 -0700 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/nashorn/rev/4193f8c6706a Added tag jdk9-b74 for changeset 2e8bb16872d7 ! .hgtags Changeset: f884dff432a7 Author:lana Date: 2015-07-23 15:28 -0700 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/nashorn/rev/f884dff432a7 Merge Changeset: 9fddd7695ded Author:mhaupt Date: 2015-07-27 09:42 +0200 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/nashorn/rev/9fddd7695ded 8132305: fix incorrect title assignment in Nashorn JavaFX samples Reviewed-by: attila, sundar ! samples/browser_dom.js ! samples/showenv.js ! samples/showsysprops.js - samples/time_color.fx + samples/time_color.js Changeset: 833a4df84bc7 Author:sundar Date: 2015-07-28 14:52 +0530 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/nashorn/rev/833a4df84bc7 8132092: Nashorn copyright has to be updated Reviewed-by: jlaskey, hannesw, mhaupt ! src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/runtime/DebuggerSupport.java ! test/script/basic/JDK-8007456.js ! test/script/basic/JDK-8035712.js ! test/script/basic/JDK-8051778.js ! test/script/basic/JDK-8058610.js ! test/script/basic/JDK-8061113.js ! test/script/basic/JDK-8062799.js ! test/script/basic/JDK-8066221.js ! test/script/basic/JDK-8066222.js ! test/script/basic/JDK-8066224.js ! test/script/basic/JDK-8066225.js ! test/script/basic/JDK-8066227.js ! test/script/basic/JDK-8066230.js ! test/script/basic/JDK-8066232.js ! test/script/basic/JDK-8066236.js ! test/script/basic/JDK-8067139.js ! test/script/basic/JDK-8067774.js ! test/script/basic/JDK-8068573.js ! test/script/basic/JDK-8068580.js ! test/script/basic/JDK-8068985.js ! test/script/basic/JDK-8069002.js ! test/script/basic/JDK-8072426.js ! test/script/basic/JDK-8072596.js ! test/script/basic/JDK-8075090.js ! test/script/basic/JDK-8079145.js ! test/script/basic/JDK-8079269.js ! test/script/basic/JDK-8079424.js ! test/script/basic/JDK-8079470.js ! test/script/basic/JDK-8080182.js ! test/script/basic/JDK-8080848.js ! test/script/basic/JDK-8081156.js ! test/script/basic/JDK-8085802.js ! test/script/basic/JDK-8087211.js ! test/script/basic/JDK-8087211_2.js ! test/script/basic/JDK-8098578.js ! test/script/basic/JDK-8129410.js ! test/script/currently-failing/gettersetter.js ! test/script/currently-failing/property_delete.js ! test/script/maptests/builtins.js ! test/script/maptests/constructor.js ! test/script/maptests/maputil.js ! test/script/maptests/object_create.js ! test/script/maptests/object_literals.js ! test/script/maptests/point.js ! test/script/maptests/property_add.js ! test/script/maptests/proto.js ! test/script/sandbox/safeprops.js ! test/src/jdk/nashorn/test/models/NullProvider.java Changeset: 0bfcbf0054f1 Author:katleman Date: 2015-07-30 11:15 -0700 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/nashorn/rev/0bfcbf0054f1 Added tag jdk9-b75 for changeset f884dff432a7 ! .hgtags Changeset: ab231613d720 Author:lana Date: 2015-07-30 15:22 -0700 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/nashorn/rev/ab231613d720 Merge - samples/time_color.fx Changeset: ed56500172f4 Author:sundar Date: 2015-08-04 18:18 +0530 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/nashorn/rev/ed56500172f4 8073733: TypeError messages with "call" and "new" could be improved Reviewed-by: attila, mhaupt ! src/jdk
[OpenJDK Rasterizer] hg: graphics-rasterizer/jdk9/langtools: 19 new changesets
Changeset: 4b7f5ea468d1 Author:katleman Date: 2015-07-23 11:54 -0700 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/langtools/rev/4b7f5ea468d1 Added tag jdk9-b74 for changeset 02681b7c4232 ! .hgtags Changeset: 827915d1e55e Author:lana Date: 2015-07-23 15:27 -0700 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/langtools/rev/827915d1e55e Merge Changeset: deb1cda4dc79 Author:jlahoda Date: 2015-07-24 13:08 +0200 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/langtools/rev/deb1cda4dc79 8086737: Add support for -release to Javadoc Reviewed-by: jjg, ksrini ! src/jdk.compiler/share/classes/com/sun/tools/javac/main/Arguments.java + src/jdk.compiler/share/classes/com/sun/tools/javac/platform/PlatformUtils.java ! src/jdk.javadoc/share/classes/com/sun/tools/javadoc/Start.java ! src/jdk.javadoc/share/classes/com/sun/tools/javadoc/ToolOption.java ! src/jdk.javadoc/share/classes/com/sun/tools/javadoc/resources/javadoc.properties + test/tools/javadoc/ReleaseOption.java + test/tools/javadoc/ReleaseOptionSource.java Changeset: dd96ac308ab8 Author:vromero Date: 2015-07-24 15:36 -0700 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/langtools/rev/dd96ac308ab8 8132215: class InferenceContext should live in a separate file Reviewed-by: mcimadamore, jlahoda ! src/jdk.compiler/share/classes/com/sun/tools/javac/comp/Attr.java ! src/jdk.compiler/share/classes/com/sun/tools/javac/comp/Check.java ! src/jdk.compiler/share/classes/com/sun/tools/javac/comp/DeferredAttr.java ! src/jdk.compiler/share/classes/com/sun/tools/javac/comp/Infer.java + src/jdk.compiler/share/classes/com/sun/tools/javac/comp/InferenceContext.java ! src/jdk.compiler/share/classes/com/sun/tools/javac/comp/Resolve.java Changeset: e0a4a04160cb Author:jlahoda Date: 2015-07-28 17:01 +0200 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/langtools/rev/e0a4a04160cb 8130826: test writes file in test source directory Summary: Setting an explicit output directory for ToolBox.JavacTask in PlatformProviderTest. Reviewed-by: jjg ! test/tools/javac/platform/PlatformProviderTest.java Changeset: 3c1da6c1ef9d Author:mcimadamore Date: 2015-07-30 13:20 +0100 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/langtools/rev/3c1da6c1ef9d 8081769: Redundant error message on bad usage of 'class' literal Summary: javac should skip to next token when an erroneous ident is found in term3rest Reviewed-by: jlahoda ! src/jdk.compiler/share/classes/com/sun/tools/javac/parser/JavacParser.java + test/tools/javac/parser/8081769/T8081769.java + test/tools/javac/parser/8081769/T8081769.out Changeset: 2289e78ae8b2 Author:mcimadamore Date: 2015-07-30 13:21 +0100 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/langtools/rev/2289e78ae8b2 8129214: Access error when unboxing a primitive whose target is a type-variable in a different package Summary: Missing erasure when unboxing type in Lower Reviewed-by: jlahoda ! src/jdk.compiler/share/classes/com/sun/tools/javac/comp/Lower.java + test/tools/javac/generics/typevars/8129214/T8129214.java + test/tools/javac/generics/typevars/8129214/pkg/Foo.java Changeset: 577e9ffab3bc Author:mcimadamore Date: 2015-07-30 13:24 +0100 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/langtools/rev/577e9ffab3bc 8131742: Syntactically meaningless code accepted by javac Summary: Receiver parameter logic for type annotations should be disabled when parsing lambda formals Reviewed-by: jlahoda ! src/jdk.compiler/share/classes/com/sun/tools/javac/parser/JavacParser.java + test/tools/javac/lambda/8131742/T8131742.java + test/tools/javac/lambda/8131742/T8131742.out Changeset: ba3a15fc0032 Author:katleman Date: 2015-07-30 11:15 -0700 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/langtools/rev/ba3a15fc0032 Added tag jdk9-b75 for changeset 827915d1e55e ! .hgtags Changeset: 3eefba079679 Author:lana Date: 2015-07-30 15:22 -0700 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/langtools/rev/3eefba079679 Merge Changeset: 80ab77fb Author:igerasim Date: 2015-07-31 01:36 +0300 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/langtools/rev/80ab77fb 8062647: Wrong indentation of arguments of annotated methods Reviewed-by: jjg, bpatel ! src/jdk.javadoc/share/classes/com/sun/tools/doclets/formats/html/ConstructorWriterImpl.java ! src/jdk.javadoc/share/classes/com/sun/tools/doclets/formats/html/HtmlDocletWriter.java ! src/jdk.javadoc/share/classes/com/sun/tools/doclets/formats/html/MethodWriterImpl.java ! test/com/sun/javadoc/testIndentation/TestIndentation.java + test/com/sun/javadoc/testIndentation/p/IndentAnnot.java Changeset: 7eef740c1482 Author:jlahoda Date: 2015-08-03 13:28 +0200 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/langtools/rev/7eef740c1482
Re: [OpenJDK Rasterizer] Fwd: Re: Fwd: RFR: Marlin renderer #3
I quickly tried but sorry it provides no gains: Float.isNaN() is just
implemented as (a != a) so it is faster with only 1 condition (a != a)
than 2 (intval == 0 && (a != a)).
Another way to make NaN tests fall out naturally from your code is to
make sure that they are always in the "numeric test failed" branch.
Currently you have them in the "numeric test succeeded branch. What if
you inverted the test? As in:
if (a > intpart
&& CHECK_OVERFLOW && !overflow)
{
return intpart +/- 1;
}
return intpart;
In this case NaN causes the first test to fail and you jump straight to
the "return intpart" code without having to explicitly test for it...?
Basically we tend to have tests of the form "if (number is bad) return
failure" and NaN isn't caught by that. If we instead use "if (number is
good) { do things }" then NaN values will fail to make it into the code
block. Usually when these are at the top of a function we'd rather test
for bad values and return instead to avoid extra indenting, but then an
alternate way is to use:
if (number is good) {
[*]
} else {
return failure;
}
[*] no code there, we only really needed the else, or:
if (!(number is good)) {
return failure;
}
But, if the code block is small and has no further indentation needs,
then generally the following is naturally NaN-resistant:
if (numbers are good) {
// do all the calcs...
}
...jim
[OpenJDK Rasterizer] hg: graphics-rasterizer/jdk9/jaxp: 14 new changesets
Changeset: bb67bf0ada7b Author:joehw Date: 2015-07-22 10:55 -0700 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/jaxp/rev/bb67bf0ada7b 8131907: Numerous threads lock during XML processing while running Weblogic 12.1.3 Reviewed-by: rriggs, dfuchs, lancea ! src/java.xml/share/classes/com/sun/org/apache/xerces/internal/impl/dv/DTDDVFactory.java Changeset: 61df977c3d7a Author:katleman Date: 2015-07-23 11:54 -0700 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/jaxp/rev/61df977c3d7a Added tag jdk9-b74 for changeset eadcb2b55cd1 ! .hgtags Changeset: 16b5e696f948 Author:lana Date: 2015-07-23 15:27 -0700 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/jaxp/rev/16b5e696f948 Merge Changeset: dcdbd67e6408 Author:dfuchs Date: 2015-07-28 11:30 +0200 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/jaxp/rev/dcdbd67e6408 8132256: jaxp: Investigate removal of com/sun/org/apache/bcel/internal/util/ClassPath.java Summary: com/sun/org/apache/bcel/internal/util/ClassPath.java removed Reviewed-by: joehw ! src/java.xml/share/classes/com/sun/org/apache/bcel/internal/Repository.java - src/java.xml/share/classes/com/sun/org/apache/bcel/internal/util/ClassPath.java ! src/java.xml/share/classes/com/sun/org/apache/bcel/internal/util/SyntheticRepository.java Changeset: 520d2f4b2b46 Author:dfuchs Date: 2015-07-29 11:00 +0200 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/jaxp/rev/520d2f4b2b46 8130059: jaxp: Investigate removal of com/sun/org/apache/xalan/internal/xslt/EnvironmentCheck.java Summary: remove com/sun/org/apache/xalan/internal/xslt/EnvironmentCheck.java Reviewed-by: joehw - src/java.xml/share/classes/com/sun/org/apache/xalan/internal/xslt/EnvironmentCheck.java - src/java.xml/share/classes/com/sun/org/apache/xml/internal/utils/Hashtree2Node.java ! test/javax/xml/jaxp/internaltest/javax/xml/common/bug6979306/Bug6979306Test.java Changeset: 60c6f80a9d1c Author:katleman Date: 2015-07-30 11:15 -0700 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/jaxp/rev/60c6f80a9d1c Added tag jdk9-b75 for changeset 16b5e696f948 ! .hgtags Changeset: 36801a89a042 Author:lana Date: 2015-07-30 15:22 -0700 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/jaxp/rev/36801a89a042 Merge - src/java.xml/share/classes/com/sun/org/apache/bcel/internal/util/ClassPath.java - src/java.xml/share/classes/com/sun/org/apache/xalan/internal/xslt/EnvironmentCheck.java - src/java.xml/share/classes/com/sun/org/apache/xml/internal/utils/Hashtree2Node.java Changeset: 13824e252a5f Author:dfuchs Date: 2015-07-31 12:05 +0200 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/jaxp/rev/13824e252a5f 8130058: jaxp: Investigate removal of com/sun/org/apache/xalan/internal/xslt/Process.java Summary: com/sun/org/apache/xalan/internal/xslt/Process.java removed. A copy modified to not depend on internal APIs is put in test/javax/xml/jaxp/internaltest/javax/xml/transform/cli/ProcessXSLT.java. The CLITest is preserved and uses that new copy. Reviewed-by: joehw, lancea - src/java.xml/share/classes/com/sun/org/apache/xalan/internal/Version.java - src/java.xml/share/classes/com/sun/org/apache/xalan/internal/xslt/Process.java - src/java.xml/share/classes/com/sun/org/apache/xerces/internal/impl/Version.java - test/javax/xml/jaxp/internaltest/javax/xml/common/bug6979306/Bug6979306Test.java ! test/javax/xml/jaxp/internaltest/javax/xml/transform/cli/CLITest.java + test/javax/xml/jaxp/internaltest/javax/xml/transform/cli/ProcessXSLT.java Changeset: 45a3bfaac32d Author:katleman Date: 2015-08-06 08:07 -0700 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/jaxp/rev/45a3bfaac32d Added tag jdk9-b76 for changeset 36801a89a042 ! .hgtags Changeset: be357705874c Author:lana Date: 2015-08-06 11:17 -0700 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/jaxp/rev/be357705874c Merge - src/java.xml/share/classes/com/sun/org/apache/xalan/internal/Version.java - src/java.xml/share/classes/com/sun/org/apache/xalan/internal/xslt/Process.java - src/java.xml/share/classes/com/sun/org/apache/xerces/internal/impl/Version.java - test/javax/xml/jaxp/internaltest/javax/xml/common/bug6979306/Bug6979306Test.java Changeset: 2b61bfcaa586 Author:joehw Date: 2015-08-10 09:52 -0700 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/jaxp/rev/2b61bfcaa586 8132660: Change jaxp unit test package name to be different with jaxp api Reviewed-by: joehw Contributed-by: frank.y...@oracle.com - test/javax/xml/jaxp/unittest/TEST.properties + test/javax/xml/jaxp/unittest/common/Bug6350682.java + test/javax/xml/jaxp/unittest/common/Bug6723276Test.java + test/javax/xml/jaxp/unittest/common/Bug6941169.xml + test/javax/xml/jaxp/unittest/common/Bug6941169.xsd + test/javax/xml/jaxp/unittest/common/Bug6941169Test.java + test/javax/x
[OpenJDK Rasterizer] hg: graphics-rasterizer/jdk9/jdk: 179 new changesets
Changeset: a55d8ffae378 Author:serb Date: 2015-07-28 18:14 +0300 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/jdk/rev/a55d8ffae378 8132355: Incorrect guard block in HPkeysym.h, awt_Event.h Reviewed-by: ant, azvegint ! src/java.desktop/unix/native/libawt_xawt/awt/HPkeysym.h ! src/java.desktop/unix/native/libawt_xawt/awt/awt_Event.h Changeset: 22054d298dff Author:ssadetsky Date: 2015-07-28 20:39 +0300 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/jdk/rev/22054d298dff 8130769: The new menu can't be shown on the menubar after clicking the "Add" button. Reviewed-by: alexsch, serb, azvegint ! src/java.desktop/share/classes/java/awt/MenuBar.java Changeset: 31015c9559b2 Author:ssadetsky Date: 2015-07-28 20:55 +0300 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/jdk/rev/31015c9559b2 8025815: Child FileDialog of modal dialog does not get focus on Gnome Reviewed-by: azvegint, serb ! src/java.desktop/unix/classes/sun/awt/X11/GtkFileDialogPeer.java ! src/java.desktop/unix/classes/sun/awt/X11/XFramePeer.java ! src/java.desktop/unix/classes/sun/awt/X11/XNETProtocol.java ! src/java.desktop/unix/native/libawt_xawt/awt/gtk2_interface.c ! src/java.desktop/unix/native/libawt_xawt/awt/gtk2_interface.h ! src/java.desktop/unix/native/libawt_xawt/awt/sun_awt_X11_GtkFileDialogPeer.c + test/java/awt/FileDialog/ModalFocus/FileDialogModalFocusTest.java Changeset: 5eca1e6a1236 Author:ssadetsky Date: 2015-07-28 20:59 +0300 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/jdk/rev/5eca1e6a1236 8130735: javax.swing.TimerQueue: timer fires late when another timer starts Reviewed-by: alexsch, serb, azvegint ! src/java.desktop/share/classes/javax/swing/TimerQueue.java Changeset: a64dcad3a7eb Author:serb Date: 2015-07-28 22:31 +0300 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/jdk/rev/a64dcad3a7eb 8013586: audioInputStream.close() does not release the resource 8130305: AudioSystem behavior depends on order that providers are located Reviewed-by: prr, amenkov ! src/java.desktop/share/classes/com/sun/media/sound/AiffFileReader.java ! src/java.desktop/share/classes/com/sun/media/sound/AuFileReader.java ! src/java.desktop/share/classes/com/sun/media/sound/SunFileReader.java ! src/java.desktop/share/classes/com/sun/media/sound/WaveExtensibleFileReader.java ! src/java.desktop/share/classes/com/sun/media/sound/WaveFileReader.java ! src/java.desktop/share/classes/com/sun/media/sound/WaveFloatFileReader.java + test/javax/sound/sampled/FileReader/AudioFileClose.java ! test/javax/sound/sampled/FileReader/ReadersExceptions.java Changeset: 2919a03653a8 Author:mhaupt Date: 2015-07-17 08:10 +0200 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/jdk/rev/2919a03653a8 8062543: Replace uses of MethodHandleImpl.castReference with Class.cast Reviewed-by: psandoz, vlivanov ! src/java.base/share/classes/java/lang/invoke/InvokerBytecodeGenerator.java ! src/java.base/share/classes/java/lang/invoke/MethodHandleImpl.java Changeset: 0dfae816d762 Author:weijun Date: 2015-07-20 20:47 +0800 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/jdk/rev/0dfae816d762 8131350: policytool can directly reference permission classes Reviewed-by: xuelei, mullan ! src/jdk.policytool/share/classes/sun/security/tools/policytool/PolicyTool.java Changeset: 01a718edc597 Author:mullan Date: 2015-07-20 09:03 -0400 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/jdk/rev/01a718edc597 8131486: SecureClassLoader key for ProtectionDomain cache also needs to take into account certificates Reviewed-by: weijun ! src/java.base/share/classes/java/security/CodeSource.java ! src/java.base/share/classes/java/security/SecureClassLoader.java ! test/java/security/SecureClassLoader/DefineClass.java ! test/java/security/SecureClassLoader/DefineClass.policy Changeset: 542cbf514c88 Author:mullan Date: 2015-07-20 09:03 -0400 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/jdk/rev/542cbf514c88 Merge Changeset: f5a02595c4c7 Author:darcy Date: 2015-07-20 13:11 -0700 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/jdk/rev/f5a02595c4c7 8129904: Add beans tests to tier 3 Reviewed-by: alanb, serb ! test/TEST.groups Changeset: f6699d8032ff Author:darcy Date: 2015-07-20 15:13 -0700 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/jdk/rev/f6699d8032ff 8081734: ConcurrentHashMap/ConcurrentAssociateTest.java, times out 90% of time on sparc with 256 cpu. Reviewed-by: chegar ! test/java/util/concurrent/ConcurrentHashMap/ConcurrentAssociateTest.java Changeset: 9eceb4a5ea96 Author:tyan Date: 2015-07-21 14:15 -0400 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/jdk/rev/9eceb4a5ea96 8068761: Test java/nio/channels/ServerSocketChannel/AdaptServerSocket.java failed with SocketTimeoutException Rev
Re: [OpenJDK Rasterizer] Fwd: Re: Fwd: RFR: Marlin renderer #3
All synchronized now... ...jim On 8/25/15 2:05 PM, Laurent Bourgès wrote: Jim, 2015-08-25 22:39 GMT+02:00 Jim Graham mailto:james.gra...@oracle.com>>: If you'll give me a few minutes, I'll synch the repo before this goes in... Ok, I was just typing 'hg push' ... PS: I will answer your email meanwhile... Laurent
[OpenJDK Rasterizer] hg: graphics-rasterizer/jdk9/jdk: Marlin renderer #3
Changeset: fec286225671 Author:lbourges Date: 2015-08-26 00:27 +0200 URL: http://hg.openjdk.java.net/graphics-rasterizer/jdk9/jdk/rev/fec286225671 Marlin renderer #3 Summary: Third marlin patch Reviewed-by: flar Contributed-by: bourges.laur...@gmail.com ! src/java.desktop/share/classes/sun/java2d/marlin/ArrayCache.java ! src/java.desktop/share/classes/sun/java2d/marlin/ByteArrayCache.java ! src/java.desktop/share/classes/sun/java2d/marlin/Curve.java ! src/java.desktop/share/classes/sun/java2d/marlin/Dasher.java ! src/java.desktop/share/classes/sun/java2d/marlin/FloatArrayCache.java ! src/java.desktop/share/classes/sun/java2d/marlin/FloatMath.java ! src/java.desktop/share/classes/sun/java2d/marlin/Helpers.java ! src/java.desktop/share/classes/sun/java2d/marlin/IntArrayCache.java ! src/java.desktop/share/classes/sun/java2d/marlin/MarlinCache.java ! src/java.desktop/share/classes/sun/java2d/marlin/MarlinConst.java + src/java.desktop/share/classes/sun/java2d/marlin/MarlinProperties.java ! src/java.desktop/share/classes/sun/java2d/marlin/MarlinRenderingEngine.java ! src/java.desktop/share/classes/sun/java2d/marlin/MarlinTileGenerator.java ! src/java.desktop/share/classes/sun/java2d/marlin/MarlinUtils.java ! src/java.desktop/share/classes/sun/java2d/marlin/MergeSort.java ! src/java.desktop/share/classes/sun/java2d/marlin/Renderer.java ! src/java.desktop/share/classes/sun/java2d/marlin/RendererContext.java ! src/java.desktop/share/classes/sun/java2d/marlin/RendererStats.java ! src/java.desktop/share/classes/sun/java2d/marlin/Stroker.java ! src/java.desktop/share/classes/sun/java2d/marlin/TransformingPathConsumer2D.java ! src/java.desktop/share/classes/sun/java2d/marlin/Version.java ! src/java.desktop/share/classes/sun/java2d/marlin/stats/Histogram.java + test/sun/java2d/marlin/CeilAndFloorTests.java
Re: [OpenJDK Rasterizer] Fwd: Re: Fwd: RFR: Marlin renderer #3
Jim, I pushed the patch #3 after sync + build + test again. Thanks, Laurent
