Re: [PATCH v3 09/16] qapi/expr.py: Consolidate check_if_str calls in check_if

2021-03-01 Thread Markus Armbruster
John Snow writes: > On 2/25/21 9:23 AM, Markus Armbruster wrote: >> John Snow writes: >> >>> This is a small rewrite to address some minor style nits. >>> >>> Don't compare against the empty list to check for the empty condition, and >>> move the normalization forward to unify the check on the

Re: [PATCH v3 09/16] qapi/expr.py: Consolidate check_if_str calls in check_if

2021-02-25 Thread John Snow
On 2/25/21 9:23 AM, Markus Armbruster wrote: John Snow writes: This is a small rewrite to address some minor style nits. Don't compare against the empty list to check for the empty condition, and move the normalization forward to unify the check on the now-normalized structure. With the chec

Re: [PATCH v3 09/16] qapi/expr.py: Consolidate check_if_str calls in check_if

2021-02-25 Thread Markus Armbruster
John Snow writes: > This is a small rewrite to address some minor style nits. > > Don't compare against the empty list to check for the empty condition, and > move the normalization forward to unify the check on the now-normalized > structure. > > With the check unified, the local nested function

[PATCH v3 09/16] qapi/expr.py: Consolidate check_if_str calls in check_if

2021-02-22 Thread John Snow
This is a small rewrite to address some minor style nits. Don't compare against the empty list to check for the empty condition, and move the normalization forward to unify the check on the now-normalized structure. With the check unified, the local nested function isn't needed anymore and can be