[webkit-reviews] review granted: [Bug 116445] Move MD5, SHA1 unit tests from WTF to TestWebKitAPI : [Attachment 203369] Patch
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Thu May 30 16:29:57 PDT 2013
Benjamin Poulain <benjamin at webkit.org> has granted Zan Dobersek
<zandobersek at gmail.com>'s request for review:
Bug 116445: Move MD5, SHA1 unit tests from WTF to TestWebKitAPI
https://bugs.webkit.org/show_bug.cgi?id=116445
Attachment 203369: Patch
https://bugs.webkit.org/attachment.cgi?id=203369&action=review
------- Additional Comments from Benjamin Poulain <benjamin at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=203369&action=review
Great!
> Tools/TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj:761
> + CD5393C91757BAC400C07123 /* SHA1.cpp */,
> + CD5393C71757BA9700C07123 /* MD5.cpp */,
The build section should be sorted alphabetically.
> Tools/TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj:1182
> 2E7765CF16C4D81100BA2BB1 /* mainMac.mm in
Sources */,
> + CD5393C81757BA9700C07123 /* MD5.cpp in Sources
*/,
> + CD5393CA1757BAC400C07123 /* SHA1.cpp in Sources
*/,
Ditto.
> Tools/TestWebKitAPI/Tests/WTF/MD5.cpp:29
> +/*
> + * Copyright (C) 2010 Google Inc. All rights reserved.
> + *
> + * Redistribution and use in source and binary forms, with or without
> + * modification, are permitted provided that the following conditions are
> + * met:
> + *
> + * * Redistributions of source code must retain the above copyright
> + * notice, this list of conditions and the following disclaimer.
> + * * Redistributions in binary form must reproduce the above
> + * copyright notice, this list of conditions and the following disclaimer
> + * in the documentation and/or other materials provided with the
> + * distribution.
> + * * Neither the name of Google Inc. nor the names of its
> + * contributors may be used to endorse or promote products derived from
> + * this software without specific prior written permission.
> + *
> + * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
> + * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
> + * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
> + * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
> + * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
> + * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
> + * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
> + * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
> + * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
> + * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
> + * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
> + */
I am not sure this is the right copyright.
If the test comes from the original sqlite code, it should be the one starting
with "This code implements the MD5 message-digest algorithm."
> Tools/TestWebKitAPI/Tests/WTF/MD5.cpp:54
> + // MD5 Test suite from http://www.ietf.org/rfc/rfc1321.txt
Missing period.
> Tools/TestWebKitAPI/Tests/WTF/SHA1.cpp:1
> +/*
ditto for the copyright.
More information about the webkit-reviews
mailing list