Bug 75073 - [EFL] Remove duplicate code and remove entry after freeing the matrix.
Summary: [EFL] Remove duplicate code and remove entry after freeing the matrix.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit EFL (show other bugs)
Version: 528+ (Nightly build)
Hardware: Other Linux
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-12-22 01:33 PST by JungJik Lee
Modified: 2011-12-27 21:14 PST (History)
6 users (show)

See Also:


Attachments
proposal patch (1.73 KB, patch)
2011-12-22 01:36 PST, JungJik Lee
no flags Details | Formatted Diff | Diff
fix type cast (1.75 KB, patch)
2011-12-22 03:32 PST, JungJik Lee
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description JungJik Lee 2011-12-22 01:33:30 PST
Remove duplicate ewk_tile_unused_cache_freeze call and reorder function call.
eina_matrixsparse_free(entry->matrix) should be called before removing the entry from inlist.
Because entry count is decreased after calling eina_matrixsparce_free in _ewk_tile_matrix_cell_free.
Comment 1 JungJik Lee 2011-12-22 01:36:25 PST
Created attachment 120288 [details]
proposal patch

Proposal patch
Comment 2 Gyuyoung Kim 2011-12-22 02:27:26 PST
Comment on attachment 120288 [details]
proposal patch

LGTM. But, please use C++ type casting instead of C casting. For example, static_cast<Eina_Inlist*>
Comment 3 JungJik Lee 2011-12-22 02:49:49 PST
(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.
Comment 4 Gyuyoung Kim 2011-12-22 03:21:36 PST
(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)"
Comment 5 JungJik Lee 2011-12-22 03:32:01 PST
Created attachment 120303 [details]
fix type cast

fix C++ style type cast.
Comment 6 JungJik Lee 2011-12-22 03:32:46 PST
(In reply to comment #5)
> Created an attachment (id=120303) [details]
> fix type cast
> 
> fix C++ style type cast.

reinterpret_cast works. good!
Comment 7 Gyuyoung Kim 2011-12-22 03:35:07 PST
Comment on attachment 120288 [details]
proposal patch

If you submit new patch, please set obsolete to previous patch.
Comment 8 Gyuyoung Kim 2011-12-22 19:25:36 PST
Comment on attachment 120303 [details]
fix type cast

Informal r+ on my side.
Comment 9 KwangHyuk 2011-12-26 18:00:14 PST
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 10 WebKit Review Bot 2011-12-27 21:14:12 PST
Comment on attachment 120303 [details]
fix type cast

Clearing flags on attachment: 120303

Committed r103756: <http://trac.webkit.org/changeset/103756>
Comment 11 WebKit Review Bot 2011-12-27 21:14:17 PST
All reviewed patches have been landed.  Closing bug.