approved.
--Semyon
On 4/5/2016 9:45 PM, Phil Race wrote:
I have an approved CCC sitting waiting for a 2nd reviewer on the code
change
-phil.
On 03/25/2016 11:31 AM, Sergey Bylokhov wrote:
Looks fine.
On 25.03.16 20:56, Phil Race wrote:
After a hallway conversation I have decided to change
I have an approved CCC sitting waiting for a 2nd reviewer on the code change
-phil.
On 03/25/2016 11:31 AM, Sergey Bylokhov wrote:
Looks fine.
On 25.03.16 20:56, Phil Race wrote:
After a hallway conversation I have decided to change it to NPE.
Not that I think it matters greatly either way si
Looks fine.
On 25.03.16 20:56, Phil Race wrote:
After a hallway conversation I have decided to change it to NPE.
Not that I think it matters greatly either way since I don't think
developers will be passing null to this method anyway :-)
The test is changed to match ..
http://cr.openjdk.java.n
After a hallway conversation I have decided to change it to NPE.
Not that I think it matters greatly either way since I don't think
developers will be passing null to this method anyway :-)
The test is changed to match ..
http://cr.openjdk.java.net/~prr/8146324.1
-phil.
On 03/24/2016 03:59 PM,
On 3/25/2016 1:43 AM, Andrej Golovnin wrote:
Hi Phil,
That is true .. although I think I have previously been convinced that IAE
is generally the better choice for such a case, I found only deriveFont
that throws IAE for null in this file.
So the question is whether to be consistent or to sta
On 25.03.16 1:22, Phil Race wrote:
That is true .. although I think I have previously been convinced that IAE
is generally the better choice for such a case, I found only deriveFont
that throws IAE for null in this file.
deriveFont were changed(NPE to IAE) in 2001, but other methods and the
ne
Hi Phil,
> That is true .. although I think I have previously been convinced that IAE
> is generally the better choice for such a case, I found only deriveFont
> that throws IAE for null in this file.
> So the question is whether to be consistent or to start picking the better
> option.
> So if t
That is true .. although I think I have previously been convinced that IAE
is generally the better choice for such a case, I found only deriveFont
that throws IAE for null in this file.
So the question is whether to be consistent or to start picking the
better option.
So if there were no existing
Hi Phil,
> bug : https://bugs.openjdk.java.net/browse/JDK-8146324
> fix : http://cr.openjdk.java.net/~prr/8146324/
maybe the line 794:
794 * @return true if the specified text will need special layout.
should be changed to:
794 * @return {@code true} if the specified text will need s
Hi, Phil.
Should this new method throw IAE? It seems that NPE can be thrown
instead, At least other methods in Font.java including the newly added
createFonts() throw NPE.
On 25.03.16 0:16, Phil Race wrote:
bug : https://bugs.openjdk.java.net/browse/JDK-8146324
fix : http://cr.openjdk.java.ne
bug : https://bugs.openjdk.java.net/browse/JDK-8146324
fix : http://cr.openjdk.java.net/~prr/8146324/
Developers have used an internal API to detect when TextLayout or
GlyphVector.layoutGlyphVectorwill be needed since that is a slower path.
This provides a public equivalent capability.
I have i
11 matches
Mail list logo