Re: [PATCH 05/12] qapi/gen: use './builtin' for the built-in module name

2020-12-16 Thread John Snow
On 12/16/20 3:35 AM, Markus Armbruster wrote: John Snow writes: Use this in preference to 'None', which helps remove some edge cases in the typing. Signed-off-by: John Snow Clearly better. Should've done it this way in commit c2e196a9b4 "qapi: Prepare for system modules other than 'builti

Re: [PATCH 05/12] qapi/gen: use './builtin' for the built-in module name

2020-12-16 Thread Markus Armbruster
John Snow writes: > Use this in preference to 'None', which helps remove some edge cases in > the typing. > > Signed-off-by: John Snow Clearly better. Should've done it this way in commit c2e196a9b4 "qapi: Prepare for system modules other than 'builtin'". > --- > scripts/qapi/gen.py | 27 +++

[PATCH 05/12] qapi/gen: use './builtin' for the built-in module name

2020-12-14 Thread John Snow
Use this in preference to 'None', which helps remove some edge cases in the typing. Signed-off-by: John Snow --- scripts/qapi/gen.py | 27 +-- 1 file changed, 13 insertions(+), 14 deletions(-) diff --git a/scripts/qapi/gen.py b/scripts/qapi/gen.py index a6dc991b1d03..0c5