[webkit-reviews] review granted: [Bug 96638] [EFL] Remove a lot of C'ism from Ewk_Tiled_Matrix : [Attachment 164051] Patch

[bugzilla-daemon at webkit.org]

Gyuyoung Kim <gyuyoung.kim at samsung.com> has granted Christophe Dumez
<christophe.dumez at intel.com>'s request for review:
Bug 96638: [EFL] Remove a lot of C'ism from Ewk_Tiled_Matrix
https://bugs.webkit.org/show_bug.cgi?id=96638

Attachment 164051: Patch
https://bugs.webkit.org/attachment.cgi?id=164051&action=review

------- Additional Comments from Gyuyoung Kim <gyuyoung.kim at samsung.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=164051&action=review


Looks fine thanks.

> Source/WebKit/efl/ewk/ewk_tiled_matrix.cpp:399
> +#ifndef NDEBUG

Don't you need to check this routine in normal mode ?