[Mono-bugs] [Bug 56510][Nor] Changed - XmlDataDocument loading stream bug

[bugzilla-daemon@bugzilla.ximian.com]

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 atsushi@ximian.com.

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

--- shadow/56510	2004-04-06 20:28:55.000000000 -0400
+++ shadow/56510.tmp.21778	2004-04-06 21:41:33.000000000 -0400
@@ -3,14 +3,14 @@
 Version: unspecified
 OS: GNU/Linux [Other]
 OS Details: Crux 1.3 / kernel 2.6.5
 Status: NEW   
 Resolution: 
 Severity: Unknown
-Priority: Blocker
-Component: Sys.XML
+Priority: Normal
+Component: Sys.Data
 AssignedTo: mono-bugs@ximian.com                            
 ReportedBy: yom@iaelu.net               
 QAContact: mono-bugs@ximian.com
 TargetMilestone: ---
 URL: 
 Cc: 
@@ -420,6 +420,20 @@
 
 ##### XmlDataDocument:Load #####
 
 i've been changing this localy on the 0.31 release inside
 mcs/class/System.Data/System.Xml and after some test, it's working now
 for both the XmlTextReader and Stream object.
+
+------- Additional Comments From atsushi@ximian.com  2004-04-06 21:41 -------
+Yes, that extraneous XmlTextReader is the bad point. However, just
+using base.Load() does not make sense since the following lines in the
+Load() method will be just ignored.
+
+People should be more careful before changing "priority" of a bug.
+Follow the linked description. Blocker means "crashes, loss of data,
+severe memory leak." (Here "loss of data" is ambiguous, but it should
+be assumed such like deleting unexpected files.)
+
+Also modified component since it is System.Data stuff.
+
+Thanks for the bug report anyways.