Re: [PATCH v6 01/19] qapi: Replace List[str] with Sequence[str] for ifcond

2021-02-16 Thread Markus Armbruster
John Snow writes: > On 2/16/21 3:43 AM, Markus Armbruster wrote: >> John Snow writes: >> >>> It does happen to be a list (as of now), but we can describe it in more >>> general terms with no loss in accuracy to allow tuples and other >>> constructs. >>> >>> In the future, we can write "ifcond:

Re: [PATCH v6 01/19] qapi: Replace List[str] with Sequence[str] for ifcond

2021-02-16 Thread John Snow
On 2/16/21 3:43 AM, Markus Armbruster wrote: John Snow writes: It does happen to be a list (as of now), but we can describe it in more general terms with no loss in accuracy to allow tuples and other constructs. In the future, we can write "ifcond: Sequence[str] = ()" as a default parameter,

Re: [PATCH v6 01/19] qapi: Replace List[str] with Sequence[str] for ifcond

2021-02-16 Thread Markus Armbruster
John Snow writes: > It does happen to be a list (as of now), but we can describe it in more > general terms with no loss in accuracy to allow tuples and other > constructs. > > In the future, we can write "ifcond: Sequence[str] = ()" as a default > parameter, which we could not do with a Mutable

[PATCH v6 01/19] qapi: Replace List[str] with Sequence[str] for ifcond

2021-02-15 Thread John Snow
It does happen to be a list (as of now), but we can describe it in more general terms with no loss in accuracy to allow tuples and other constructs. In the future, we can write "ifcond: Sequence[str] = ()" as a default parameter, which we could not do with a Mutable type like a List.