From e9dcc3cfb985292b5ff96772b73029a32b18ff11 Mon Sep 17 00:00:00 2001 From: Colin Walters Date: Mon, 16 Mar 2009 22:25:19 +0000 Subject: Bug 575613 - Enum stripping with common prefix, also use "_" consistently Some enums have members which have a common prefix which doesn't match that of the enum name, but it also longer than the global namespace prefix. Instead, try stripping the common prefix first, and only if that fails fall back to the global strip. Also, for glib-registered enums we were using the nick, which typically has "-" as a separator. Replace that with "_" for consistency between unregistered enums and registered. utils.py:strip_common_prefix is now unused, delete. --- diff --git a/giscanner/glibtransformer.py b/giscanner/glibtransformer.py index 61d4cef..0f5b0c9 100644 --- a/giscanner/glibtransformer.py +++ b/giscanner/glibtransformer.py @@ -640,7 +640,10 @@ class GLibTransformer(object): def _introspect_enum(self, node): members = [] for member in node.findall('member'): - members.append(GLibEnumMember(member.attrib['nick'], + # Keep the name closer to what we'd take from C by default; + # see http://bugzilla.gnome.org/show_bug.cgi?id=575613 + name = member.attrib['nick'].replace('-', '_') + members.append(GLibEnumMember(name, member.attrib['value'], member.attrib['name'], member.attrib['nick'])) diff --git a/giscanner/transformer.py b/giscanner/transformer.py index 9aebbd8..2746b6e 100644 --- a/giscanner/transformer.py +++ b/giscanner/transformer.py @@ -37,7 +37,7 @@ from .sourcescanner import ( CSYMBOL_TYPE_ENUM, CSYMBOL_TYPE_UNION, CSYMBOL_TYPE_OBJECT, CSYMBOL_TYPE_MEMBER, CSYMBOL_TYPE_ELLIPSIS, CSYMBOL_TYPE_CONST, TYPE_QUALIFIER_CONST) -from .utils import strip_common_prefix, to_underscores +from .utils import to_underscores _xdg_data_dirs = [x for x in os.environ.get('XDG_DATA_DIRS', '').split(':') \ + [DATADIR, '/usr/share'] if x] @@ -208,11 +208,46 @@ class Transformer(object): raise NotImplementedError( 'Transformer: unhandled symbol: %r' % (symbol, )) + def _enum_common_prefix(self, symbol): + def common_prefix(a, b): + alen = len(a) + blen = len(b) + l = min(alen, blen) + for i in xrange(l): + if a[i] != b[i]: + return a[:i] + if alen > blen: + return b + return a + # Nothing less than 2 has a common prefix + if len(list(symbol.base_type.child_list)) < 2: + return None + prefix = None + for child in symbol.base_type.child_list: + if prefix is None: + prefix = child.ident + else: + prefix = common_prefix(prefix, child.ident) + if prefix == '': + return None + return prefix + def _create_enum(self, symbol): + prefix = self._enum_common_prefix(symbol) + if prefix: + prefixlen = len(prefix) + else: + prefixlen = 0 members = [] for child in symbol.base_type.child_list: - name = strip_common_prefix(symbol.ident, child.ident).lower() - members.append(Member(name, + if prefixlen > 0: + name = child.ident[prefixlen:] + else: + # Ok, the enum members don't have a consistent prefix + # among them, so let's just remove the global namespace + # prefix. + name = self.remove_prefix(child.ident) + members.append(Member(name.lower(), child.const_int, child.ident)) diff --git a/giscanner/utils.py b/giscanner/utils.py index 021ffea..d2752e7 100644 --- a/giscanner/utils.py +++ b/giscanner/utils.py @@ -57,14 +57,3 @@ def extract_libtool(libname): # and pre-2.2. Johan 2008-10-21 libname = libname.replace('.libs/.libs', '.libs') return libname - - -def strip_common_prefix(first, second): - max_index = second.rfind('_') - second_len = len(second) - second = second.replace('_', '') - max_index -= second_len - len(second) - 1 - for i, c in enumerate(first.upper()): - if i >= len(second) or c != second[i] or i == max_index: - return second[i:] - return second[i + 1:] diff --git a/tests/scanner/foo-1.0-expected.gir b/tests/scanner/foo-1.0-expected.gir index 59efffd..b28d861 100644 --- a/tests/scanner/foo-1.0-expected.gir +++ b/tests/scanner/foo-1.0-expected.gir @@ -15,6 +15,11 @@ and/or use gtk-doc annotations. --> + + + + + + + + + + + + + + diff --git a/tests/scanner/foo-1.0-expected.tgir b/tests/scanner/foo-1.0-expected.tgir index e834876..bd94ed3 100644 --- a/tests/scanner/foo-1.0-expected.tgir +++ b/tests/scanner/foo-1.0-expected.tgir @@ -7,6 +7,9 @@ + + + @@ -398,6 +401,17 @@ + + + + + + + + + + + diff --git a/tests/scanner/foo.h b/tests/scanner/foo.h index e9d00dd..8097cfc 100644 --- a/tests/scanner/foo.h +++ b/tests/scanner/foo.h @@ -341,6 +341,24 @@ GType foo_error_get_type (void); GQuark foo_error_quark (void); +typedef enum +{ + FOO_LAYER_DESKTOP = 0, + FOO_LAYER_BOTTOM = 1, + FOO_LAYER_NORMAL = 2, + FOO_LAYER_TOP = 4, /* Same as DOCK; see EWMH and bug 330717 */ + FOO_LAYER_DOCK = 4, + FOO_LAYER_FULLSCREEN = 5, + FOO_LAYER_FOCUSED_WINDOW = 6, + FOO_LAYER_OVERRIDE_REDIRECT = 7, + FOO_LAYER_LAST = 8 +} FooStackLayer; + +typedef enum +{ + FOO_SOME_SINGLE_ENUM +} FooASingle; + /* Should be skipped */ void foo_some_variant (guint x, va_list args); void foo_some_variant_ptr (guint x, va_list *args); -- cgit v0.9.1