diff options
| author | Markus Armbruster | 2021-03-23 10:40:09 +0100 |
|---|---|---|
| committer | Markus Armbruster | 2021-03-23 22:31:05 +0100 |
| commit | 00ffe242d64f7622965c52c62adb06fd9664ada8 (patch) | |
| tree | b6d076e25e9345bcffa1fe854ba9ad84de1e23f6 | |
| parent | qapi: Move uppercase rejection to check_name_lower() (diff) | |
| download | qemu-00ffe242d64f7622965c52c62adb06fd9664ada8.tar.gz qemu-00ffe242d64f7622965c52c62adb06fd9664ada8.tar.xz qemu-00ffe242d64f7622965c52c62adb06fd9664ada8.zip | |
qapi: Consistently permit any case in downstream prefixes
We require lowercase __RFQDN_ downstream prefixes only where we
require the prefixed name to be lowercase. Don't; permit any case in
__RFQDN_ prefixes anywhere.
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Message-Id: <20210323094025.3569441-13-armbru@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
| -rw-r--r-- | scripts/qapi/expr.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/scripts/qapi/expr.py b/scripts/qapi/expr.py index a815060ee2..b5fb0be48b 100644 --- a/scripts/qapi/expr.py +++ b/scripts/qapi/expr.py @@ -51,7 +51,7 @@ def check_name_upper(name, info, source): def check_name_lower(name, info, source, permit_upper=False): stem = check_name_str(name, info, source) - if not permit_upper and name.lower() != name: + if not permit_upper and re.search(r'[A-Z]', stem): raise QAPISemError( info, "%s uses uppercase in name" % source) # TODO reject '_' in stem |