[lldb-dev] patch proposal: replacing strcasestr() w/strncasecmp() for beg-of-haystack comparison

kirk.beitz at nokia.com kirk.beitz at nokia.com
Thu Feb 3 22:15:00 PST 2011


Hello LLDB Developers ++

I would like to propose the following patch to replace selected instances of strcasestr() with strcasecmp()

All of these instances are instances where the string being searched for is being compared against only the beginning of the test string.  this can be seen in the patch in the value of the equality (i.e. ==) test.

The man page http://linux.die.net/man/3/strcasestr confirms that strcasestr() is a non-standard extension.

This patch build with xcode … and with gcc under MinGW.

thanks in advance for any review/feedback/commit .

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-dev/attachments/20110204/50bffbff/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: strcasestr-strncasecmp.124846.patch
Type: application/octet-stream
Size: 21760 bytes
Desc: strcasestr-strncasecmp.124846.patch
URL: <http://lists.llvm.org/pipermail/lldb-dev/attachments/20110204/50bffbff/attachment.obj>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-dev/attachments/20110204/50bffbff/attachment.htm>


More information about the lldb-dev mailing list