<html>
<head>
<base href="https://bugs.webkit.org/" />
</head>
<body><span class="vcard"><a class="email" href="mailto:mcatanzaro@igalia.com" title="Michael Catanzaro <mcatanzaro@igalia.com>"> <span class="fn">Michael Catanzaro</span></a>
</span> changed
<a class="bz_bug_link
bz_status_NEW "
title="NEW - [GTK] Restore webkit_dom_dom_token_list_remove"
href="https://bugs.webkit.org/show_bug.cgi?id=158942">bug 158942</a>
<br>
<table border="1" cellspacing="0" cellpadding="8">
<tr>
<th>What</th>
<th>Removed</th>
<th>Added</th>
</tr>
<tr>
<td style="text-align:right;">CC</td>
<td>
</td>
<td>mcatanzaro@igalia.com
</td>
</tr></table>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - [GTK] Restore webkit_dom_dom_token_list_remove"
href="https://bugs.webkit.org/show_bug.cgi?id=158942#c5">Comment # 5</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - [GTK] Restore webkit_dom_dom_token_list_remove"
href="https://bugs.webkit.org/show_bug.cgi?id=158942">bug 158942</a>
from <span class="vcard"><a class="email" href="mailto:mcatanzaro@igalia.com" title="Michael Catanzaro <mcatanzaro@igalia.com>"> <span class="fn">Michael Catanzaro</span></a>
</span></b>
<pre>(In reply to <a href="show_bug.cgi?id=158942#c4">comment #4</a>)
<span class="quote">> I skipped variadic functions in <a class="bz_bug_link
bz_status_RESOLVED bz_closed"
title="RESOLVED FIXED - Improve code generator for functions with variadic parameters"
href="show_bug.cgi?id=158529">Bug 158529</a> for GObject bindings because I
> thought variadic parameters were not correctly handled.
>
> "return 1 if $param->isVariadic;" line from CodeGeneratorGObject.pm can be
> removed to restore webkit_dom_dom_token_list_remove and
> webkit_dom_dom_token_list_add functions. I can upload a patch if needed.
>
> However, the generated code seems to process only the first token passed in
> argument of the function.</span >
Yeah, the generated functions were not variadic. I guess you would have to call the function once for each argument to get the desired effect. The original declaration, present in 2.12, was this:
/**
* webkit_dom_dom_token_list_remove:
* @self: A #WebKitDOMDOMTokenList
* @tokens: A #gchar
* @error: #GError
*
* Stability: Unstable
**/
WEBKIT_API void
webkit_dom_dom_token_list_remove(WebKitDOMDOMTokenList* self, const gchar* tokens, GError** error);
The GError parameter disappeared in trunk a month or two ago. Not a problem, it's unstable API, after all.
Of course, it would be best to generate real variadic functions, but reverting to this form would suffice for Epiphany.
<span class="quote">> Which tests should be run to test such kinds of regression?</span >
We have GObject API break tests that would have caught this if it was stable API. We intentionally don't run the tests for the unstable API, because we don't guarantee this API will continue to exist. The problem is that we have two GNOME applications -- Epiphany and Yelp -- that are unfortunately using the unstable API, and Epiphany used this function in particular. The ideal solution would be for them to use only the stable API, but I'm not sure if it's possible.</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>