toshok at ximian.com
Tue Apr 11 12:30:31 EDT 2006
Look, I have reviewed the patch.
1) There's no need for the patch, as there's already code in the repo
that does what's needed.
2) You're suggesting changing to a different string method than what is
used in all of the other cases where we compare directive strings.
Check out all the other parser classes and their overridden AddDirective
methods. If you're not going to change all of them, don't change any of
3) I can't read ms-help urls.
On Tue, 2006-04-11 at 08:31 -0700, Andrew Skiba wrote:
> > I disagree that it is more appropriate - it's case sensitive.
> I checked, and found that you are right and directives are case
> insensitive. FYI TemplateControlParser uses case-sensitive comparison
> But anyway OrdinalComparison is more appropriate in this patch (with
> ignore case), you can look here for an example of unexpected results:
> So please review the fixed patch.
> Mono-devel-list mailing list
> Mono-devel-list at lists.ximian.com
More information about the Mono-devel-list