1196917 - Filenames missing from jump-to-definition on macro-defined functions

Status: RESOLVED WONTFIX

(Webtools Graveyard :: DXR, defect)

Description

[Nathan Froyd [:froydnj]]

STR:

1. Go to DXR.
2. Search for RegisterNonJSSizeOfTab.
3. Observe three hits:
   - 1 call;
   - 1 declaration in an idl file;
   - 1 declaration in the generated header from the idl file.
4. Click on the generated header file result.
5. Click on "RegisterNonJSSizeOfTab", select "jump to definition"

Expected results:

DXR points me at or close to:

http://mxr.mozilla.org/mozilla-central/source/xpcom/base/nsMemoryReporterManager.cpp#2423

Actual results:

DXR points me at:

https://dxr.mozilla.org/mozilla-central/source/#2423

DXR has gotten the line number correct, but has somehow lost the filename along the way.

Alternate STR, starting after step 3 above:

4. Click on the result with the file featuring the call, nsWindowMemoryReporter.cpp.
5. Click on "RegisterNonJSSizeOfTab", select "jump to definition".

This takes me to the generated header file, rather than the actual definition in the .cpp.

Comment 1

[Erik Rose [:erik][:erikrose]]

Thanks for the report! I'm going to split this into 2 bugs: one for the missing filename and another for macros that define functions.

Comment 2

[Erik Rose [:erik][:erikrose]]

Split-off bug: 1197839

Comment 6

[Sylvestre Ledru [:Sylvestre]]

DXR is no longer available. Searchfox is now replacing it.
See meta bug 1669906 & https://groups.google.com/g/mozilla.dev.platform/c/jDRjrq3l-CY for more details.