Quoting Thomas Butter <[EMAIL PROTECTED]>:

> For example if in line 257:           
>            requestLine.uri[readCount] = (char) buf[pos];
>            readCount++;
>            pos++;
> 
> buf[pos] is undefined the last fill called by the last read() only got 
> one byte (which was enough for the last read()).

Yes, but (pos >= count) should be true then (on the next loop iteration), so it 
should attempt to read again from the input stream (right ?). I tested that 
stream with very small buffers (a 4 bytes internal buffer), so the situation 
you described must have happened at some point during my testing.

> I would volunteer rewriting this class if you agree with my
> assumptions.

If extending BIS is indeed a problem, I would vote +1 in duplicating the BIS 
code into SocketInputStream, and -1 for layering another IS (for performance 
reasons).

Remy

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, email: [EMAIL PROTECTED]

Reply via email to