Re: [PATCH v5 25/25] qapi: Dumb down QAPISchema.lookup_entity()

2024-03-19 Thread John Snow
On Fri, Mar 15, 2024, 11:23 AM Markus Armbruster wrote: > QAPISchema.lookup_entity() takes an optional type argument, a subtype > of QAPISchemaDefinition, and returns that type or None. Callers can > use this to save themselves an isinstance() test. > > The only remaining user of this

Re: [PATCH v5 25/25] qapi: Dumb down QAPISchema.lookup_entity()

2024-03-18 Thread Markus Armbruster
Markus Armbruster writes: > QAPISchema.lookup_entity() takes an optional type argument, a subtype > of QAPISchemaDefinition, and returns that type or None. Callers can > use this to save themselves an isinstance() test. > > The only remaining user of this convenience feature is .lookup_type().

[PATCH v5 25/25] qapi: Dumb down QAPISchema.lookup_entity()

2024-03-15 Thread Markus Armbruster
QAPISchema.lookup_entity() takes an optional type argument, a subtype of QAPISchemaDefinition, and returns that type or None. Callers can use this to save themselves an isinstance() test. The only remaining user of this convenience feature is .lookup_type(). But we don't actually save anything