diff options
| author |   Marc Hartmayer <mhartmay@linux.vnet.ibm.com> | 2018-01-09 13:27:04 +0100 |
|---|---|---|
| committer |   Paolo Bonzini <pbonzini@redhat.com> | 2018-02-24 01:43:40 +0100 |
| commit | 369d5a85bb782ecf63c5bae9686c7e6104eea991 (patch) | |
| tree | 224a1ffa29e984ecb85dc213bdf6d4293c2bffaf /tools/kvm | |
| parent | tools/kvm_stat: use a more pythonic way to iterate over dictionaries (diff) | |
| download | linux-dev-369d5a85bb782ecf63c5bae9686c7e6104eea991.tar.xz linux-dev-369d5a85bb782ecf63c5bae9686c7e6104eea991.zip | |
tools/kvm_stat: avoid 'is' for equality checks
Use '==' for equality checks and 'is' when comparing identities.
An example where '==' and 'is' behave differently:
>>> a = 4242
>>> a == 4242
True
>>> a is 4242
False
Signed-off-by: Marc Hartmayer <mhartmay@linux.vnet.ibm.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Diffstat (limited to 'tools/kvm')
| -rwxr-xr-x | tools/kvm/kvm_stat/kvm_stat | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tools/kvm/kvm_stat/kvm_stat b/tools/kvm/kvm_stat/kvm_stat index f0da954a856c..e3f0becb6632 100755 --- a/tools/kvm/kvm_stat/kvm_stat +++ b/tools/kvm/kvm_stat/kvm_stat @@ -1085,7 +1085,7 @@ class Tui(object): stats = self.stats.get(self._display_guests) total = 0. for key, values in stats.items(): - if key.find('(') is -1: + if key.find('(') == -1: total += values.value if self._sorting == SORT_DEFAULT: @@ -1110,7 +1110,7 @@ class Tui(object): self.screen.addstr(row, 1, '%-40s %10d%7.1f %8s' % (key, values.value, values.value * 100 / total, cur)) - if cur is not '' and key.find('(') is -1: + if cur != '' and key.find('(') == -1: tavg += cur row += 1 if row == 3: |