<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Sep 23, 2014 at 11:13 AM, Matthew Jordan <span dir="ltr"><<a href="mailto:mjordan@digium.com" target="_blank" onclick="window.open('https://mail.google.com/mail/?view=cm&tf=1&to=mjordan@digium.com&cc=&bcc=&su=&body=','_blank','location=yes,menubar=yes,resizable=yes,width=800,height=600');return false;">mjordan@digium.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Sep 23, 2014 at 11:29 AM, Paul Belanger <span dir="ltr"><<a href="mailto:paul.belanger@polybeacon.com" target="_blank" onclick="window.open('https://mail.google.com/mail/?view=cm&tf=1&to=paul.belanger@polybeacon.com&cc=&bcc=&su=&body=','_blank','location=yes,menubar=yes,resizable=yes,width=800,height=600');return false;">paul.belanger@polybeacon.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div>On Tue, Sep 23, 2014 at 11:45 AM, George Joseph<br>
<<a href="mailto:george.joseph@fairview5.com" target="_blank" onclick="window.open('https://mail.google.com/mail/?view=cm&tf=1&to=george.joseph@fairview5.com&cc=&bcc=&su=&body=','_blank','location=yes,menubar=yes,resizable=yes,width=800,height=600');return false;">george.joseph@fairview5.com</a>> wrote:<br>
> I've been working on some changes for config.c and in the process I've found<br>
> 5 instances of someone attempting to do "cat->name == category_name" instead<br>
> of "strcmp(cat->name, category_name)".    Example:<br>
><br></div></div></blockquote></div></div></div></blockquote><div><snip> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div>> My question is...  Should I fix these so the case sensitive match works and<br>
> runs first or just remove the first loop so the match is always<br>
> case-insensitive?   I'm hoping the latter not only because it makes the code<br>
> simpler but because that's how it's worked for years and suddenly making the<br>
> match case sensitive might cause unexpected problems.<br>
><br>
> Thoughts?<br>
><br>
</div></div>For me, case sensitive.  Because I config files that do have:<br>
<br>
[Foo]<br>
<br>
[foo]<br>
<br>
[fOO]<br>
<br>
don't ask, long story.<br>
<span><font color="#888888"></font></span><br clear="all"></blockquote></div><br></div></div></blockquote><div><br></div><div>Which file?  I'd like to test a before and after scenario to make sure whatever I change still works.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"></div><div class="gmail_extra">And it currently works? :-D<br><br></div><div class="gmail_extra">(Just kidding)<br><br></div><div class="gmail_extra">If that is the case, then obviously any change we make here should be done to trunk only.<br><br>I'll leave the impending case-war to others... :-)<span><font color="#888888"><br></font></span></div><span><font color="#888888"><div class="gmail_extra"><br></div></font></span></div></blockquote><div> </div><div>Ok, I think I'll fix the ones that are definitely broken so they do case sensitive first then fall back to case insensitive.   That should preserve the current behavior.</div><div><br></div></div></div></div>