clair icon indicating copy to clipboard operation
clair copied to clipboard

qualified names of operator methods have additional spaces (sometimes)

Open jurgenvinju opened this issue 1 year ago • 2 comments

an operator like operator <<(Stream x) would become |cpp+method://bla/operator%20%20%3C%3C...| because there are accidentally two spaces between operator and <<, while at the resolution of a usage site the name would not contain the spaces. This way we miss links between declarations and usages of operators in the call graph and we have false negatives in the downstream analyses.

jurgenvinju avatar Feb 26 '24 15:02 jurgenvinju

Wouldn't it already be enough to remove all whitespaces from before ( in the file section of an URI since method names cannot contain spaces?

linuswagner avatar Feb 26 '24 15:02 linuswagner

yes that would definitely work, but then we would never observe any hidden other issues anymore. So I prefer a more "surgical" approach. I'll do exactly that, but only if the resolved method name is an operator.

jurgenvinju avatar Feb 26 '24 16:02 jurgenvinju