Allon Mureinik has uploaded a new change for review.

Change subject: core: Remove unused LdapGroup constructors
......................................................................

core: Remove unused LdapGroup constructors

Inlined constructors that where only called by other constructors.

Change-Id: Ifcb148810260c25c376f070f72bea0aa37282c4a
Signed-off-by: Allon Mureinik <amure...@redhat.com>
---
M 
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/LdapGroup.java
1 file changed, 2 insertions(+), 10 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/30/18130/1

diff --git 
a/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/LdapGroup.java
 
b/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/LdapGroup.java
index 0a3515e..41a3a43 100644
--- 
a/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/LdapGroup.java
+++ 
b/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/LdapGroup.java
@@ -51,23 +51,15 @@
         status = value;
     }
 
-    public LdapGroup(Guid id, String name) {
+    public LdapGroup(Guid id, String name, String domain) {
         this.id = id;
         this.name = name;
-    }
-
-    public LdapGroup(Guid id, String name, String domain) {
-        this(id, name);
         this.domain = domain;
     }
 
-    public LdapGroup(Guid id, String name, String domain, String 
distinguishedName) {
+    public LdapGroup(Guid id, String name, String domain, String 
distinguishedName, List<String> memberOf) {
         this(id, name, domain);
         this.distinguishedName = distinguishedName;
-    }
-
-    public LdapGroup(Guid id, String name, String domain, String 
distinguishedName, List<String> memberOf) {
-        this(id, name, domain, distinguishedName);
         this.setMemberOf(memberOf);
     }
 


-- 
To view, visit http://gerrit.ovirt.org/18130
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ifcb148810260c25c376f070f72bea0aa37282c4a
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik <amure...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to