Closed
Bug 1268205
Opened 8 years ago
Closed 8 years ago
Normalize clang function qualnames to prevent missing results for 'callers:' queries
Categories
(Webtools Graveyard :: DXR, defect)
Webtools Graveyard
DXR
Tracking
(firefox49 affected)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
firefox49 | --- | affected |
People
(Reporter: twointofive, Assigned: twointofive)
References
Details
We have several open bugs of at least two types for which a 'callers:' query is missing some expected results. Type 1: the query misses results because some function refs for the function in question write certain qualname parameters as 'class mytype' (or 'enum myenum' or 'struct mystruct' etc.) while others write simply 'mytype'. See bug 1223671 for example. Type 2: some function ref qualnames include a namespace specifier (as in 'Maybe<gfx::IntSize>'), others skip the namespace (just 'Maybe<IntSize>'). See bug 1266577 for example. In both cases the issue is that in our clang plugin, we ask for function parameters (using 'getQualifiedName') at each function ref we see, but as it turns out clang doesn't always return the same string for each parameter at each ref, at least not the way we're currently asking for them.
Fix up for review at https://github.com/mozilla/dxr/pull/530
Fixed by https://github.com/mozilla/dxr/pull/530
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Updated•3 years ago
|
Product: Webtools → Webtools Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•