On 9/26/20 7:31 AM, Helio Loureiro wrote:
On Fri, Sep 25, 2020, 16:16 John Snow <js...@redhat.com
<mailto:js...@redhat.com>> wrote:
On 9/25/20 2:03 AM, Helio Loureiro wrote:
> Hi,
>
> I would replace the word variable "kind" by "category".
>
Hi, welcome to the list!
Tkz!
For patch reviews, we try to reply in-line, below the original post.
I realized that later. It has been more than 20 years I don't use this
formating. But if I intend to join the pack, I need to follow the pack.
(For more information on the QAPI Schema Language that we are parsing
and validating here, see docs/devel/qapi-code-gen.txt if you are
curious. Ultimately it is a JSON-like format that permits multiple
objects per document and allows comments. We use these structures to
generate types and command interfaces for our API protocol, QMP.)
Based on that I would suggest 'type_ref' instead to match the
definitions over there and since word 'type' itself is reserved.
One of the unsolvable problems in computer science is naming things:
"TYPE-REF" also has a specific meaning in QAPI, as it is the name of one
of the BNF grammar tokens we use.
So I might suggest (if "kind" is too ambiguous), that I might use
"statement_type" or "expression_type" if that helps clarify things.
--js