summaryrefslogtreecommitdiff
path: root/unusedcode.README
diff options
context:
space:
mode:
authorCaolán McNamara <caolanm@redhat.com>2012-04-04 11:53:34 +0100
committerCaolán McNamara <caolanm@redhat.com>2012-04-05 13:59:05 +0100
commitdd20a8c6e2ebeceff90567597d712e4c03c2ac7d (patch)
tree9085c266969602eac6f16dcb36984182d38434fd /unusedcode.README
parentfe630f3b097ca2bf173f21de77ed1535c767d0b9 (diff)
SvtFileDialogURLSelector::OpenURL can go apparently
Diffstat (limited to 'unusedcode.README')
-rw-r--r--unusedcode.README4
1 files changed, 3 insertions, 1 deletions
diff --git a/unusedcode.README b/unusedcode.README
index 08f51b7bc85d..c721d6237327 100644
--- a/unusedcode.README
+++ b/unusedcode.README
@@ -28,7 +28,9 @@ d) gcc will only emit code for inlines if those inlines are used, so
e) if a constructor is listed as unused, and it's the *only* ctor in the class,
then no object of that class can be construsted, so the whole thing is
unused, which can lead to a whole cascade of tricky but logical fallout.
-f) there's more actually unused code then what's listed. The idea is that what's
+f) if a destructor is listed as unused, and a constructor isn't, then there's
+ a leak somewhere, and the destructor most likely *should* be called.
+g) there's more actually unused code then what's listed. The idea is that what's
listed is definitely unused under the generation configuration, not that
it's a list of all unused code. If the count of unused easy hits 0 then
we can have a look at the non-easy and if that hits 0, then strip out