Bug 276180
| Summary: | WebKitDOMEventTarget.h:95: Warning: WebKit2WebExtension: invalid "scope" annotation: only valid on callback parameters | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Michael Catanzaro <mcatanzaro> |
| Component: | WebKitGTK | Assignee: | Michael Catanzaro <mcatanzaro> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | bugs-noreply, mcatanzaro |
| Priority: | P2 | ||
| Version: | Other | ||
| Hardware: | PC | ||
| OS: | Linux | ||
Michael Catanzaro
Since https://gitlab.gnome.org/GNOME/gobject-introspection/-/merge_requests/477 it's no longer possible to build for GTK 3 because we have fatal introspection warnings enabled:
Source/WebKit/WebProcess/InjectedBundle/API/gtk/DOM/WebKitDOMEventTarget.h:95: Warning: WebKit2WebExtension: invalid "scope" annotation: only valid on callback parameters
Emmanuele recommends using (type gpointer) for the GCallback parameter of webkit_dom_event_target_remove_event_listener(), since it won't be called and is therefore not functioning as a callback.
Also:
"""
the `(scope async)` on add_event_listener() is also broken, because it's not an async function; but since the function is missing a GDestroyNotify to begin with, there's no way to properly annotate it
`(scope forever)` is the closest option, but that'll leak anything that tries to use it
"""
But I'll leave this broken because it's not clear whether it's fixable and this API is deprecated.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Michael Catanzaro
Pull request: https://github.com/WebKit/WebKit/pull/30446
EWS
Committed 280639@main (fff1b1773bff): <https://commits.webkit.org/280639@main>
Reviewed commits have been landed. Closing PR #30446 and removing active labels.