Re: [PATCH v3 14/15] qapi/parser: allow 'ch' variable name

On 5/20/21 5:36 AM, Markus Armbruster wrote: Uh, one more little thing... this commit message isn't quite right: John Snow writes: We can have a two-letter variable name, as a treat. Signed-off-by: John Snow -- This line is not a separator. I don't want to use 'chr' or 'char', and in

Re: [PATCH v3 14/15] qapi/parser: allow 'ch' variable name

Uh, one more little thing... this commit message isn't quite right: John Snow writes: > We can have a two-letter variable name, as a treat. > > Signed-off-by: John Snow > > -- This line is not a separator. > > I don't want to use 'chr' or 'char', and in context 'ch' works well > enough. I wil

[PATCH v3 14/15] qapi/parser: allow 'ch' variable name

We can have a two-letter variable name, as a treat. Signed-off-by: John Snow -- I don't want to use 'chr' or 'char', and in context 'ch' works well enough. I will assume that any possible future uses will also be obvious enough. Signed-off-by: John Snow --- scripts/qapi/pylintrc | 1 + 1 fil