[Fwd: Re: [Mono-devel-list] System.Data.DataSet.ReadXml reimplementation]

Atsushi Eno atsushi at ximian.com
Tue Jun 21 03:49:34 EDT 2005


Hello Konstantin,

Konstantin Triger wrote:
> Hi Eno,
> 
> I committed the code before I saw your comments on Boris email. Actually 
> I missed it at all and Boris forwarded it to me. In any case I didn't 
> want to ignore it and would like, with your help, create the best 
> implementation possible.
> 
> The new implementation solves some bugs, little bit cleaner as it does 
> not have "gotos", but performs worse. I think that with a very small 
> effort it can perform better. I really need your help to make this 
> change safe.
> Can you please give your opinion on an assumption that right after the 
> schema must reside data? If you agree, with a small change I can make a 
> real perf improvement and we will have a really better implementation.

I don't think you could quote my post without seeing it. Anyways,
Am not seeing what you see. Maybe it would be the best if you
post testcases that shows why the patch is required. Next, the
patch has several changes in one file. There are things that could
be justified and that might not be. They do not look tied
together, so it is better if they are split.

Without them I don't think any of discussions can go efficiently.
Depending on the testcases that performant way might not be
possible. As long as I know MS.NET implementation is so cheap
that always reads document into DOM or something.

If XML schema is the document root, then there is no data content.
Existing implementation did handle content schema (at least it was
designed to do that) as long as it is the top-level content of the 
document element. I am not aware of practical cases that such
schemas in further depth are consumed (since 99.999% of .NET
DataSet users is not likely to have such documents).

Atsushi Eno



More information about the Mono-devel-list mailing list