ChunkedInputStream/test.txt

Joseph D. Darcy Joe.Darcy at Sun.COM
Wed Sep 24 10:34:32 PDT 2008


Mark Wielaard wrote:
> On Tue, 2008-09-23 at 09:46 -0700, Kelly O'Hair wrote:
>   
>> Probably true that the test can be re-written, and you are right, a 500K
>> binary file is not something I want in the repository either.
>> Unfortunately, some engineers have the historical and bad habit of just
>> adding in files to workspaces and repositories without thinking about it
>> much, especially with tests. This habit must stop now, hopefully we can
>> catch this during the code review phase.
>>     
>
> And it seems Christopher Hegarty already took care of the rewrite:
> http://hg.openjdk.java.net/jdk7/jsn/jdk/rev/de7864ec310c
>
> I didn't see any code review on list, but basically the byte[] used for
> handling the request is now filled with data during runtime with a
> simple for loop stuffing the array in a static block in a simple
> pattern. Which eliminates the need for the test.txt file (removed in the
> patch above).
>
> Probably a nice idea to pull this into jdk6 also.
>
> Cheers,
>   

Yes, I've already asked Chris to fix this in OpenJDK 6 too.

Regards,

-Joe




More information about the jdk6-dev mailing list