[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 11/37] qapi/common.py: Replace one-letter 'c' variable
From: |
Markus Armbruster |
Subject: |
Re: [PATCH 11/37] qapi/common.py: Replace one-letter 'c' variable |
Date: |
Thu, 17 Sep 2020 16:17:51 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.1 (gnu/linux) |
John Snow <jsnow@redhat.com> writes:
> Signed-off-by: John Snow <jsnow@redhat.com>
> ---
> scripts/qapi/common.py | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/scripts/qapi/common.py b/scripts/qapi/common.py
> index c665e67495..4c079755d3 100644
> --- a/scripts/qapi/common.py
> +++ b/scripts/qapi/common.py
> @@ -30,14 +30,14 @@ def camel_to_upper(value):
> new_name = ''
> length = len(c_fun_str)
> for i in range(length):
> - c = c_fun_str[i]
> - # When c is upper and no '_' appears before, do more checks
> - if c.isupper() and (i > 0) and c_fun_str[i - 1] != '_':
> + char = c_fun_str[i]
> + # When char is upper and no '_' appears before, do more checks
Good opportunity to tweak the language: "When char is upper case".
> + if char.isupper() and (i > 0) and c_fun_str[i - 1] != '_':
> if i < length - 1 and c_fun_str[i + 1].islower():
> new_name += '_'
> elif c_fun_str[i - 1].isdigit():
> new_name += '_'
> - new_name += c
> + new_name += char
> return new_name.lstrip('_').upper()
- Re: [PATCH 09/37] qapi/common.py: Add indent manager, (continued)
- [PATCH 12/37] qapi/common.py: check with pylint, John Snow, 2020/09/15
- [PATCH 11/37] qapi/common.py: Replace one-letter 'c' variable, John Snow, 2020/09/15
- Re: [PATCH 11/37] qapi/common.py: Replace one-letter 'c' variable,
Markus Armbruster <=
- [PATCH 14/37] qapi/common.py: Move comments into docstrings, John Snow, 2020/09/15
- Re: [PATCH 14/37] qapi/common.py: Move comments into docstrings, Markus Armbruster, 2020/09/17
- Re: [PATCH 14/37] qapi/common.py: Move comments into docstrings, John Snow, 2020/09/17
- Re: [PATCH 14/37] qapi/common.py: Move comments into docstrings, Eduardo Habkost, 2020/09/17
- Re: [PATCH 14/37] qapi/common.py: Move comments into docstrings, John Snow, 2020/09/17
- Re: [PATCH 14/37] qapi/common.py: Move comments into docstrings, Markus Armbruster, 2020/09/24
- Re: [PATCH 14/37] qapi/common.py: Move comments into docstrings, John Snow, 2020/09/24
- Re: [PATCH 14/37] qapi/common.py: Move comments into docstrings, Markus Armbruster, 2020/09/25
- Re: [PATCH 14/37] qapi/common.py: Move comments into docstrings, John Snow, 2020/09/25