[prev in list] [next in list] [prev in thread] [next in thread]
List: openjdk-2d-dev
Subject: Re: [OpenJDK 2D-Dev] [10] RFR 8078192: Path2D storage trimming
From: Philip Race <philip.race () oracle ! com>
Date: 2017-06-07 20:56:38
Message-ID: 59386886.8050309 () oracle ! com
[Download RAW message or body]
Oh this one is interesting. It already has a CSR :-)
That's because we had created a CCC for it back in JDK 9 and then
withdrew it.
All existing CSRs were ported over but as "confidential" since honestly
no one had time to look at all N thousand of them and vet them ...
we vetted and opened enough to give people a flavour.
So I think that "create a csr" doesn't appear if there already is one
since you can't see it.
I opened it so you can now see it.
https://bugs.openjdk.java.net/browse/CCC-8078192
I re-opened it but don't seem to be able to target it to 10 or assign it
to you.
I'll need to ask.
-phil
On 6/7/17, 1:35 PM, Laurent Bourgès wrote:
> Phil,
>
> Could you initiate the CSR request in JBS ?
> I do not see how to create it from the current bug: I looked at the
> More menu but I do not have any Create CSR action.
>
> Cheers,
> Laurent
>
> Le 17 mai 2017 22:05, "Phil Race" <philip.race@oracle.com
> <mailto:philip.race@oracle.com>> a écrit :
>
> Early next week is the hope.
>
> -phil
>
> On 05/16/2017 02:20 PM, Laurent Bourgès wrote:
> > Phil,
> >
> > Did you get any answer from the CSR process on this bug ?
> >
> > Laurent
> >
> >
> > 2017-04-21 0:17 GMT+02:00 Philip Race <philip.race@oracle.com
> > <mailto:philip.race@oracle.com>>:
> >
> > OK. Although we still need to wait for the CSR process.
> >
> > -phil.
> >
> >
> > On 4/20/17, 3:05 PM, Laurent Bourgès wrote:
> > > Sorry (bad shortcut);
> > >
> > > Here is the fixed webrev:
> > > http://cr.openjdk.java.net/~lbourges/path2D/Path2D-8078192.3/ \
> > > <http://cr.openjdk.java.net/%7Elbourges/path2D/Path2D-8078192.3/>
> > > Laurent
> > >
> > > 2017-04-21 0:04 GMT+02:00 Laurent Bourgès
> > > <bourges.laurent@gmail.com <mailto:bourges.laurent@gmail.com>>:
> > >
> > > Sorry for the typo, I added also a newline before @since:
> > >
> > >
> > > 2017-04-20 23:04 GMT+02:00 Philip Race
> > > <philip.race@oracle.com <mailto:philip.race@oracle.com>>:
> > >
> > > You have a capital letter here and I think it must
> > > be lower case ..
> > >
> > >
> > > 2499 * @Since 10
> > >
> > > -phil.
> > >
> > >
> > > On 4/20/17, 1:58 PM, Laurent Bourgès wrote:
> > > > Hi Phil & Jim,
> > > >
> > > > Here is the updated webrev:
> > > > http://cr.openjdk.java.net/~lbourges/path2D/Path2D-8078192.2/
> > > > <http://cr.openjdk.java.net/%7Elbourges/path2D/Path2D-8078192.2/>
> > > >
> > > > Changes:
> > > > - trimToSize() return void
> > > > - fixed test + jtreg passed
> > > >
> > > > Bye,
> > > > Laurent
> > > >
> > > > 2017-04-20 21:30 GMT+02:00 Jim Graham
> > > > <james.graham@oracle.com
> > > > <mailto:james.graham@oracle.com>>:
> > > >
> > > > Hi Laurent,
> > > >
> > > > The implementation looks good, except that the
> > > > method chaining-style return value seems out of
> > > > place here. Similar trimToSize() methods in
> > > > Collections return void and none of the other
> > > > methods in this area use the method chaining
> > > > paradigm. In the interest of maintaining a
> > > > common design theme throughout 2D this method
> > > > should just return void.
> > > >
> > > > ...jim
> > > >
> > > >
> > > > On 4/18/17 11:49 PM, Laurent Bourgès wrote:
> > > >
> > > > Hi,
> > > >
> > > > Here is a first attempt to propose a Path2D
> > > > patch (based on JDK10):
> > > > http://cr.openjdk.java.net/~lbourges/path2D/Path2D-8078192.0/
> > > > <http://cr.openjdk.java.net/%7Elbourges/path2D/Path2D-8078192.0/>
> > > >
> > > > JBS:
> > > > https://bugs.openjdk.java.net/browse/JDK-8078192
> > > > <https://bugs.openjdk.java.net/browse/JDK-8078192>
> > > >
> > > > Please review the Path2D changes, notably
> > > > the javadoc (english) and the modified
> > > > Path2DCopyConstructor test which checks
> > > > all public Path2D methods on concrete
> > > > classes (Path2D.Float, Path2D.Double,
> > > > GeneralPath) after calling path.trimToSize()
> > > >
> > > > Cheers,
> > > > Laurent
> > > >
> > > >
> > > >
> > > >
> > > > --
> > > > --
> > > > Laurent Bourgès
> > >
> > >
> > >
> > >
> > > --
> > > --
> > > Laurent Bourgès
> > >
> > >
> > >
> > >
> > > --
> > > --
> > > Laurent Bourgès
> >
> >
> >
> >
> > --
> > --
> > Laurent Bourgès
>
[Attachment #3 (text/html)]
<html>
<head>
<meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
Oh this one is interesting. It already has a CSR :-)<br>
<br>
<br>
<br>
That's because we had created a CCC for it back in JDK 9 and then
withdrew it.<br>
<br>
All existing CSRs were ported over but as "confidential" since
honestly<br>
no one had time to look at all N thousand of them and vet them ...<br>
we vetted and opened enough to give people a flavour.<br>
<br>
So I think that "create a csr" doesn't appear if there already is
one<br>
since you can't see it.<br>
<br>
I opened it so you can now see it.<br>
<br>
<a class="moz-txt-link-freetext" \
href="https://bugs.openjdk.java.net/browse/CCC-8078192">https://bugs.openjdk.java.net/browse/CCC-8078192</a><br>
<br>
I re-opened it but don't seem to be able to target it to 10 or
assign it to you.<br>
<br>
I'll need to ask.<br>
<br>
-phil<br>
<br>
On 6/7/17, 1:35 PM, Laurent Bourgès wrote:
<blockquote
cite="mid:CAKjRUT6HyVDGHgiSun5TJs1xUcMs07Ubg6Zv-DzPamHb_VVbRA@mail.gmail.com"
type="cite">
<div dir="auto">Phil,
<div dir="auto"><br>
<div dir="auto">Could you initiate the CSR request in JBS ? </div>
<div dir="auto">I do not see how to create it from the current
bug: I looked at the More menu but I do not have any Create
CSR action.</div>
<div dir="auto"><br>
</div>
<div dir="auto">Cheers,</div>
<div dir="auto">Laurent</div>
</div>
</div>
<div class="gmail_extra"><br>
<div class="gmail_quote">Le 17 mai 2017 22:05, "Phil Race" <<a
moz-do-not-send="true" \
href="mailto:philip.race@oracle.com">philip.race@oracle.com</a>> a écrit :<br \
type="attribution"> <blockquote class="gmail_quote" style="margin:0 0 0
.8ex;border-left:1px #ccc solid;padding-left:1ex">
<div bgcolor="#FFFFFF" text="#000000"> Early next week is
the hope.<br>
<br>
-phil<br>
<br>
<div class="m_479405004460737818moz-cite-prefix">On
05/16/2017 02:20 PM, Laurent Bourgès wrote:<br>
</div>
<blockquote type="cite">
<div dir="ltr">
<div>
<div>Phil,<br>
<br>
</div>
Did you get any answer from the CSR process on this
bug ?<br>
<br>
</div>
Laurent<br>
<br>
</div>
<div class="gmail_extra"><br>
<div class="gmail_quote">2017-04-21 0:17 GMT+02:00
Philip Race <span dir="ltr"><<a
moz-do-not-send="true"
href="mailto:philip.race@oracle.com"
target="_blank">philip.race@oracle.com</a>></span>:<br>
<blockquote class="gmail_quote" style="margin:0 0 0
.8ex;border-left:1px #ccc solid;padding-left:1ex">
<div bgcolor="#FFFFFF" text="#000000"> OK.
Although we still need to wait for the CSR
process.<br>
<br>
-phil.
<div>
<div class="m_479405004460737818h5"><br>
<br>
On 4/20/17, 3:05 PM, Laurent Bourgès wrote:
<blockquote type="cite">
<div dir="ltr">
<div>Sorry (bad shortcut);<br>
<br>
Here is the fixed webrev:<br>
<a moz-do-not-send="true"
\
href="http://cr.openjdk.java.net/%7Elbourges/path2D/Path2D-8078192.3/"
\
target="_blank">http://cr.openjdk.java.net/~lb<wbr>ourges/path2D/Path2D-8078192.<wbr>3/</a><br>
<br>
</div>
Laurent<br>
</div>
<div class="gmail_extra"><br>
<div class="gmail_quote">2017-04-21 0:04
GMT+02:00 Laurent Bourgès <span
dir="ltr"><<a
moz-do-not-send="true"
href="mailto:bourges.laurent@gmail.com"
\
target="_blank">bourges.laurent@gmail.com</a>></span>:<br> <blockquote \
class="gmail_quote" style="margin:0 0 0
.8ex;border-left:1px #ccc
solid;padding-left:1ex">
<div dir="ltr">Sorry for the typo, I
added also a newline before
@since:<br>
<br>
</div>
<div class="gmail_extra">
<div>
<div
\
class="m_479405004460737818m_2760107019594871579h5"><br>
<div class="gmail_quote">2017-04-20
23:04 GMT+02:00 Philip Race
<span dir="ltr"><<a
moz-do-not-send="true"
href="mailto:philip.race@oracle.com"
\
target="_blank">philip.race@oracle.com</a>></span>:<br> <blockquote
class="gmail_quote"
style="margin:0 0 0
.8ex;border-left:1px #ccc
solid;padding-left:1ex">
<div bgcolor="#FFFFFF"
text="#000000"> You
have a capital letter
here and I think it must
be lower case ..<br>
<br>
<br>
2499 * @Since 10<br>
<br>
-phil.
<div>
<div
\
class="m_479405004460737818m_2760107019594871579m_2453146926484059732h5"><br> <br>
On 4/20/17, 1:58 PM,
Laurent Bourgès
wrote:
<blockquote
type="cite">
<div dir="ltr">
<div>
<div>Hi Phil
& Jim,<br>
<br>
</div>
Here is the
updated
webrev:<br>
<a
moz-do-not-send="true"
href="http://cr.openjdk.java.net/%7Elbourges/path2D/Path2D-8078192.2/"
\
target="_blank">http://cr.openjdk.java.net/~lb<wbr>ourges/path2D/Path2D-8078192.2<wbr>/</a><br>
<br>
</div>
<div>Changes:<br>
</div>
<div>-
trimToSize()
return void<br>
</div>
<div>- fixed
test + jtreg
passed<br>
</div>
<div><br>
</div>
<div>Bye,<br>
</div>
Laurent<br>
<div
class="gmail_extra"><br>
<div
\
class="gmail_quote">2017-04-20
21:30
GMT+02:00 Jim
Graham <span
dir="ltr"><<a
moz-do-not-send="true" href="mailto:james.graham@oracle.com"
\
target="_blank">james.graham@oracle.com</a>></span>:<br> <blockquote
class="gmail_quote"
style="margin:0
0 0
.8ex;border-left:1px
#ccc
solid;padding-left:1ex">Hi
Laurent,<br>
<br>
The
implementation
looks good,
except that
the method
chaining-style
return value
seems out of
place here.
Similar
trimToSize()
methods in
Collections
return void
and none of
the other
methods in
this area use
the method
chaining
paradigm. In
the interest
of maintaining
a common
design theme
throughout 2D
this method
should just
return void.<span
class="m_479405004460737818m_2760107019594871579m_2453146926484059732m_-3829295938273823275HOEnZb"><font
color="#888888"><br>
<br>
...jim</font></span>
<div
class="m_479405004460737818m_2760107019594871579m_2453146926484059732m_-3829295938273823275HOEnZb">
<div
class="m_479405004460737818m_2760107019594871579m_2453146926484059732m_-3829295938273823275h5"><br>
<br>
On 4/18/17
11:49 PM,
Laurent
Bourgès wrote:<br>
<blockquote
class="gmail_quote"
style="margin:0
0 0
.8ex;border-left:1px
#ccc
solid;padding-left:1ex">
Hi,<br>
<br>
Here is a
first attempt
to propose a
Path2D patch
(based on
JDK10):<br>
<a
moz-do-not-send="true"
href="http://cr.openjdk.java.net/%7Elbourges/path2D/Path2D-8078192.0/"
rel="noreferrer"
target="_blank">http://cr.openjdk.java.net/~lb<wbr>ourges/path2D/Path2D-8078192.0<wbr>/</a><br>
<br>
JBS: <a
moz-do-not-send="true"
href="https://bugs.openjdk.java.net/browse/JDK-8078192" rel="noreferrer"
target="_blank">https://bugs.openjdk.java.net/<wbr>browse/JDK-8078192</a><br>
<br>
Please review
the Path2D
changes,
notably the
javadoc
(english) and
the modified
Path2DCopyConstructor
test which
checks<br>
all public
Path2D methods
on concrete
classes
(Path2D.Float,
Path2D.Double,
GeneralPath)
after calling
path.trimToSize()<br>
<br>
Cheers,<br>
Laurent<br>
</blockquote>
</div>
</div>
</blockquote>
</div>
<br>
<br
clear="all">
<br>
-- <br>
<div
class="m_479405004460737818m_2760107019594871579m_2453146926484059732m_-3829295938273823275gmail_signature"
data-smartmail="gmail_signature">-- <br>
Laurent
Bourgès</div>
</div>
</div>
</blockquote>
</div>
</div>
</div>
</blockquote>
</div>
<br>
<br clear="all">
<br>
</div>
</div>
<span
\
class="m_479405004460737818m_2760107019594871579HOEnZb"><font color="#888888">-- \
<br> <div
class="m_479405004460737818m_2760107019594871579m_2453146926484059732gmail_signature"
data-smartmail="gmail_signature">-- <br>
Laurent Bourgès</div>
</font></span></div>
</blockquote>
</div>
<br>
<br clear="all">
<br>
-- <br>
<div
\
class="m_479405004460737818m_2760107019594871579gmail_signature" \
data-smartmail="gmail_signature">-- <br> Laurent Bourgès</div>
</div>
</blockquote>
</div>
</div>
</div>
</blockquote>
</div>
<br>
<br clear="all">
<br>
-- <br>
<div class="m_479405004460737818gmail_signature"
data-smartmail="gmail_signature">-- <br>
Laurent Bourgès</div>
</div>
</blockquote>
<br>
</div>
</blockquote>
</div>
</div>
</blockquote>
</body>
</html>
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic