[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