[Github-comments] Re: [geany/geany] items redundantly prefixed with the current filename in symbols panel (#2650)
Closed #2650 as completed via #3837. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/2650#event-12530469384 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] items redundantly prefixed with the current filename in symbols panel (#2650)
@fbrau Your suggestion above will remove the module prefix, but will just remove the module from being the parent of its content. See #3837 for an equivalent to my initial suggestion I failed to actually integrate despite your testing -- sorry about that. If really the information that a symbol is part of a module is bad *per se*, then your change makes sense, but should be carried over to universal-ctags. If it's semantically OK, but you really don't want to see it at all (unlike in my patch), then we'll need to add support for this, but I'd rather not :) -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/2650#issuecomment-2053783160 You are receiving this because you are subscribed to this thread. Message ID:
[Github-comments] Re: [geany/geany] items redundantly prefixed with the current filename in symbols panel (#2650)
the patch posted by @b4n is not compatible anymore.
For the record Im posting a new patch which works with geany 2
```
--- a/ctags/parsers/erlang.c
+++ b/ctags/parsers/erlang.c
@@ -112,7 +112,7 @@
{
vString *const identifier = vStringNew ();
parseIdentifier (cp, identifier);
- makeMemberTag (identifier, K_FUNCTION, module);
+ makeMemberTag (identifier, K_FUNCTION, NULL);
vStringDelete (identifier);
}
```
Im not an expert in geany codebase can someone verify Im not breaking something
with that patch?
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/2650#issuecomment-2049077269
You are receiving this because you are subscribed to this thread.
Message ID:
