This is the mail archive of the java-patches@sources.redhat.com mailing list for the Java project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Re: [PATCH] java.util.zip code updates


>>>>> "Mark" == Mark Wielaard <mark@klomp.org> writes:

Mark> OK to commit?

Mark> -    c = crc_table[(c ^ bval) & 0xff] ^ (c >>> 8);
Mark> +    c = crc_table[(c ^ (bval & 0xff)) & 0xff] ^ (c >>> 8);

When can this change matter?

Mark> +  public int available() {

This brace should be on the next line.

Other than that, this patch is ok.

Tom

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]