[Webkit-unassigned] [Bug 72017] [EFL] Add NULL checks after memory allocation

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Jan 3 07:14:35 PST 2012


https://bugs.webkit.org/show_bug.cgi?id=72017


Grzegorz <g.czajkowski at samsung.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #117409|0                           |1
        is obsolete|                            |
 Attachment #117409|review?                     |
               Flag|                            |
 Attachment #120943|                            |review?
               Flag|                            |




--- Comment #11 from Grzegorz <g.czajkowski at samsung.com>  2012-01-03 07:14:35 PST ---
Created an attachment (id=120943)
 --> (https://bugs.webkit.org/attachment.cgi?id=120943&action=review)
proposed patch

As a conclusion of above discussion NULL checking isn't needed in these cases.
I only replaced calling alloca to C++ new placement for three reasons:
1. alloca it is not very portable.
2. NULL checking isn't required as new never returns NULL,
3. in compliance with WebKit coding style.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list