tests/migration: remove the unused local variable

[PATCH] tests/migration: remove the unused local variable

dinglimin posted 1 patch 1 year, 7 months ago

Download mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20220928080555.2263-1-dinglimin@cmss.chinamobile.com

Maintainers: Juan Quintela <quintela@redhat.com>, "Dr. David Alan Gilbert" <dgilbert@redhat.com>

tests/migration/guestperf/engine.py | 1 -
1 file changed, 1 deletion(-)

Expand all Fold all

[PATCH] tests/migration: remove the unused local variable

Posted by dinglimin 1 year, 7 months ago

From: "dinglimin@cmss.chinamobile.com" <dinglimin@cmss.chinamobile.com>

Remove the unused local variable "records".

Signed-off-by: dinglimin <dinglimin@cmss.chinamobile.com>
---
 tests/migration/guestperf/engine.py | 1 -
 1 file changed, 1 deletion(-)

diff --git a/tests/migration/guestperf/engine.py b/tests/migration/guestperf/engine.py
index 87a6ab2009..59fca2c70b 100644
--- a/tests/migration/guestperf/engine.py
+++ b/tests/migration/guestperf/engine.py
@@ -65,7 +65,6 @@ def _vcpu_timing(self, pid, tid_list):
         return records
 
     def _cpu_timing(self, pid):
-        records = []
         now = time.time()
 
         jiffies_per_sec = os.sysconf(os.sysconf_names['SC_CLK_TCK'])
-- 
2.30.0.windows.2

Re: [PATCH] tests/migration: remove the unused local variable

Posted by Ján Tomko 1 year, 7 months ago

On a Wednesday in 2022, dinglimin wrote:
>From: "dinglimin@cmss.chinamobile.com" <dinglimin@cmss.chinamobile.com>
>
>Remove the unused local variable "records".
>
>Signed-off-by: dinglimin <dinglimin@cmss.chinamobile.com>
>---
> tests/migration/guestperf/engine.py | 1 -
> 1 file changed, 1 deletion(-)

Unused since its introduction in 409437e16df273fc5f78f6cd1cb53023eaeb9b72

Reviewed-by: Ján Tomko <jtomko@redhat.com>

Jano