conf: Avoid extra scope when formatting 'smm' feature

Use an early break and remove the temporary variable.

Signed-off-by: Peter Krempa <pkrempa@redhat.com>
Reviewed-by: Ján Tomko <jtomko@redhat.com>

diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
index a1ed889e1d5403abcf0b1f4c46c3a7ae4020747f..b42c63fb4d5a5427f11a5bfe9af7224e05c44402 100644 (file)
--- a/src/conf/domain_conf.c
+++ b/src/conf/domain_conf.c
@@ -27814,27 +27814,26 @@ virDomainDefFormatFeatures(virBufferPtr buf,
             break;
 
         case VIR_DOMAIN_FEATURE_SMM:
-            if (def->features[i] != VIR_TRISTATE_SWITCH_ABSENT) {
-                virTristateSwitch state = def->features[i];
-
-                virBufferAsprintf(&tmpAttrBuf, " state='%s'",
-                                  virTristateSwitchTypeToString(state));
+            if (def->features[i] == VIR_TRISTATE_SWITCH_ABSENT)
+                break;
 
-                if (state == VIR_TRISTATE_SWITCH_ON &&
-                    def->tseg_specified) {
-                    const char *unit;
-                    unsigned long long short_size = virFormatIntPretty(def->tseg_size,
-                                                                       &unit);
+            virBufferAsprintf(&tmpAttrBuf, " state='%s'",
+                              virTristateSwitchTypeToString(def->features[i]));
 
-                    virBufferSetChildIndent(&tmpChildBuf, buf);
-                    virBufferAsprintf(&tmpChildBuf, "<tseg unit='%s'>%llu</tseg>\n",
-                                      unit, short_size);
-                }
+            if (def->features[i] == VIR_TRISTATE_SWITCH_ON &&
+                def->tseg_specified) {
+                const char *unit;
+                unsigned long long short_size = virFormatIntPretty(def->tseg_size,
+                                                                   &unit);
 
-                if (virXMLFormatElement(buf, "smm", &tmpAttrBuf, &tmpChildBuf) < 0)
-                    return -1;
+                virBufferSetChildIndent(&tmpChildBuf, buf);
+                virBufferAsprintf(&tmpChildBuf, "<tseg unit='%s'>%llu</tseg>\n",
+                                  unit, short_size);
             }
 
+            if (virXMLFormatElement(buf, "smm", &tmpAttrBuf, &tmpChildBuf) < 0)
+                return -1;
+
             break;
 
         case VIR_DOMAIN_FEATURE_APIC: