Summary: | [EFL] Use Evas function instead of memcpy | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Jaehun Lim <ljaehun.lim> | ||||||
Component: | WebKit EFL | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | gyuyoung.kim, lucas.de.marchi, rakuco, webkit.review.bot | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Linux | ||||||||
Attachments: |
|
Description
Jaehun Lim
2012-10-04 22:18:57 PDT
Created attachment 167247 [details]
Patch
Comment on attachment 167247 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=167247&action=review > Source/WebKit/efl/ChangeLog:10 > + Please mention to change variable name. Comment on attachment 167247 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=167247&action=review > Source/WebKit/efl/ewk/ewk_util.cpp:39 > + int width, height, stride; Let's move this to 66,67,68 line. > Source/WebKit/efl/ewk/ewk_util.cpp:41 > + void* data; Ditto. > Source/WebKit/efl/ewk/ewk_util.cpp:94 > + ERR("could not get source data."); > + return 0; Don't you need to delete image? Created attachment 167944 [details]
Patch
Comment on attachment 167944 [details]
Patch
Looks fine.
(In reply to comment #3) > (From update of attachment 167247 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=167247&action=review > > > Source/WebKit/efl/ewk/ewk_util.cpp:39 > > + int width, height, stride; > > Let's move this to 66,67,68 line. Done > > > Source/WebKit/efl/ewk/ewk_util.cpp:41 > > + void* data; > > Ditto. Done > > > Source/WebKit/efl/ewk/ewk_util.cpp:94 > > + ERR("could not get source data."); > > + return 0; > > Don't you need to delete image? I moved these lines before "image" was created. Comment on attachment 167944 [details] Patch Clearing flags on attachment: 167944 Committed r130875: <http://trac.webkit.org/changeset/130875> All reviewed patches have been landed. Closing bug. |