On Tue, Sep 22, 2020 at 05:13:09PM -0400, John Snow wrote: > Signed-off-by: John Snow <js...@redhat.com> > --- [...] > @@ -160,6 +244,18 @@ def check_type(value: Optional[object], > source: str, > allow_array: bool = False, > allow_dict: Union[bool, str] = False) -> None: > + """ > + Check the QAPI type of `value`. [RW] > + > + Python types of `str` or `None` are always allowed. > + > + :param value: The value to check. > + :param info: QAPI Source file information. > + :param source: Human readable string describing "what" the value is. > + :param allow_array: Allow a `List[str]` of length 1, > + which indicates an Array<T> type. > + :param allow_dict: Allow a dict, treated as an anonymous type.
I was hoping the docstring would explain what happens when allow_dict is a string. -- Eduardo