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 678794 - g-ir-scanner doesn’t like inline asm in system headers
g-ir-scanner doesn’t like inline asm in system headers
Status: RESOLVED FIXED
Product: gobject-introspection
Classification: Platform
Component: general
1.36.x
Other Linux
: Normal trivial
: ---
Assigned To: gobject-introspection Maintainer(s)
gobject-introspection Maintainer(s)
: 638220 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2012-06-25 14:49 UTC by Thorsten Glaser
Modified: 2015-02-07 16:58 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
scannerlexer.l: Parse and ignore more __asm and __volatile (1.60 KB, patch)
2013-04-20 11:45 UTC, Patrick Welche
accepted-commit_now Details | Review

Description Thorsten Glaser 2012-06-25 14:49:15 UTC
While compiling software that’s part of the GNOME stack:

/usr/include/m68k-linux-gnu/bits/byteswap.h:50: syntax error, unexpected identifier, expecting ',' or ';' in '  __asm__ __volatile__ ("ror%.w %#8, %0;"' at '_'

This is g-ir-scanner from gobject-introspection 1.32.1-1 on Debian/m68k.
It doesn’t appear to have any negative outcome, but I thought I’d better report it.
Comment 1 Colin Walters 2012-10-26 17:21:51 UTC
You may be able to prefix it with __extension__ .  I believe some of the other headers use that.
Comment 2 Patrick Welche 2013-04-20 11:45:22 UTC
Created attachment 241984 [details] [review]
scannerlexer.l: Parse and ignore more __asm and __volatile

In the spirit of c9708af2 also parse and ignore
    
    __asm    [whitespace] volatile
    __asm__  [whitespace] volatile
    __volatile
    __volatile__
    
Ignoring __volatile is cosmetic, but the __asm cases avoid an unwanted VOLATILE token.

https://git.gnome.org/browse/gobject-introspection/commit/?id=c9708af2d5b0bed20cf88e3ec62365a1899ceb2f
Comment 3 Colin Walters 2013-04-20 22:12:33 UTC
Review of attachment 241984 [details] [review]:

Looks reasonable, thanks.
Comment 4 Emmanuele Bassi (:ebassi) 2013-08-26 10:32:54 UTC
*** Bug 638220 has been marked as a duplicate of this bug. ***
Comment 5 André Klapper 2015-02-07 16:58:01 UTC
[Mass-moving gobject-introspection tickets to its own Bugzilla product - see bug 708029. Mass-filter your bugmail for this message: introspection20150207 ]