From: Ian Jackson <ijackson@chiark.greenend.org.uk>
Date: Thu, 24 Oct 2019 16:18:37 +0000 (+0100)
Subject: make-secnet-sites: Apply list() to keys in delempty
X-Git-Tag: v0.5.0~29
X-Git-Url: http://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?p=secnet.git;a=commitdiff_plain;h=92eddd582b64891d360ad7bbafdc1e228ee2037e

make-secnet-sites: Apply list() to keys in delempty

It is not permitted (in Python3) to modify a dictionary like this,
while iterating over keys().  We have to make a list of the keys,
copying them.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
---

diff --git a/make-secnet-sites b/make-secnet-sites
index 76d74f1..1cc979b 100755
--- a/make-secnet-sites
+++ b/make-secnet-sites
@@ -527,7 +527,7 @@ def live(n):
 	return 0
 def delempty(n):
 	"Delete nodes that have no leafnode children"
-	for i in n.children.keys():
+	for i in list(n.children.keys()):
 		delempty(n.children[i])
 		if not live(n.children[i]):
 			del n.children[i]