diff options
| author | Martin Basti <mbasti@redhat.com> | 2016-09-26 14:08:17 +0200 |
|---|---|---|
| committer | Martin Basti <mbasti@redhat.com> | 2016-09-27 13:35:58 +0200 |
| commit | 0f88f8fe889ae4801fc8d5ece1ad51c5246718ac (patch) | |
| tree | 4f38fb6210abaa9f354353df2abda96d1b8fb612 /ipapython/install | |
| parent | 452b08754d02b89c0e3117b83d9156e6110943c9 (diff) | |
| download | freeipa-0f88f8fe889ae4801fc8d5ece1ad51c5246718ac.tar.gz freeipa-0f88f8fe889ae4801fc8d5ece1ad51c5246718ac.tar.xz freeipa-0f88f8fe889ae4801fc8d5ece1ad51c5246718ac.zip | |
Remove unused variables in the code
This commit removes unused variables or rename variables as "expected to
be unused" by using "_" prefix.
This covers only cases where fix was easy or only one unused variable
was in a module
Reviewed-By: Florence Blanc-Renaud <frenaud@redhat.com>
Reviewed-By: Stanislav Laznicka <slaznick@redhat.com>
Diffstat (limited to 'ipapython/install')
| -rw-r--r-- | ipapython/install/cli.py | 2 | ||||
| -rw-r--r-- | ipapython/install/common.py | 4 |
2 files changed, 3 insertions, 3 deletions
diff --git a/ipapython/install/cli.py b/ipapython/install/cli.py index 9cd9ec768..39c4c4491 100644 --- a/ipapython/install/cli.py +++ b/ipapython/install/cli.py @@ -152,7 +152,7 @@ class ConfigureTool(admintool.AdminTool): **kwargs ) - for group, opt_group in groups.items(): + for opt_group in groups.values(): parser.add_option_group(opt_group) super(ConfigureTool, cls).add_options(parser, diff --git a/ipapython/install/common.py b/ipapython/install/common.py index fc8e33291..42890374a 100644 --- a/ipapython/install/common.py +++ b/ipapython/install/common.py @@ -60,7 +60,7 @@ class Step(Installable): raise AttributeError('parent') def _install(self): - for nothing in self._installer(self.parent): + for _nothing in self._installer(self.parent): yield from_(super(Step, self)._install()) @staticmethod @@ -68,7 +68,7 @@ class Step(Installable): yield def _uninstall(self): - for nothing in self._uninstaller(self.parent): + for _nothing in self._uninstaller(self.parent): yield from_(super(Step, self)._uninstall()) @staticmethod |