[PATH] bug 4254082 & 4479751 & 6192696 draft 2

[Pascal S. de Kloe]

On Sunday 08 July 2007 19:45:05 you wrote:
> Speed of patch generation is not a priority when changing the
> behavior of a core Java library.
>
> Frankly, the quality of your patches convinces me that you shouldn't
> be submitting patches to
> the core Java libraries.

The reason why I post those early drafts is to get feedback before the 
polishing. Maybe some experienced developer can tell if I'm on the 
right track or not until one day I can see it myself.


> Among other problems, your most recent patch changed the behavior
> when skip is called on a closed BufferedInputStream. Rather than
> throwing IOException, you changed the behavior so that it now throws
> a NullPointerException.

That's exactly the kind of feedback we need. :)

Note the first line of skip "verifyNotClose();". That method throws a 
IOException if the stream was closed, i.e., if close() was called.


> I discourage you from submitting more patches. Your eagerness exceeds
> your proficiency.

I'm not convinced that I'm that bad yet...
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 827 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.openjdk.java.net/pipermail/core-libs-dev/attachments/20070708/02b6e138/signature.asc>