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 622659 - Clean up annotation parsing, don't try to parse invalid annotations
Clean up annotation parsing, don't try to parse invalid annotations
Status: RESOLVED FIXED
Product: gobject-introspection
Classification: Platform
Component: general
unspecified
Other All
: Normal normal
: ---
Assigned To: gobject-introspection Maintainer(s)
gobject-introspection Maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2010-06-24 21:03 UTC by Colin Walters
Modified: 2015-02-07 16:48 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Clean up annotation parsing, don't try to parse invalid annotations (8.26 KB, patch)
2010-06-24 21:03 UTC, Colin Walters
none Details | Review
Clean up annotation parsing, don't try to parse invalid annotations (44.66 KB, patch)
2010-06-24 21:19 UTC, Colin Walters
committed Details | Review

Description Colin Walters 2010-06-24 21:03:11 UTC
The old parser tried to parse:
@foo: some text here (other text)
Reject this.  We strictly require another colon at the end to parse
the text in between as (option) (other option).
Futher ensure we only attempt to do option parse if we find something
that matches the strict regexp ([A-Za-z]+).  This could be tightened
further.
Comment 1 Colin Walters 2010-06-24 21:03:13 UTC
Created attachment 164551 [details] [review]
Clean up annotation parsing, don't try to parse invalid annotations
Comment 2 Colin Walters 2010-06-24 21:05:45 UTC
Note this patch has the scanner pick up a lot of documentation values it missed before, but there are still a few regressions.
Comment 3 Colin Walters 2010-06-24 21:19:45 UTC
Created attachment 164553 [details] [review]
Clean up annotation parsing, don't try to parse invalid annotations

The old parser tried to parse:
@foo: some text here (other text)
Reject this.  We strictly require another colon at the end to parse
the text in between as (option) (other option).
Futher ensure we only attempt to do option parse if we find something
that matches the strict regexp ([A-Za-z]+).  This could be tightened
further.

TODO: Have a warning for something that looks like an annotation,
but isn't.
Comment 4 Colin Walters 2010-06-25 13:19:41 UTC
Attachment 164553 [details] pushed as 0a67406 - Clean up annotation parsing, don't try to parse invalid annotations
Comment 5 André Klapper 2015-02-07 16:48:55 UTC
[Mass-moving gobject-introspection tickets to its own Bugzilla product - see bug 708029. Mass-filter your bugmail for this message: introspection20150207 ]