Skip to content

Commit 173404a

Browse files
committed
Issue #451: add EmptyBlock to checkstyle config
1 parent c088650 commit 173404a

2 files changed

Lines changed: 22 additions & 16 deletions

File tree

config/checkstyle-checks.xml

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -300,6 +300,24 @@
300300
<module name="AvoidNestedBlocks">
301301
<property name="allowInSwitchCase" value="true"/>
302302
</module>
303+
<module name="EmptyBlock">
304+
<property name="tokens" value="LITERAL_CATCH"/>
305+
<property name="tokens" value="ARRAY_INIT"/>
306+
<property name="tokens" value="LITERAL_DEFAULT"/>
307+
<property name="tokens" value="LITERAL_CASE"/>
308+
<property name="tokens" value="INSTANCE_INIT"/>
309+
<property name="tokens" value="LITERAL_DO"/>
310+
<property name="tokens" value="LITERAL_ELSE"/>
311+
<property name="tokens" value="LITERAL_FINALLY"/>
312+
<property name="tokens" value="LITERAL_FOR"/>
313+
<property name="tokens" value="LITERAL_IF"/>
314+
<property name="tokens" value="LITERAL_SWITCH"/>
315+
<property name="tokens" value="LITERAL_SYNCHRONIZED"/>
316+
<property name="tokens" value="LITERAL_TRY"/>
317+
<property name="tokens" value="LITERAL_WHILE"/>
318+
<property name="tokens" value="STATIC_INIT"/>
319+
<property name="option" value="text"/>
320+
</module>
303321
<module name="EmptyCatchBlock"/>
304322
<module name="LeftCurly"/>
305323
<module name="NeedBraces"/>

net.sf.eclipsecs.core/src/net/sf/eclipsecs/core/transformer/CheckstyleSetting.java

Lines changed: 4 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -97,14 +97,8 @@ public void addSetting(final CheckstyleSetting setting) {
9797
String property;
9898
while (propit.hasNext()) {
9999
property = propit.next();
100-
if (mCheckerModules.get(module).containsKey(property)) {
101-
if (!mCheckerModules.get(module).get(property)
102-
.equals(setting.getmCheckerModules().get(module).get(property))) {
103-
}
104-
} else {
105-
mCheckerModules.get(module).put(property,
106-
setting.getmCheckerModules().get(module).get(property));
107-
}
100+
mCheckerModules.get(module)
101+
.putIfAbsent(property, setting.getmCheckerModules().get(module).get(property));
108102
}
109103
} else {
110104
mCheckerModules.put(module, setting.getmCheckerModules().get(module));
@@ -121,14 +115,8 @@ public void addSetting(final CheckstyleSetting setting) {
121115
String property;
122116
while (propit.hasNext()) {
123117
property = propit.next();
124-
if (mTreeWalkerModules.get(module).containsKey(property)) {
125-
if (!mTreeWalkerModules.get(module).get(property)
126-
.equals(setting.getmTreeWalkerModules().get(module).get(property))) {
127-
}
128-
} else {
129-
mTreeWalkerModules.get(module).put(property,
130-
setting.getmTreeWalkerModules().get(module).get(property));
131-
}
118+
mTreeWalkerModules.get(module)
119+
.putIfAbsent(property, setting.getmTreeWalkerModules().get(module).get(property));
132120
}
133121
} else {
134122
mTreeWalkerModules.put(module, setting.getmTreeWalkerModules().get(module));

0 commit comments

Comments
 (0)