RFR JDK-8176379: java.util.Base64 mime encoder behaves incorrectly if initialized with a line length of size 1-3

Xueming Shen xueming.shen at oracle.com
Mon Jan 29 17:52:46 UTC 2018


On 01/26/2018 03:53 PM, Paul Sandoz wrote:
> Base64
>>
> 138 lineLength = lineLength>>  2<<  2;
>
> I know the code was shuffled up but i find the following clearer in terms of bit manipulation:
>
>    // Round down to nearest mulitple of 4
>    lineLength&= ~0b11
>

sure, we can go the straightforward bit manipulation.

> TestBase64
>>
> Is there a test passing in a negative value for len?
>

Actually I have a dedicated test case for "mime encoder maxlen".
*test/jdk/java/util/Base64/Base64GetEncoderTest.java*
I have updated it to test the negative and the < 4 maxlen.

http://cr.openjdk.java.net/~sherman/8176379/webrev/

webrev has been updated according.

Thanks!
Sherman


More information about the core-libs-dev mailing list