[pulseaudio-tickets] [Bug 99199] Tiny correction to the .vapi

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=99199

--- Comment #3 from Corentin Noël <corentin at elementary.io> ---
(In reply to Tanu Kaskinen from comment #2)
> Thanks for the patch!
> 
> So "static" was dropped from the declarations. Can you explain why "static"
> shouldn't be used? Do you have any guesses for why "static" was used in the
> first place? I'm sure your patch is correct, but I'd like to understand the
> patches that I apply, and I'm not familiar with Vala syntax.

This syntax was allowed on previous Vala version but this is a misuse of the
"static" keyword. Only variables can be "static" and static means that the same
content of the variable is accessible to any instances of an object.

In Vala, const is the equivalent to preprocessor constant used in C, so here
the right way to declare them is to use const and not static, or here "static
const"

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/pulseaudio-bugs/attachments/20170105/2352be9b/attachment.html>