> I checked the `sysverilog.sv` test which had "package" test unchecked in the 
> top comment so I thought it wasn't mapped.

Oh, I see :)
My aim was to put those in both files so that it could be easily seen that the 
"overlap" of both covered all the captured (non-`undef`) cases.  Didn't work 
all that well apparently :smile: 

> I think I know what is going on - it doesn't work for types that are declared 
> _after_ their members

I think you may be onto something.  I've found that if I write `typedef struct 
packed struct_t;` one line above, then delete it, under some weird 
circumstances the original definition works!  Like I somehow tricked Geany into 
realizing that struct_t is a struct/typedef and now it remembers, and displays 
the members inside.  But it all breaks if I reopen the file, of course.

> At least for C/C++ we dealt with such situations already for typedefs and I 
> think it would work if you used `tm_tag_typedef_t` mapping

Well I got some bad news, I was already using `tm_tag_typedef_t` for typedefs 
just in case, but that didn't help :'( Maybe that only works in C.
I will try to look into the issue and at least submit an issue on ctags.

> In any case, nothing for this PR which I think is ready to be merged.

Woohoo!  Thanks :)

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/4039#issuecomment-2489511167
You are receiving this because you are subscribed to this thread.

Message ID: <geany/geany/pull/4039/[email protected]>

Reply via email to