Laurent,
Investigation has determined that since this was ported over in the "CCC
migration project",
due to the way that is set up it can't be assigned to non-Oracle employees.
It won't affect any new CSRs that are created.
Since the likelihood of any other similar case is extremely low, the
simplest answer is
that either Jim or myself will need to own and edit this one on your behalf.
Send Jim or myself the changes you would like to make off-line and we'll
take care of it.
-phil.
On 06/07/2017 01:56 PM, Philip Race wrote:
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.r...@oracle.com
<mailto:philip.r...@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.r...@oracle.com
<mailto:philip.r...@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.laur...@gmail.com <mailto:bourges.laur...@gmail.com>>:
Sorry for the typo, I added also a newline before @since:
2017-04-20 23:04 GMT+02:00 Philip Race
<philip.r...@oracle.com <mailto:philip.r...@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.gra...@oracle.com
<mailto:james.gra...@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