Closed GoogleCodeExporter closed 9 years ago
Where is the patch with the fix?
I'll be glad to apply it.
Original comment by rafael2k...@gmail.com
on 6 May 2010 at 7:18
I just realized this bug is present in the stream I'm testing right now.
Original comment by rafael2k...@gmail.com
on 6 May 2010 at 7:29
This lines you posted are the fix?
Original comment by rafael2k...@gmail.com
on 6 May 2010 at 7:31
Index: src/BufferedSink.cpp
===================================================================
--- src/BufferedSink.cpp (revision 467)
+++ src/BufferedSink.cpp (working copy)
@@ -330,10 +330,15 @@
// the internal buffer is empty, try to write the fresh data
soFar = 0;
- if ( inp != outp ) {
+ if ( inp == outp ) { /* Edwin van den Oetelaar bug fix 26/4/2010 */
while ( soFar < len && sink->canWrite( 0, 0) ) {
- soFar += sink->write( b + soFar, len - soFar);
- }
+ try {
+ soFar += sink->write( b + soFar, len - soFar);
+ } catch (Exception &e) {
+ reportEvent(3,"Exception caught in BufferedSink :: write3\n");
+ throw; /* up a level */
+ }
+ }
}
length = soFar;
Original comment by rafael2k...@gmail.com
on 6 May 2010 at 7:45
Any more comments about this bug?
I'll apply the patch.
Original comment by rafael2k...@gmail.com
on 6 May 2010 at 8:10
applied in 468.
Original comment by rafael2k...@gmail.com
on 6 May 2010 at 8:20
Original issue reported on code.google.com by
oetelaar.automatisering
on 25 Apr 2010 at 11:34