/[eiffelstudio]/branches/eth/eve/Src/framework/code_analysis/ca_code_analyzer.e
ViewVC logotype

Contents of /branches/eth/eve/Src/framework/code_analysis/ca_code_analyzer.e

Parent Directory Parent Directory | Revision Log Revision Log


Revision 93299 - (show annotations)
Sun Nov 10 21:37:47 2013 UTC (6 years ago) by zurfluhs
File size: 4403 byte(s)
Code Analysis: implemented 'feature never called' rule

(for trying out system-wide rules that do not depend on AST visits)
1 note
2 description: "Summary description for {CA_CODE_ANALYZER}."
3 author: "Stefan Zurfluh"
4 date: "$Date$"
5 revision: "$Revision$"
6
7 class
8 CA_CODE_ANALYZER
9
10 inherit {NONE}
11 SHARED_EIFFEL_PROJECT
12 CA_SHARED_NAMES
13
14 create
15 make
16
17 feature {NONE} -- Initialization
18
19 make
20 -- Initialization for `Current'.
21 do
22 create settings
23 create rules.make
24 -- Adding example rules
25 rules.extend (create {CA_SELF_ASSIGNMENT_RULE}.make)
26 rules.extend (create {CA_UNUSED_ARGUMENT_RULE}.make)
27 rules.extend (create {CA_NPATH_RULE}.make)
28 rules.extend (create {CA_EMPTY_IF_RULE}.make)
29 rules.extend (create {CA_FEATURE_NEVER_CALLED_RULE}.make)
30 -- Issues (contract violations, exceptions) with {EPA_CFG_BUILDER} and
31 -- classes from library 'program_analysis' in general.
32 -- rules.extend (create {CA_VARIABLE_NOT_READ_RULE}.make)
33
34 create classes_to_analyze.make
35 create rule_violations.make (100)
36 end
37
38 feature -- Analysis interface
39
40 analyze
41 local
42 l_rules_checker: CA_ALL_RULES_CHECKER
43 do
44 create l_rules_checker.make
45 across rules as l_rules loop
46 if l_rules.item.is_enabled then -- important: only add enabled rules
47 if system_wide_check or else (not l_rules.item.is_system_wide) then
48 -- do not add system wide rules if we check only parts of the system
49 if attached {CA_STANDARD_RULE} l_rules.item as l_std_rule then
50 l_std_rule.prepare_checking (l_rules_checker)
51 -- TODO: prepare rules of other types?
52 end
53 end
54 end
55 end
56
57 across classes_to_analyze as l_classes loop
58
59 -- TODO: more elegant and performant solution?
60 across rules as l_rules loop
61 l_rules.item.set_checking_class (l_classes.item)
62 -- If rule is non-standard then it will not be checked by l_rules_checker.
63 -- We will have the rule check the current class here:
64 if attached {CA_CFG_RULE} l_rules.item as l_cfg_rule then
65 l_cfg_rule.check_class (l_classes.item)
66 end
67 end
68
69 l_rules_checker.run_on_class (l_classes.item)
70
71 rule_violations.extend (create {SORTED_TWO_WAY_LIST[CA_RULE_VIOLATION]}.make, l_classes.item)
72 end
73
74 across rules as l_rules loop
75 across l_rules.item.violations as l_v loop
76 rule_violations.at (l_v.item.affected_class).extend (l_v.item)
77 end
78 end
79
80 clear_classes_to_analyze
81 ensure
82 violation_list_exists: analysis_successful implies rule_violations /= Void
83 end
84
85 clear_classes_to_analyze
86 do
87 classes_to_analyze.wipe_out
88 end
89
90 add_whole_system
91 local
92 l_groups: LIST [CONF_GROUP]
93 l_cluster: CLUSTER_I
94 do
95 from
96 l_groups := eiffel_universe.groups
97 l_groups.start
98 until
99 l_groups.after
100 loop
101 l_cluster ?= l_groups.item_for_iteration
102 -- Only load top-level clusters, as they are loaded recursively afterwards
103 if l_cluster /= Void and then l_cluster.parent_cluster = Void then
104 add_cluster (l_cluster)
105 end
106 l_groups.forth
107 end
108
109 system_wide_check := True
110 end
111
112 add_cluster (a_cluster: CLUSTER_I)
113 local
114 l_conf_class: CONF_CLASS
115 l_class_i: CLASS_I
116 do
117 system_wide_check := False
118
119 from
120 a_cluster.classes.start
121 until
122 a_cluster.classes.after
123 loop
124 l_conf_class := a_cluster.classes.item_for_iteration
125 l_class_i := eiffel_universe.class_named (l_conf_class.name, a_cluster)
126 add_class (l_class_i)
127 a_cluster.classes.forth
128 end
129 if a_cluster.sub_clusters /= Void then
130 from
131 a_cluster.sub_clusters.start
132 until
133 a_cluster.sub_clusters.after
134 loop
135 add_cluster (a_cluster.sub_clusters.item_for_iteration)
136 a_cluster.sub_clusters.forth
137 end
138 end
139 end
140
141 add_classes (a_classes: ITERABLE[CLASS_I])
142 do
143 system_wide_check := False
144
145 across a_classes as l_classes loop
146 add_class (l_classes.item)
147 end
148 end
149
150 add_class (a_class: CLASS_I)
151 local
152 l_class_c: CLASS_C
153 do
154 system_wide_check := False
155
156 if a_class.is_compiled then
157 l_class_c := a_class.compiled_class
158 check l_class_c /= Void end
159 classes_to_analyze.extend (l_class_c)
160 else
161 print ("Class " + a_class.name + " not compiled (skipped).%N")
162 end
163 end
164
165 feature -- Properties
166
167 analysis_successful: BOOLEAN
168
169 rules: LINKED_LIST[CA_RULE]
170
171 rule_violations: detachable HASH_TABLE[SORTED_TWO_WAY_LIST[CA_RULE_VIOLATION], CLASS_C]
172
173 feature {NONE} -- Implementation
174
175 settings: CA_SETTINGS
176
177 classes_to_analyze: LINKED_SET[CLASS_C]
178
179 system_wide_check: BOOLEAN
180
181
182 end

Properties

Name Value
svn:eol-style native
svn:keywords Author Date ID Revision

  ViewVC Help
Powered by ViewVC 1.1.23