From af88deac1773a83ccab5417efbb5f6b5b479a9cc Mon Sep 17 00:00:00 2001 From: Chris PeBenito Date: Sat, 16 Sep 2017 14:32:34 -0400 Subject: [PATCH] PermissionMapEditor, EdgeAttrList: Use list .clear() method. Closes #110 --- setools/descriptors.py | 4 +--- setoolsgui/apol/permmapedit.py | 4 +--- 2 files changed, 2 insertions(+), 6 deletions(-) diff --git a/setools/descriptors.py b/setools/descriptors.py index 560e9a8..8f67957 100644 --- a/setools/descriptors.py +++ b/setools/descriptors.py @@ -197,9 +197,7 @@ class EdgeAttrList(NetworkXGraphEdgeDescriptor): raise ValueError("{0} lists should not be assigned directly".format(self.name)) def __delete__(self, obj): - # in Python3 a .clear() function was added for lists - # keep this implementation for Python 2 compat - del obj.G[obj.source][obj.target][self.name][:] + obj.G[obj.source][obj.target][self.name].clear() # diff --git a/setoolsgui/apol/permmapedit.py b/setoolsgui/apol/permmapedit.py index 9217a1c..2215fea 100644 --- a/setoolsgui/apol/permmapedit.py +++ b/setoolsgui/apol/permmapedit.py @@ -138,9 +138,7 @@ class PermissionMapEditor(SEToolsWidget, QDialog): for mapping in self.widgets: mapping.close() - # in Python3 a .clear() function was added for lists - # keep this implementation for Python 2 compat - del self.widgets[:] + self.widgets.clear() index_to_setting = ["r", "w", "b", "n"]