[EMAIL PROTECTED]">
  • 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.
 

[EMAIL PROTECTED]">
  • Need some perf verification
Are you going to do this whitebox w/ programmatic timing (clock checks in the download code?)?
I was going to use TestProtocols.
[EMAIL PROTECTED]">
  • Check for regressions
Agreed. We need to really regress these changes before dropping them in.
Any help would be appreciated.

Reply via email to