Bug 86610

Summary: [EFL] Do not check NULL pointer allocated by new
Product: WebKit Reporter: Grzegorz Czajkowski <g.czajkowski>
Component: WebKit EFLAssignee: Grzegorz Czajkowski <g.czajkowski>
Status: RESOLVED FIXED    
Severity: Minor CC: gyuyoung.kim, lucas.de.marchi, rakuco, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: Linux   
Attachments:
Description Flags
proposed patch none

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.