Skip to content
  • Eric Blake's avatar
    46534309
    qapi: Forbid 'any' inside an alternate · 46534309
    Eric Blake authored
    
    
    The whole point of an alternate is to allow some type-safety while
    still accepting more than one JSON type.  Meanwhile, the 'any'
    type exists to bypass type-safety altogether.  The two are
    incompatible: you can't accept every type, and still tell which
    branch of the alternate to use for the parse; fix this to give a
    sane error instead of a Python stack trace.
    
    Note that other types that can't be alternate members are caught
    earlier, by check_type().
    
    Signed-off-by: default avatarEric Blake <eblake@redhat.com>
    Message-Id: <1455778109-6278-4-git-send-email-eblake@redhat.com>
    [Commit message tweaked]
    Signed-off-by: default avatarMarkus Armbruster <armbru@redhat.com>
    46534309
    qapi: Forbid 'any' inside an alternate
    Eric Blake authored
    
    
    The whole point of an alternate is to allow some type-safety while
    still accepting more than one JSON type.  Meanwhile, the 'any'
    type exists to bypass type-safety altogether.  The two are
    incompatible: you can't accept every type, and still tell which
    branch of the alternate to use for the parse; fix this to give a
    sane error instead of a Python stack trace.
    
    Note that other types that can't be alternate members are caught
    earlier, by check_type().
    
    Signed-off-by: default avatarEric Blake <eblake@redhat.com>
    Message-Id: <1455778109-6278-4-git-send-email-eblake@redhat.com>
    [Commit message tweaked]
    Signed-off-by: default avatarMarkus Armbruster <armbru@redhat.com>
Loading