Judson Valeski wrote:
> Doug Turner wrote:
>
> >>> * Removed Available() call in OnDataAvailable. We should just
> >>> the byte count that was passed to us.
> >>>
> >> agreed. I wonder if Available() is of any use?
> >
> >
> > Ugh. all over the place. It is all wrong too. Do a lxr search for
> > OnDataAvailable(). The result from Available() does not always yield
> > the same value as what the read/write count is. If you read over this
> > count, you may block or possible (In passing darin mentioned this to
> > me) screw up the internal buffering. I wrote up a bug but can not
> > find it right now.
> >
> Sure. I know it's used everywhere, I was just wondering if all the usage
> cases are just after an OnDataAvailable() call (where they could easily
> be removed).
Yes. and should be. The bug:
http://bugzilla.mozilla.org/show_bug.cgi?id=63242
