From f68c2c765714020c5791d8cf12f486867703f47b Mon Sep 17 00:00:00 2001
From: Tim Edwards <tim@opencircuitdesign.com>
Date: Mon, 6 Dec 2021 09:52:59 -0500
Subject: [PATCH] One small change to the ext2spice topVisit code to ignore
 nodes that have already been output;  i.e., that have EF_PORT set.  However,
 since EF_PORT is now set on all implicit ports, it is likely that this part
 of the code is no longer exercised at all, and may be removed.

---
 VERSION               | 2 +-
 ext2spice/ext2spice.c | 3 ++-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/VERSION b/VERSION
index db5c44e5..f3f03f43 100644
--- a/VERSION
+++ b/VERSION
@@ -1 +1 @@
-8.3.237
+8.3.238
diff --git a/ext2spice/ext2spice.c b/ext2spice/ext2spice.c
index 174b76ad..abf3c695 100644
--- a/ext2spice/ext2spice.c
+++ b/ext2spice/ext2spice.c
@@ -1869,7 +1869,8 @@ topVisit(def, doStub)
 	    if (sname == NULL) continue;
 	    snode = sname->efnn_node;
 
-	    if (snode && (snode->efnode_flags & EF_SUBS_PORT))
+	    if (snode && (snode->efnode_flags & EF_SUBS_PORT) &&
+		    !(snode->efnode_flags & EF_PORT))
 	    {
 		if (snode->efnode_name->efnn_port < 0)
 		{