Skip to content

refs #13698 - Library: avoid impossible type lookups - #7373

Closed
firewave wants to merge 4 commits into
cppcheck-opensource:mainfrom
firewave:lib-lookup
Closed

refs #13698 - Library: avoid impossible type lookups#7373
firewave wants to merge 4 commits into
cppcheck-opensource:mainfrom
firewave:lib-lookup

Conversation

@firewave

Copy link
Copy Markdown
Collaborator

No description provided.

@firewavefirewave changed the title refs #13698 - Library: avoid some lookups of empty stringsrefs #13698 - Library: avoid impossible type lookupsMar 13, 2025
@firewave
firewave marked this pull request as draft March 13, 2025 08:49
@firewave

Copy link
Copy Markdown
CollaboratorAuthor

Okay - this appears to be quite a mess (surprise, surprise).

The function declaration stuff has lots of other cases. And I am also seeing function calls getting through which do not seem to be recognized as such.

@firewave
firewaveforce-pushed the lib-lookup branch 3 times, most recently from c212dfc to 4885b2fCompareMarch 13, 2025 09:12
@firewave

Copy link
Copy Markdown
CollaboratorAuthor

And I am also seeing function calls getting through which do not seem to be recognized as such.

I wonder if that is a chicken/egg issue. In the --dump output it is associated with a function but in the lookup it isn't. I guess because that relation has not yet been established at that point.

Maybe those type lookups need to be deferred?

@firewave

This comment was marked as resolved.

Comment threadlib/library.cpp Outdated
@sonarqubecloud

Copy link
Copy Markdown

@firewave

Copy link
Copy Markdown
CollaboratorAuthor

Closing as there is not much of value left here.

@firewave
firewave deleted the lib-lookup branch September 15, 2025 18:34
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@firewave@chrchr-github