After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 743043 - gstglapi.h not compiling with MSVC (Visual Studio 2013) - Part 2
gstglapi.h not compiling with MSVC (Visual Studio 2013) - Part 2
Status: RESOLVED DUPLICATE of bug 743041
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
1.4.5
Other Windows
: Normal normal
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2015-01-16 14:40 UTC by lasse
Modified: 2015-01-16 15:08 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description lasse 2015-01-16 14:40:22 UTC
In gstglapi.h, Line 128 is as follows:

  ret GSTGLAPI (*name) args;

which in MSVC in Vs2013 causes the following intellisense error:

	109	IntelliSense: a calling convention may not be followed by a nested declarator	d:\SDKS\gstreamer\1.0\x86\include\gstreamer-1.0\gst\gl\glprototypes\base.h	68

As far as I can tell the issue has to do with how the compiler likes its convention of __stdcall. The MSVC way is to extend the paraentheses to include the call itself, thusly:

  ret (GSTGLAPI *name) args;

Voila. Working in MSVC. Probably needs some more #define's around it to make sure it doesn't break something in other compilers.
Comment 1 Sebastian Dröge (slomo) 2015-01-16 15:04:45 UTC
Let's track that together with the other bug :)

*** This bug has been marked as a duplicate of bug 743041 ***
Comment 2 Sebastian Dröge (slomo) 2015-01-16 15:08:42 UTC
There should be a macro that evaluates to __stdcall on Windows and somethnig else elsewhere already. GST_EXPORT from gstconfig.h looks like a candidate