On Fri, Feb 2, 2018 at 2:03 PM, Markus Armbruster <arm...@redhat.com> wrote: > Signed-off-by: Markus Armbruster <arm...@redhat.com>
It's not obvious the motivation behind this change (beside behind more elegant), but Reviewed-by: Marc-André Lureau <marcandre.lur...@redhat.com> > --- > scripts/qapi/common.py | 15 +++++++++------ > 1 file changed, 9 insertions(+), 6 deletions(-) > > diff --git a/scripts/qapi/common.py b/scripts/qapi/common.py > index d334e1db5a..7a327bfe9f 100644 > --- a/scripts/qapi/common.py > +++ b/scripts/qapi/common.py > @@ -286,8 +286,12 @@ class QAPISchemaParser(object): > if not isinstance(include, str): > raise QAPISemError(info, > "Value of 'include' must be a string") > - self._include(include, info, os.path.dirname(self.fname), > - previously_included) > + exprs_include = self._include(include, info, > + os.path.dirname(self.fname), > + previously_included) > + if exprs_include: > + self.exprs.extend(exprs_include.exprs) > + self.docs.extend(exprs_include.docs) > elif "pragma" in expr: > self.reject_expr_doc(cur_doc) > if len(expr) != 1: > @@ -330,14 +334,13 @@ class QAPISchemaParser(object): > > # skip multiple include of the same file > if incl_abs_fname in previously_included: > - return > + return None > + > try: > fobj = open(incl_fname, 'r') > except IOError as e: > raise QAPISemError(info, '%s: %s' % (e.strerror, incl_fname)) > - exprs_include = QAPISchemaParser(fobj, previously_included, info) > - self.exprs.extend(exprs_include.exprs) > - self.docs.extend(exprs_include.docs) > + return QAPISchemaParser(fobj, previously_included, info) > > def _pragma(self, name, value, info): > global doc_required, returns_whitelist, name_case_whitelist > -- > 2.13.6 >
