Bug 66542 - [EFL] Rename ewk_view_(single|tiled).c to ewk_view_(single|tiled).cpp
Summary: [EFL] Rename ewk_view_(single|tiled).c to ewk_view_(single|tiled).cpp
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit EFL (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Ryuan Choi
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-08-18 23:33 PDT by Ryuan Choi
Modified: 2011-09-04 21:17 PDT (History)
5 users (show)

See Also:


Attachments
Patch (60.32 KB, patch)
2011-08-19 00:01 PDT, Ryuan Choi
no flags Details | Formatted Diff | Diff
comment_changed (60.13 KB, patch)
2011-08-21 18:16 PDT, Ryuan Choi
no flags Details | Formatted Diff | Diff
rebased (59.56 KB, patch)
2011-09-04 18:14 PDT, Ryuan Choi
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Ryuan Choi 2011-08-18 23:33:59 PDT
It's discussed in https://lists.webkit.org/pipermail/webkit-efl/2011-August/thread.html

In short,

ewk_view_single.c and ewk_view_tiled.c should include ewk_private.h to remove warning,
but they can't because their extension are ".c" and ewk_private.h contains WebCore namespace.
Comment 1 Ryuan Choi 2011-08-19 00:01:01 PDT
Created attachment 104468 [details]
Patch
Comment 2 Leandro Pereira 2011-08-19 10:11:49 PDT
Comment on attachment 104468 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=104468&action=review

> Source/WebKit/efl/ChangeLog:9
> +        Rename c files to cpp and include ewk_private.h to remove warning in compile time.
> +        It's because these files require ewk_repaint_add which is in ewk_private.h.
> +        However these files can't include ewk_private.h because of some functions containing
> +        namespace.

Informal r+ -- I'd just change the above to something along the lines of "Use a C++ compiler for these files, so that they can include the C++-using ewk_private.h."
Comment 3 Ryuan Choi 2011-08-21 18:16:17 PDT
Created attachment 104635 [details]
comment_changed
Comment 4 Ryuan Choi 2011-09-04 18:14:11 PDT
Created attachment 106298 [details]
rebased
Comment 5 Daniel Bates 2011-09-04 20:09:37 PDT
Comment on attachment 106298 [details]
rebased

r=me
Comment 6 Ryuan Choi 2011-09-04 20:18:11 PDT
Comment on attachment 106298 [details]
rebased

Thank you.
Comment 7 WebKit Review Bot 2011-09-04 21:17:11 PDT
Comment on attachment 106298 [details]
rebased

Clearing flags on attachment: 106298

Committed r94509: <http://trac.webkit.org/changeset/94509>
Comment 8 WebKit Review Bot 2011-09-04 21:17:17 PDT
All reviewed patches have been landed.  Closing bug.