Hi Jim, Here is the new webrev: http://cr.openjdk.java.net/~lbourges/marlin/marlin.6/
> Just some minor stuff this time... Great news ! > MarlinRenderingEngine.java: > > line 378 - my eye just caught this. "between the stroker and Dasher"? It looks like this is going between the stroking engine and the Renderer, no? Right, I fixed that comment. > MarlinUtils.java: > > the getCallerInfo stuff may eventually need to be in a doPrivileged context, but we don't need to deal with that now. No need like other jdk classes I looked at. > Renderer.java: > > lines 46-50 - something looks off on the indentation there Fixed. > line 53 - I believe that getUnsafe() would need a doPrivileged as well No need neither. > lines 1385-1400 - lots of indentation issues Done. I spent a lot of my spare time up to now and I expect it is now time to let marlin go into the graphics repository ... Cheers, Laurent
