[llvm-commits] FoldingSet of string with NULLs inside

Nick Lewycky nicholas at mxc.ca
Thu Feb 5 23:35:20 PST 2009


I noticed that FoldingSet has two copies of the same code for AddString. 
I took a stab at refactoring them into one shared function that just 
takes two char*'s. This has the added benefit of allowing strings that 
have NULLs in the middle.

I'm not familiar with this code so I thought I'd send it out for review 
first.

Nick Lewycky
-------------- next part --------------
A non-text attachment was scrubbed...
Name: folding-string.patch
Type: text/x-diff
Size: 2970 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20090205/d9ea96d4/attachment.patch>


More information about the llvm-commits mailing list