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

[Howard Hinnant]

On Dec 9, 2011, at 2:45 PM, Craig Silverstein wrote:

> } I'm not a lawyer either; I'll ask someone who is.
> 
> I talked to the lawyer, and he confirmed there are no licensing issues
> (not that there was necessarily any doubt), and as for acknolwedging
> copyright, we are happy to do whatever is standard.
> 
> What is the standard way to do this?  Should I add the text from llvm
> -- "The following pieces of software have additional or alternate
> copyrights, licenses, and/or restrictions:" -- to libcxx/LICENSE.TXT?
> Where should I put the copyright designation? -- in include/memory?
> 
> We can go along with whatever you think best, so just let us know what
> that is. :-)
> 
> craig

I think a Google entry in CREDITS.TXT would be good, and that line should state that Google is the owner and contributor of the CityHash algorithm.  I think that is the proper thing to do from reading this line:

> The LLVM project does not require copyright assignments, which means that the copyright for the code in the project is held by its respective contributors who have each agreed to release their contributed code under the terms of the LLVM License.

>From this link:  http://llvm.org/docs/DeveloperPolicy.html#clp

Howard