qapi-visit.py: Clean up confusing push_indent() / pop_indent() use

Changing implicit indentation in the middle of generating a block
makes following the code being generated unnecessarily hard.

Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Luiz Capitulino <lcapitulino@redhat.com>

scripts/qapi-visit.py

@@ -128,12 +128,14 @@ if (!err) {
 ''',
         name=full_name)
 
-    pop_indent()
     ret += mcgen('''
     /* Always call end_struct if start_struct succeeded.  */
     visit_end_struct(m, &err);
-    }
-    error_propagate(errp, err);
+}
+error_propagate(errp, err);
+''')
+    pop_indent()
+    ret += mcgen('''
 }
 ''')
     return ret
@@ -289,8 +291,6 @@ void visit_type_%(name)s(Visitor *m, %(name)s ** obj, const char *name, Error **
 ''',
                  name=name)
 
-
-    push_indent()
     push_indent()
     push_indent()
 
@@ -300,8 +300,6 @@ void visit_type_%(name)s(Visitor *m, %(name)s ** obj, const char *name, Error **
 ''',
             name=name)
 
-    pop_indent()
-
     if not discriminator:
         disc_key = "type"
     else:
@@ -343,19 +341,17 @@ void visit_type_%(name)s(Visitor *m, %(name)s ** obj, const char *name, Error **
         }
         error_propagate(errp, err);
         err = NULL;
-    }
 ''')
     pop_indent()
+    pop_indent()
+
     ret += mcgen('''
+            }
             /* Always call end_struct if start_struct succeeded.  */
             visit_end_struct(m, &err);
         }
         error_propagate(errp, err);
-}
-''')
-
-    pop_indent();
-    ret += mcgen('''
+    }
 }
 ''')