[webkit-efl] Use of pimpl idiom for Ewk classes

Dumez, Christophe christophe.dumez at intel.com
Thu Oct 18 09:27:54 PDT 2012


I have updated the Ewk_Download class based on the alternative mentioned by
Thiago and updated a patch here for comparison:
(pimpl patch is still at

One possible issue that I noticed with this approach is that we need to use
"const char*" instead of "String" type in our C++ methods API due to Eina
string sharing. This is a bit unfortunate since C++ code uses String type
in WebKit.

As I said in my previous email, we also need to write a lot more C++
methods than with the pimpl approach. It will be a lot more work to
refactor all Ewk classes.
This approach however has the benefit of being of simpler design (no
macros, C++ methods can access members directly without dereferencing a

I'm personally open to both approaches so I'd like to hear others' opinion.

Christophe Dumez
Linux Software Engineer, PhD
Intel Finland Oy - Open Source Technology Center
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-efl/attachments/20121018/c6a0f6d4/attachment.html>

More information about the webkit-efl mailing list