Bug 86610 - [EFL] Do not check NULL pointer allocated by new
: [EFL] Do not check NULL pointer allocated by new
Status: RESOLVED FIXED
Product: WebKit
Classification: Unclassified
Component: WebKit EFL
: 528+ (Nightly build)
: All Linux
: P2 Minor
Assigned To: Grzegorz Czajkowski
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-05-16 05:28 PDT by Grzegorz Czajkowski
Modified: 2012-05-17 01:21 PDT (History)
4 users (show)

See Also:


Attachments
proposed patch (3.06 KB, patch)
2012-05-16 06:20 PDT, Grzegorz Czajkowski
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Grzegorz Czajkowski 2012-05-16 05:28:14 PDT
WebKit-Efl doesn't check pointers that are initialized by new.
Those checks are not needed because new operator doesn't return NULL.
Only an overloaded version of new returns NULL pointer.
Comment 1 Grzegorz Czajkowski 2012-05-16 06:20:53 PDT
Created attachment 142239 [details]
proposed patch
Comment 2 Raphael Kubo da Costa (:rakuco) 2012-05-16 07:30:08 PDT
Comment on attachment 142239 [details]
proposed patch

Looks good.
Comment 3 Gyuyoung Kim 2012-05-16 17:37:34 PDT
Comment on attachment 142239 [details]
proposed patch

Looks good to me, thank you.
Comment 4 WebKit Review Bot 2012-05-17 01:21:08 PDT
Comment on attachment 142239 [details]
proposed patch

Clearing flags on attachment: 142239

Committed r117414: <http://trac.webkit.org/changeset/117414>
Comment 5 WebKit Review Bot 2012-05-17 01:21:13 PDT
All reviewed patches have been landed.  Closing bug.