diff options
author | Rico Tzschichholz <ricotz@ubuntu.com> | 2021-03-16 18:09:51 +0100 |
---|---|---|
committer | Rico Tzschichholz <ricotz@ubuntu.com> | 2021-03-16 18:09:51 +0100 |
commit | 99ffbdfdafedb575ec8302872d34121b24b7a047 (patch) | |
tree | 146bbf4fc60e6fed56aeb0e6f654b71608b74129 /vala/valausedattr.vala | |
parent | 3c474cb0f70db5042763889ed22df412f6f1548d (diff) | |
download | vala-99ffbdfdafedb575ec8302872d34121b24b7a047.tar.gz |
vala: Check for unused attributes unconditionally
Diffstat (limited to 'vala/valausedattr.vala')
-rw-r--r-- | vala/valausedattr.vala | 19 |
1 files changed, 8 insertions, 11 deletions
diff --git a/vala/valausedattr.vala b/vala/valausedattr.vala index b9d8ceb22..a2102acf4 100644 --- a/vala/valausedattr.vala +++ b/vala/valausedattr.vala @@ -132,17 +132,14 @@ public class Vala.UsedAttr : CodeVisitor { } void check_unused_attr (Symbol sym) { - // optimize by not looking at all the symbols - if (sym.used) { - foreach (unowned Attribute attr in sym.attributes) { - var set = marked.get (attr.name); - if (set == null) { - Report.warning (attr.source_reference, "Attribute `%s' never used", attr.name); - } else { - foreach (var arg in attr.args.get_keys()) { - if (!set.contains (arg)) { - Report.warning (attr.source_reference, "Argument `%s' never used", arg); - } + foreach (unowned Attribute attr in sym.attributes) { + var set = marked.get (attr.name); + if (set == null) { + Report.warning (attr.source_reference, "Attribute `%s' never used", attr.name); + } else { + foreach (var arg in attr.args.get_keys()) { + if (!set.contains (arg)) { + Report.warning (attr.source_reference, "Argument `%s' never used", arg); } } } |