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

Greg Clayton gclayton at apple.com
Fri Feb 4 10:59:12 PST 2011


% svn commit
Sending        source/Plugins/Disassembler/llvm/DisassemblerLLVM.cpp
Sending        source/Plugins/Process/MacOSX-User/source/ProcessMacOSXLog.cpp
Sending        source/Plugins/Process/gdb-remote/ProcessGDBRemoteLog.cpp
Sending        source/lldb-log.cpp
Transmitting file data ....
Committed revision 124889.


On Feb 3, 2011, at 10:15 PM, kirk.beitz at nokia.com wrote:

> 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 .
> 
> <strcasestr-strncasecmp.124846.patch>
> 
> ++ kirk beitz : nokia : austin + san diego : kirk.beitz at nokia.com ++
> 
> 
> 
> _______________________________________________
> lldb-dev mailing list
> lldb-dev at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev





More information about the lldb-dev mailing list