diff mbox series

[11/11] qapi2texi: De-duplicate code to add blank line before symbol

Message ID 20171002141341.24616-12-armbru@redhat.com
State New
Headers show
Series qapi: Cleanups around qapi2texi | expand

Commit Message

Markus Armbruster Oct. 2, 2017, 2:13 p.m. UTC
Signed-off-by: Markus Armbruster <armbru@redhat.com>
---
 scripts/qapi2texi.py | 12 ++----------
 1 file changed, 2 insertions(+), 10 deletions(-)

Comments

Marc-André Lureau Oct. 4, 2017, 10:35 a.m. UTC | #1
On Mon, Oct 2, 2017 at 4:13 PM, Markus Armbruster <armbru@redhat.com> wrote:
> Signed-off-by: Markus Armbruster <armbru@redhat.com>

Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com>


> ---
>  scripts/qapi2texi.py | 12 ++----------
>  1 file changed, 2 insertions(+), 10 deletions(-)
>
> diff --git a/scripts/qapi2texi.py b/scripts/qapi2texi.py
> index 58add26c11..92e2af2cd6 100755
> --- a/scripts/qapi2texi.py
> +++ b/scripts/qapi2texi.py
> @@ -206,8 +206,6 @@ class QAPISchemaGenDocVisitor(qapi.QAPISchemaVisitor):
>
>      def visit_enum_type(self, name, info, values, prefix):
>          doc = self.cur_doc
> -        if self.out:
> -            self.out += '\n'
>          self.out += TYPE_FMT(type='Enum',
>                               name=doc.symbol,
>                               body=texi_entity(doc, 'Values',
> @@ -217,16 +215,12 @@ class QAPISchemaGenDocVisitor(qapi.QAPISchemaVisitor):
>          doc = self.cur_doc
>          if base and base.is_implicit():
>              base = None
> -        if self.out:
> -            self.out += '\n'
>          self.out += TYPE_FMT(type='Object',
>                               name=doc.symbol,
>                               body=texi_entity(doc, 'Members', base, variants))
>
>      def visit_alternate_type(self, name, info, variants):
>          doc = self.cur_doc
> -        if self.out:
> -            self.out += '\n'
>          self.out += TYPE_FMT(type='Alternate',
>                               name=doc.symbol,
>                               body=texi_entity(doc, 'Members'))
> @@ -234,8 +228,6 @@ class QAPISchemaGenDocVisitor(qapi.QAPISchemaVisitor):
>      def visit_command(self, name, info, arg_type, ret_type,
>                        gen, success_response, boxed):
>          doc = self.cur_doc
> -        if self.out:
> -            self.out += '\n'
>          if boxed:
>              body = texi_body(doc)
>              body += ('\n@b{Arguments:} the members of @code{%s}\n'
> @@ -249,13 +241,13 @@ class QAPISchemaGenDocVisitor(qapi.QAPISchemaVisitor):
>
>      def visit_event(self, name, info, arg_type, boxed):
>          doc = self.cur_doc
> -        if self.out:
> -            self.out += '\n'
>          self.out += MSG_FMT(type='Event',
>                              name=doc.symbol,
>                              body=texi_entity(doc, 'Arguments'))
>
>      def symbol(self, doc, entity):
> +        if self.out:
> +            self.out += '\n'
>          self.cur_doc = doc
>          entity.visit(self)
>          self.cur_doc = None
> --
> 2.13.6
>
>
diff mbox series

Patch

diff --git a/scripts/qapi2texi.py b/scripts/qapi2texi.py
index 58add26c11..92e2af2cd6 100755
--- a/scripts/qapi2texi.py
+++ b/scripts/qapi2texi.py
@@ -206,8 +206,6 @@  class QAPISchemaGenDocVisitor(qapi.QAPISchemaVisitor):
 
     def visit_enum_type(self, name, info, values, prefix):
         doc = self.cur_doc
-        if self.out:
-            self.out += '\n'
         self.out += TYPE_FMT(type='Enum',
                              name=doc.symbol,
                              body=texi_entity(doc, 'Values',
@@ -217,16 +215,12 @@  class QAPISchemaGenDocVisitor(qapi.QAPISchemaVisitor):
         doc = self.cur_doc
         if base and base.is_implicit():
             base = None
-        if self.out:
-            self.out += '\n'
         self.out += TYPE_FMT(type='Object',
                              name=doc.symbol,
                              body=texi_entity(doc, 'Members', base, variants))
 
     def visit_alternate_type(self, name, info, variants):
         doc = self.cur_doc
-        if self.out:
-            self.out += '\n'
         self.out += TYPE_FMT(type='Alternate',
                              name=doc.symbol,
                              body=texi_entity(doc, 'Members'))
@@ -234,8 +228,6 @@  class QAPISchemaGenDocVisitor(qapi.QAPISchemaVisitor):
     def visit_command(self, name, info, arg_type, ret_type,
                       gen, success_response, boxed):
         doc = self.cur_doc
-        if self.out:
-            self.out += '\n'
         if boxed:
             body = texi_body(doc)
             body += ('\n@b{Arguments:} the members of @code{%s}\n'
@@ -249,13 +241,13 @@  class QAPISchemaGenDocVisitor(qapi.QAPISchemaVisitor):
 
     def visit_event(self, name, info, arg_type, boxed):
         doc = self.cur_doc
-        if self.out:
-            self.out += '\n'
         self.out += MSG_FMT(type='Event',
                             name=doc.symbol,
                             body=texi_entity(doc, 'Arguments'))
 
     def symbol(self, doc, entity):
+        if self.out:
+            self.out += '\n'
         self.cur_doc = doc
         entity.visit(self)
         self.cur_doc = None