Summary: | [GTK] Move WebKit GType macros to WTF | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Carlos Garcia Campos <cgarcia> | ||||
Component: | WebKitGTK | Assignee: | Nobody <webkit-unassigned> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | benjamin, berto, bugs-noreply, buildbot, cdumez, cmarcelo, dbates, gustavo, mcatanzaro, zan | ||||
Priority: | P2 | Keywords: | Gtk | ||||
Version: | WebKit Nightly Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Carlos Garcia Campos
2017-06-12 01:58:17 PDT
Created attachment 312642 [details]
Patch
Thanks for the patch. If this patch contains new public API please make sure it follows the guidelines for new WebKit2 GTK+ API. See http://trac.webkit.org/wiki/WebKitGTK/AddingNewWebKit2API Attachment 312642 [details] did not pass style-queue:
ERROR: Source/WTF/wtf/glib/WebKitGType.h:74: g_define_type_id__volatile is incorrectly named. Don't use underscores in your identifier names. [readability/naming/underscores] [4]
ERROR: Source/WTF/wtf/glib/WebKitGType.h:76: g_define_type_id is incorrectly named. Don't use underscores in your identifier names. [readability/naming/underscores] [4]
Total errors found: 2 in 4 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 312642 [details]
Patch
It looks OK. WEBKIT prefixes seem a bit strange for code that's located in WTF though.
(In reply to Zan Dobersek from comment #4) > Comment on attachment 312642 [details] > Patch > > It looks OK. WEBKIT prefixes seem a bit strange for code that's located in > WTF though. Yes, they are currently helpers for glib based APIs that use WebKit prefix. They can be renamed to use WTF prefix once the JSC glib bindings are added, since we will not use WebKit prefix there. Committed r218096: <http://trac.webkit.org/changeset/218096> |