Skip to content
This repository has been archived by the owner on Dec 14, 2021. It is now read-only.

Nodes for different symbols with the same name are merged #233

Open
mlangkabel opened this issue Oct 25, 2016 · 12 comments
Open

Nodes for different symbols with the same name are merged #233

mlangkabel opened this issue Oct 25, 2016 · 12 comments
Assignees
Labels

Comments

@mlangkabel
Copy link
Contributor

Use case: 2 projects, each defining its own main function, are indexed by a single Coati project. In this case the two main functions should have distinct nodes.

@mlangkabel mlangkabel added the bug label Oct 25, 2016
@mlangkabel mlangkabel self-assigned this Dec 13, 2016
@egraether egraether changed the title Coati merges nodes for different symbols with the same name Nodes for different symbols with the same name are merged Feb 5, 2018
@egraether
Copy link
Contributor

+1 by mail

1 similar comment
@egraether
Copy link
Contributor

+1 by mail

@egraether
Copy link
Contributor

  • 1 by mail

@egraether
Copy link
Contributor

Added intermediate fix for multiple main() definitions in C and C++ with v2019.2.1, since this is the most reported issue regarding this bug.

@DavidAntliff
Copy link

Added intermediate fix for multiple main() definitions in C and C++ with v2019.2.1

Is this fix present and active in 2019.4.102? I'm currently seeing four distinct mains and their references in a single project squished into a single instance.

@egraether
Copy link
Contributor

This seems to be broken, thanks for reporting!

@egraether
Copy link
Contributor

Fixed the main fix.

@DavidAntliff
Copy link

@egraether thanks - I'd be happy to test this against my codebase if there's a build available, please? Do you have nightly builds?

@egraether
Copy link
Contributor

@DavidAntliff Well, if you don't happen to be a macOS user, then you can grab a build from our almost released Sourcetrail 2020.1 major release here: https://github.com/CoatiSoftware/Sourcetrail/releases/tag/2020.1.117

@DavidAntliff
Copy link

@egraether great, will test with Ubuntu 18.04 - thank you.

@egraether
Copy link
Contributor

@DavidAntliff You need to do a full index refresh. The fix affects the way symbols are stored.

@DavidAntliff
Copy link

@egraether that's much better - seems to be working correctly now (in terms of this issue anyway). Thank you.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

3 participants