[OpenJDK 2D-Dev] RFR 8148886: SEGV in sun.java2d.marlin.Renderer._endRendering
Laurent Bourgès
bourges.laurent at gmail.com
Thu Feb 18 08:41:31 UTC 2016
Phil & Jim,
Here is an updated webrev including jim's changes to comments:
http://cr.openjdk.java.net/~lbourges/marlin/marlin-8148886.5/
Hope it is ready to push,
Regards,
Laurent
2016-02-12 9:46 GMT+01:00 Laurent Bourgès <bourges.laurent at gmail.com>:
> Jim & Phil,
>
> 2016-02-11 23:20 GMT+01:00 Jim Graham <james.graham at oracle.com>:
>
>> It all looks great. The comments are fine for internal documentation,
>> but here is a suggestion for the initial comment on a couple of the
>> Provider classes. I don't need to review the changes to comments.
>>
>
> Thanks Jim for your carefull reviews and your suggestions: I integrated
> your changes to javadoc with care.
>
> FYI I always use the following command to remove redundant white spaces:
> find . -type f -name '*.java' -exec sed --in-place 's/[[:space:]]\+$//' {}
> \;
>
> To summarize, I can count on jim's review but another one is missing.
>
> Phil, do you approve too ?
>
> Regards,
> Laurent
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/2d-dev/attachments/20160218/f304e0f9/attachment.html>
More information about the 2d-dev
mailing list