451497 - Dehydra: Report (all) bad |ns*String| (and |nsCOMPtr|) usages

Status: RESOLVED WONTFIX

(Developer Infrastructure :: Source Code Analysis, enhancement)

Description

[Serge Gautherie (:sgautherie)]

(In addition to bug 451491,)
We briefly discussed some cases to add to the static analysis
<xpcom/analysis/stack.js> or others.

At a starting point,
I'm interested in bugs like bug 52352 (which I got patches landed then backed out due to ("triggered") leaks) and bug 450174,

IIRC, Mark wrote that
"nsCOMPtr & ns*String shouldn't be static because they don't get cleaned up right"

Unless there would be a way to fix them to cleanup right, they should be reported as bad usages.
There are probably other wrong patterns.

***

In addition to this, is there (already) a page which lists what is wrong and currently look for by this tool ?
(And possibly, suggestions to fix these cases.)

Comment 1

[Sylvestre Ledru [:Sylvestre]]

10 years without any activity. I think we can live without it.