On 2/20/2014 12:05 AM, Henry Jen wrote:
On 02/19/2014 02:59 PM, Henry Jen wrote:
On 02/19/2014 01:46 PM, Phil Race wrote:
W.r.t the following change ...
http://cr.openjdk.java.net/~henryjen/jdk9/8033716/1/webrev/src/share/classes/com/sun/imageio/plugins/jpeg/DHTMarkerSegment.java.sdiff.html
Hello Henry,
please see my comments regrading the fallthrough warnings inline.
On 2/20/2014 12:05 PM, Henry Jen wrote:
On 02/19/2014 02:59 PM, Henry Jen wrote:
On 02/19/2014 01:46 PM, Phil Race wrote:
W.r.t the following change ...
http://cr.openjdk.java.net/~henryjen/jdk9/8033716/1/webrev
On 02/19/2014 02:59 PM, Henry Jen wrote:
On 02/19/2014 01:46 PM, Phil Race wrote:
W.r.t the following change ...
http://cr.openjdk.java.net/~henryjen/jdk9/8033716/1/webrev/src/share/classes/com/sun/imageio/plugins/jpeg/DHTMarkerSegment.java.sdiff.html
145 class Htable implements Clonea
On 02/19/2014 01:46 PM, Phil Race wrote:
.
http://cr.openjdk.java.net/~henryjen/jdk9/8033716/1/webrev/src/share/classes/com/sun/imageio/plugins/gif/GIFImageReader.java.sdiff.html
230 public Iterator getImageTypes(int
imageIndex) throws IIOException {
Nitpicky perhaps, but this looks > 80 c
http://cr.openjdk.java.net/~henryjen/jdk9/8033716/1/webrev/src/share/classes/com/sun/imageio/plugins/gif/GIFImageReader.java.sdiff.html
230 public Iterator getImageTypes(int imageIndex)
throws IIOException {
Nitpicky perhaps, but this looks > 80 chars. If so please split it.
-
W.r.t
As there is no other comments so far, I posted updated version adapted
comments from Phil.
- removed public field comments from BMPMetadata
- removed not needed comments from GIFImageMetadata
http://cr.openjdk.java.net/~henryjen/jdk9/8033716/1/webrev/
Cheers,
Henry
On 02/07/2014 04:27 PM, He
On 02/07/2014 03:00 PM, Phil Race wrote:
BMPMetadata.java
94 // Fields from CommentExtension
95 // List of byte[]
96 public List comments = null; // new ArrayList();
hmm .. how did you decide this was correct, other than trusting the
comment?
For this one, I took it from t
On 2/7/2014 2:20 PM, Phil Race wrote:
Yes, it should get 2d review and I will look at this soon as priorities
permit but the *conclusion* is that the client team
ask that such changes go into the client forest. If this is a problem for
you then we will do it on your behalf. We do not want client
BMPMetadata.java
94 // Fields from CommentExtension
95 // List of byte[]
96 public List comments = null; // new ArrayList();
hmm .. how did you decide this was correct, other than trusting the comment?
The thing is I can't actually see where this field is used and I'm incline
Yes, it should get 2d review and I will look at this soon as priorities
permit but the *conclusion* is that the client team
ask that such changes go into the client forest. If this is a problem for
you then we will do it on your behalf. We do not want client changes
directly into dev. That is a ve
Thanks Joe for reviewing.
I would like to get 2d developer review as well before pushing this, let
me know if that's not necessary.
Also there was a discussion ealier on whether such change should go to
client or jdk9/dev repo, do we have a conclusion?
Cheers,
Henry
On 02/05/2014 06:01 PM,
Hi Henry,
On 02/05/2014 12:19 PM, Henry Jen wrote:
Hi,
Please review the webrev to clean up raw and unchecked warnings in
com.sun.imageio packag at,
http://cr.openjdk.java.net/~henryjen/jdk9/8033716/0/webrev/
The more significant change in this webrev is that I have changed the
clone() met
Hi,
Please review the webrev to clean up raw and unchecked warnings in
com.sun.imageio packag at,
http://cr.openjdk.java.net/~henryjen/jdk9/8033716/0/webrev/
The more significant change in this webrev is that I have changed the
clone() method of MarkerSegment-derived classes to return exact
13 matches
Mail list logo