Change in ovirt-engine[master]: core: Removed the LongCompat class
tnisan at redhat.com
tnisan at redhat.com
Tue Feb 5 11:16:46 UTC 2013
Tal Nisan has submitted this change and it was merged.
Change subject: core: Removed the LongCompat class
......................................................................
core: Removed the LongCompat class
Removed the LongCompat class and replaced all calls in Long.parse with
catching NumberFormatException, there are error handling anyway in each
of the references so it added no boilerplate code
Change-Id: If2b672f3406ce7b3316a5ff1b50379d92eedb062
Signed-off-by: Tal Nisan <tnisan at redhat.com>
---
D backend/manager/modules/compat/src/main/java/org/ovirt/engine/core/compat/LongCompat.java
M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/pm/VdsFenceOptions.java
M backend/manager/modules/utils/src/test/java/org/ovirt/engine/core/utils/RandomUtilsSeedingRule.java
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsBrokerObjectsBuilder.java
M frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/core/Compat.gwt.xml
5 files changed, 12 insertions(+), 34 deletions(-)
Approvals:
Tal Nisan: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/11666
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: If2b672f3406ce7b3316a5ff1b50379d92eedb062
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Alissa Bonas <abonas at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
More information about the Engine-commits
mailing list