[cfe-commits] [patch] change libc++'s string hash function to cityhash64

Chandler Carruth chandlerc at google.com
Fri Dec 9 17:15:53 PST 2011


On Fri, Dec 9, 2011 at 5:14 PM, Chandler Carruth <chandlerc at google.com>wrote:

> On Fri, Dec 9, 2011 at 4:50 PM, Howard Hinnant <hhinnant at apple.com> wrote:
>
>> On Dec 9, 2011, at 7:17 PM, Craig Silverstein wrote:
>>
>> > craig
>> > Index: include/memory
>>
>> I'm drawing a blank.
>>
>> $ patch  -p0 < /Users/hhinnant/Desktop/temp.patch
>> patching file include/memory
>> patch: **** malformed patch at line 7: template <class _Tp> struct hash;
>>
>> Anyone have a clue as to why the enclosed temp.patch is malformed?
>
>
> Some email client somewhere has stripped leading whitespace. Unified diff
> patch files need a single leading space character on the context lines, and
> all of the leading whitespace on the patch you posted is gone.
>

I've re-attached Craig's original patch which looks to still have the
correct formatting...
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20111209/66739a02/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: cityhash.patch
Type: application/octet-stream
Size: 9971 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20111209/66739a02/attachment.obj>


More information about the cfe-commits mailing list