Summary: | [EFL] Remove duplicate code and remove entry after freeing the matrix. | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | JungJik Lee <jungjik.lee> | ||||||
Component: | WebKit EFL | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | gyuyoung.kim, gyuyoung.kim, hyuki.kim, lucas.de.marchi, rakuco, webkit.review.bot | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Other | ||||||||
OS: | Linux | ||||||||
Attachments: |
|
Description
JungJik Lee
2011-12-22 01:33:30 PST
Created attachment 120288 [details]
proposal patch
Proposal patch
Comment on attachment 120288 [details]
proposal patch
LGTM. But, please use C++ type casting instead of C casting. For example, static_cast<Eina_Inlist*>
(In reply to comment #2) > (From update of attachment 120288 [details]) > LGTM. But, please use C++ type casting instead of C casting. For example, static_cast<Eina_Inlist*> I've tried to change it to C++ style, but it can not be compiled. (In reply to comment #3) > (In reply to comment #2) > > (From update of attachment 120288 [details] [details]) > > LGTM. But, please use C++ type casting instead of C casting. For example, static_cast<Eina_Inlist*> > > I've tried to change it to C++ style, but it can not be compiled. I succeed to build using "reinterpret_cast<Eina_Inlist*>(entry)" Created attachment 120303 [details]
fix type cast
fix C++ style type cast.
(In reply to comment #5) > Created an attachment (id=120303) [details] > fix type cast > > fix C++ style type cast. reinterpret_cast works. good! Comment on attachment 120288 [details]
proposal patch
If you submit new patch, please set obsolete to previous patch.
Comment on attachment 120303 [details]
fix type cast
Informal r+ on my side.
LGTM too. This patch seems to fix below error message. ERR<32424>:Ewk_Tiled_Backing_Store /home/hyuki/webkit/WebKit/Source/WebKit/efl/ewk/ewk_tiled_matrix.cpp:116 Comment on attachment 120303 [details] fix type cast Clearing flags on attachment: 120303 Committed r103756: <http://trac.webkit.org/changeset/103756> All reviewed patches have been landed. Closing bug. |