[Mono-dev] PageThemeFileParser.AddDirective
Chris Toshok
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
them.
3) I can't read ms-help urls.
Chris
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
> everywhere.
>
> But anyway OrdinalComparison is more appropriate in this patch (with
> ignore case), you can look here for an example of unexpected results:
>
> ms-help://MS.VSCC.v80/MS.MSDN.v80/MS.NETDEVFX.v20.en/cpref2/html/M_Syste
> m_String_Compare_2_16e9e883.htm
>
> So please review the fixed patch.
>
> Regards,
> Andrew.
> _______________________________________________
> Mono-devel-list mailing list
> Mono-devel-list at lists.ximian.com
> http://lists.ximian.com/mailman/listinfo/mono-devel-list
More information about the Mono-devel-list
mailing list