commit ca2424576217c21e87afcf5ec3f7be3629c1bc38
author Dan Kortschak <dan@kortschak.io> Tue Dec 01 21:30:08 2020 +1030
committer Dan Kortschak <dan@kortschak.io> Wed Dec 02 06:12:09 2020 +1030
tree 7221f76d2e39efb0c34a64dd558c5904efe03573
parent e2a07cfc01e46013446a1c8f779b678bc31cd053

graph/encoding/dot: factor out repeated forward keys

graph/encoding/dot/encode.go

diff --git a/graph/encoding/dot/encode.go b/graph/encoding/dot/encode.go
index 1cba7c1..2ff893e 100644
--- a/graph/encoding/dot/encode.go
+++ b/graph/encoding/dot/encode.go
@@ -210,16 +210,17 @@
 		sort.Sort(ordered.ByID(to))
 		for _, t := range to {
 			tid := t.ID()
+			f := edge{inGraph: name, from: nid, to: tid}
 			if isDirected {
-				if p.visited[edge{inGraph: name, from: nid, to: tid}] {
+				if p.visited[f] {
 					continue
 				}
-				p.visited[edge{inGraph: name, from: nid, to: tid}] = true
+				p.visited[f] = true
 			} else {
-				if p.visited[edge{inGraph: name, from: nid, to: tid}] {
+				if p.visited[f] {
 					continue
 				}
-				p.visited[edge{inGraph: name, from: nid, to: tid}] = true
+				p.visited[f] = true
 				p.visited[edge{inGraph: name, from: tid, to: nid}] = true
 			}
 
@@ -512,16 +513,17 @@
 
 			for _, l := range lines {
 				lid := l.ID()
+				f := line{inGraph: name, from: nid, to: tid, id: lid}
 				if isDirected {
-					if p.visited[line{inGraph: name, from: nid, to: tid, id: lid}] {
+					if p.visited[f] {
 						continue
 					}
-					p.visited[line{inGraph: name, from: nid, to: tid, id: lid}] = true
+					p.visited[f] = true
 				} else {
-					if p.visited[line{inGraph: name, from: nid, to: tid, id: lid}] {
+					if p.visited[f] {
 						continue
 					}
-					p.visited[line{inGraph: name, from: nid, to: tid, id: lid}] = true
+					p.visited[f] = true
 					p.visited[line{inGraph: name, from: tid, to: nid, id: lid}] = true
 				}