[webkit-reviews] review denied: [Bug 133571] [JavaScriptCore] Fix FTL on platform EFL. : [Attachment 232611] Patch
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Fri Jun 6 10:11:31 PDT 2014
Filip Pizlo <fpizlo at apple.com> has denied László Langó
<llango.u-szeged at partner.samsung.com>'s request for review:
Bug 133571: [JavaScriptCore] Fix FTL on platform EFL.
https://bugs.webkit.org/show_bug.cgi?id=133571
Attachment 232611: Patch
https://bugs.webkit.org/attachment.cgi?id=232611&action=review
------- Additional Comments from Filip Pizlo <fpizlo at apple.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=232611&action=review
You appear to have lifted this code, without attribution, from
https://llvm.org/svn/llvm-project/libcxxabi/trunk/src/Unwind/AddressSpace.hpp.
That's not cool, man. In particular, the LLVM license clearly states that
LLVM's copyright notice must be preserved, and you have failed to do that.
> Source/JavaScriptCore/ftl/FTLUnwindInfo.cpp:234
> +typedef uintptr_t intPtr;
Why?
More information about the webkit-reviews
mailing list