Skip to content

Commit d1f7161

Browse files
committed
fix some pmd warnings
1 parent cd22842 commit d1f7161

File tree

1 file changed

+12
-18
lines changed

1 file changed

+12
-18
lines changed

src/main/java/org/htmlunit/css/CssStyleSheet.java

Lines changed: 12 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -687,9 +687,8 @@ static boolean selects(final BrowserVersion browserVersion,
687687
return true;
688688

689689
case IS_PSEUDO_CLASS_CONDITION:
690-
final IsPseudoClassCondition isPseudoCondition = (IsPseudoClassCondition) condition;
691-
final SelectorList isSelectorList = isPseudoCondition.getSelectors();
692-
for (final Selector selector : isSelectorList) {
690+
final IsPseudoClassCondition conditionIsPseudo = (IsPseudoClassCondition) condition;
691+
for (final Selector selector : conditionIsPseudo.getSelectors()) {
693692
if (selects(browserVersion, selector, element, null, fromQuerySelectorAll, throwOnSyntax)) {
694693
return true;
695694
}
@@ -698,19 +697,17 @@ static boolean selects(final BrowserVersion browserVersion,
698697

699698
case WHERE_PSEUDO_CLASS_CONDITION:
700699
// same as is
701-
final WherePseudoClassCondition wherePseudoCondition = (WherePseudoClassCondition) condition;
702-
final SelectorList whereSelectorList = wherePseudoCondition.getSelectors();
703-
for (final Selector selector : whereSelectorList) {
700+
final WherePseudoClassCondition conditionWherePseudo = (WherePseudoClassCondition) condition;
701+
for (final Selector selector : conditionWherePseudo.getSelectors()) {
704702
if (selects(browserVersion, selector, element, null, fromQuerySelectorAll, throwOnSyntax)) {
705703
return true;
706704
}
707705
}
708706
return false;
709707

710708
case HAS_PSEUDO_CLASS_CONDITION:
711-
final HasPseudoClassCondition hasPseudoCondition = (HasPseudoClassCondition) condition;
712-
final SelectorList hasSelectorList = hasPseudoCondition.getSelectors();
713-
for (final Selector selector : hasSelectorList) {
709+
final HasPseudoClassCondition conditionHasPseudo = (HasPseudoClassCondition) condition;
710+
for (final Selector selector : conditionHasPseudo.getSelectors()) {
714711
if (selects(browserVersion, selector, element, null, fromQuerySelectorAll, throwOnSyntax)) {
715712
return true;
716713
}
@@ -1215,27 +1212,24 @@ private static boolean isValidCondition(final Condition condition, final DomNode
12151212
}
12161213
return true;
12171214
case IS_PSEUDO_CLASS_CONDITION:
1218-
final IsPseudoClassCondition isPseudoCondition = (IsPseudoClassCondition) condition;
1219-
final SelectorList isSelectorList = isPseudoCondition.getSelectors();
1220-
for (final Selector selector : isSelectorList) {
1215+
final IsPseudoClassCondition conditionIsPseudo = (IsPseudoClassCondition) condition;
1216+
for (final Selector selector : conditionIsPseudo.getSelectors()) {
12211217
if (!isValidSelector(selector, domNode)) {
12221218
return false;
12231219
}
12241220
}
12251221
return true;
12261222
case WHERE_PSEUDO_CLASS_CONDITION:
1227-
final WherePseudoClassCondition wherePseudoCondition = (WherePseudoClassCondition) condition;
1228-
final SelectorList whereSelectorList = wherePseudoCondition.getSelectors();
1229-
for (final Selector selector : whereSelectorList) {
1223+
final WherePseudoClassCondition conditionWherePseudo = (WherePseudoClassCondition) condition;
1224+
for (final Selector selector : conditionWherePseudo.getSelectors()) {
12301225
if (!isValidSelector(selector, domNode)) {
12311226
return false;
12321227
}
12331228
}
12341229
return true;
12351230
case HAS_PSEUDO_CLASS_CONDITION:
1236-
final HasPseudoClassCondition hasPseudoCondition = (HasPseudoClassCondition) condition;
1237-
final SelectorList hasSelectorList = hasPseudoCondition.getSelectors();
1238-
for (final Selector selector : hasSelectorList) {
1231+
final HasPseudoClassCondition conditionHasPseudo = (HasPseudoClassCondition) condition;
1232+
for (final Selector selector : conditionHasPseudo.getSelectors()) {
12391233
if (!isValidSelector(selector, domNode)) {
12401234
return false;
12411235
}

0 commit comments

Comments
 (0)