Allon Mureinik has uploaded a new change for review. Change subject: core: TagsDirector control flow braces ......................................................................
core: TagsDirector control flow braces Added curly braces to control structures, as per proper Java conventions. Change-Id: I3d627091c2c8ef53f762f111f7c7efbdfbb300bb Signed-off-by: Allon Mureinik <[email protected]> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/TagsDirector.java 1 file changed, 6 insertions(+), 4 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/74/25674/1 diff --git a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/TagsDirector.java b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/TagsDirector.java index dfc5438..5609186 100644 --- a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/TagsDirector.java +++ b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/TagsDirector.java @@ -308,15 +308,17 @@ // (we prevent searching a regular expression match on them - // unnecessary). if (sb.length() == 0) { - if (indicator == TagReturnValueIndicator.ID) + if (indicator == TagReturnValueIndicator.ID) { sb.append(child.getTagIdAndChildrenIds()); - else + } else { sb.append(child.getTagNameAndChildrenNames()); + } } else { - if (indicator == TagReturnValueIndicator.ID) + if (indicator == TagReturnValueIndicator.ID) { sb.append(String.format(",%1$s", child.getTagIdAndChildrenIds())); - else + } else { sb.append(String.format(",%1$s", child.getTagNameAndChildrenNames())); + } } } else { RecursiveGetTagsAndChildrenByRegExp(tagNameRegExp, sb, child, indicator); -- To view, visit http://gerrit.ovirt.org/25674 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I3d627091c2c8ef53f762f111f7c7efbdfbb300bb Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
