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

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

Parent Directory Parent Directory | Revision Log Revision Log


Revision 93299 - (hide 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 zurfluhs 93168 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 zurfluhs 93179 CA_SHARED_NAMES
13 zurfluhs 93168
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 zurfluhs 93216 rules.extend (create {CA_NPATH_RULE}.make)
28     rules.extend (create {CA_EMPTY_IF_RULE}.make)
29 zurfluhs 93299 rules.extend (create {CA_FEATURE_NEVER_CALLED_RULE}.make)
30 zurfluhs 93298 -- 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 zurfluhs 93173
34     create classes_to_analyze.make
35     create rule_violations.make (100)
36 zurfluhs 93168 end
37    
38 zurfluhs 93173 feature -- Analysis interface
39 zurfluhs 93168
40 zurfluhs 93173 analyze
41 zurfluhs 93168 local
42 zurfluhs 93173 l_rules_checker: CA_ALL_RULES_CHECKER
43 zurfluhs 93168 do
44 zurfluhs 93173 create l_rules_checker.make
45 zurfluhs 93168 across rules as l_rules loop
46     if l_rules.item.is_enabled then -- important: only add enabled rules
47 zurfluhs 93174 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 zurfluhs 93181 if attached {CA_STANDARD_RULE} l_rules.item as l_std_rule then
50     l_std_rule.prepare_checking (l_rules_checker)
51 zurfluhs 93298 -- TODO: prepare rules of other types?
52 zurfluhs 93181 end
53 zurfluhs 93174 end
54 zurfluhs 93168 end
55     end
56    
57 zurfluhs 93173 across classes_to_analyze as l_classes loop
58 zurfluhs 93177
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 zurfluhs 93181 -- 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 zurfluhs 93298 if attached {CA_CFG_RULE} l_rules.item as l_cfg_rule then
65     l_cfg_rule.check_class (l_classes.item)
66 zurfluhs 93181 end
67 zurfluhs 93177 end
68    
69 zurfluhs 93173 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 zurfluhs 93177 end
73 zurfluhs 93173
74 zurfluhs 93177 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 zurfluhs 93173 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 zurfluhs 93168 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 zurfluhs 93173 add_cluster (l_cluster)
105 zurfluhs 93168 end
106     l_groups.forth
107     end
108 zurfluhs 93174
109     system_wide_check := True
110 zurfluhs 93168 end
111    
112 zurfluhs 93173 add_cluster (a_cluster: CLUSTER_I)
113 zurfluhs 93168 local
114     l_conf_class: CONF_CLASS
115     l_class_i: CLASS_I
116     do
117 zurfluhs 93174 system_wide_check := False
118    
119 zurfluhs 93168 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 zurfluhs 93173 add_class (l_class_i)
127 zurfluhs 93168 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 zurfluhs 93173 add_cluster (a_cluster.sub_clusters.item_for_iteration)
136 zurfluhs 93168 a_cluster.sub_clusters.forth
137     end
138     end
139     end
140    
141 zurfluhs 93173 add_classes (a_classes: ITERABLE[CLASS_I])
142 zurfluhs 93168 do
143 zurfluhs 93174 system_wide_check := False
144    
145 zurfluhs 93173 across a_classes as l_classes loop
146     add_class (l_classes.item)
147     end
148 zurfluhs 93168 end
149    
150 zurfluhs 93173 add_class (a_class: CLASS_I)
151     local
152     l_class_c: CLASS_C
153 zurfluhs 93168 do
154 zurfluhs 93174 system_wide_check := False
155    
156 zurfluhs 93173 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 zurfluhs 93168 end
163     end
164    
165 zurfluhs 93173 feature -- Properties
166 zurfluhs 93168
167     analysis_successful: BOOLEAN
168    
169     rules: LINKED_LIST[CA_RULE]
170    
171 zurfluhs 93173 rule_violations: detachable HASH_TABLE[SORTED_TWO_WAY_LIST[CA_RULE_VIOLATION], CLASS_C]
172 zurfluhs 93168
173     feature {NONE} -- Implementation
174    
175     settings: CA_SETTINGS
176    
177 zurfluhs 93173 classes_to_analyze: LINKED_SET[CLASS_C]
178 zurfluhs 93168
179 zurfluhs 93174 system_wide_check: BOOLEAN
180    
181 zurfluhs 93298
182 zurfluhs 93168 end

Properties

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

  ViewVC Help
Powered by ViewVC 1.1.23