Copilot commented on code in PR #13397:
URL: https://github.com/apache/cloudstack/pull/13397#discussion_r3394366795
##########
engine/schema/src/main/java/com/cloud/network/dao/FirewallRulesDaoImpl.java:
##########
@@ -47,8 +48,8 @@ public class FirewallRulesDaoImpl extends
GenericDaoBase<FirewallRuleVO, Long> i
protected final SearchBuilder<FirewallRuleVO> AllFieldsSearch;
protected final SearchBuilder<FirewallRuleVO> NotRevokedSearch;
protected final SearchBuilder<FirewallRuleVO> ReleaseSearch;
- protected SearchBuilder<FirewallRuleVO> VmSearch;
- protected SearchBuilder<FirewallRuleVO> FirewallByPortsAndNetwork;
+ protected final SearchBuilder<FirewallRuleVO> VmSearch =
createSearchBuilder();;
Review Comment:
There’s an extra trailing semicolon here (`...createSearchBuilder();;`).
While it compiles, it’s likely to trip style/checkstyle rules and should be
removed.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
