[Mono-bugs] [Bug 76218][Nor] Changed - Tokenizer/Seekable stream bugs.

bugzilla-daemon at bugzilla.ximian.com bugzilla-daemon at bugzilla.ximian.com
Tue Sep 27 12:23:54 EDT 2005


Please do not reply to this email- if you want to comment on the bug, go to the
URL shown below and enter your comments there.

Changed by rharinath at novell.com.

http://bugzilla.ximian.com/show_bug.cgi?id=76218

--- shadow/76218	2005-09-27 12:14:26.000000000 -0400
+++ shadow/76218.tmp.14964	2005-09-27 12:23:54.000000000 -0400
@@ -10,12 +10,13 @@
 Component: C#
 AssignedTo: rharinath at novell.com                            
 ReportedBy: miguel at ximian.com               
 QAContact: mono-bugs at ximian.com
 TargetMilestone: ---
 URL: 
+Cc: atsushi at ximian.com,martin at ximian.com
 Summary: Tokenizer/Seekable stream bugs.
 
 > 
 >     Please download this:
 > 
 >       http://primates.ximian.com/~miguel/tmp/fileCopier.tar.bz2
@@ -120,6 +121,19 @@
 When I saw this patch for the first time on the list, I remember
 asking  Atsushi about the performance implications of adding more
 tests to the pipeline. 
 
 But it turned out that his path consumed less memory and was faster
 than our extra layer (SeekableStream) which impacts every read.
+
+------- Additional Comments From rharinath at novell.com  2005-09-27 12:23 -------
+I have a rewrite that gives up any hope of trying to match up with the
+underlying Stream -- so no more bugs due to us guessing the number of
+bytes a given number of chars occupied originally.
+
+For the gain in simplicity, we lose the ability to seek arbitrarily in
+the file.  We can only seek within 1024 chars, effective limiting our
+lookahead to that.  It's much cheaper than the old code, and supports
+almost arbitrary (upto 1024 chars) seeks.  If a parser cannot decide
+in 1024 bytes, it might as well give up ;-)
+
+Patch attached.


More information about the mono-bugs mailing list