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)

defect
Not set
normal

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.
Assignee: nobody → twointofive
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
Product: Webtools → Webtools Graveyard
You need to log in before you can comment on or make changes to this bug.