Allon Mureinik has posted comments on this change.
Change subject: core: Add no-argument constructor check
......................................................................
Patch Set 2:
(2 comments)
Looking at
build-tools-root/checkstyles/src/main/resources/checkstyle-suppressions.xml I
don't understand why I need both that and the annotation.
Wouldn't it be simpler (and less intrusive) to just list all the problematic
files there?
[last time we discussed annotations, I was under the impression that the
<surpress> concept was out the window for some reason]
....................................................
File
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/LocationInfo.java
Line 6: public LocationInfo(ConnectionMethod connectionMethod) {
Line 7: this.connectionMethod = connectionMethod;
Line 8: }
Line 9:
Line 10: public LocationInfo() {
why not disable the check here too?
Line 11: }
Line 12:
Line 13: public ConnectionMethod getConnectionMethod() {
Line 14: return connectionMethod;
....................................................
File
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/utils/ListUtils.java
Line 37: * Creates a filteres list with the elements evaluated to
<strong>true</strong>.
Line 38: *
Line 39: * @param <T>
Line 40: */
Line 41: @NoArgConstructorCheck.Disable
Frankly, this class should be migrated to /dev/null, but not in scope for this
patch.
Line 42: public static class PredicateFilter<T> implements Filter<T> {
Line 43:
Line 44: public PredicateFilter(Predicate<T> predicate) {
Line 45: super();
--
To view, visit http://gerrit.ovirt.org/20274
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia4f9a00d6289374b433ed4419552420a3337da50
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Vojtech Szocs <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Daniel Erez <[email protected]>
Gerrit-Reviewer: Einav Cohen <[email protected]>
Gerrit-Reviewer: Vojtech Szocs <[email protected]>
Gerrit-Reviewer: Yair Zaslavsky <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches
