From 225250ddf1dbe4a6e8438eeef941dd76fcaa1eab Mon Sep 17 00:00:00 2001
From: Pierre Neidhardt <ambrevar@gmail.com>
Date: Wed, 10 Aug 2016 15:36:50 +0530
Subject: [PATCH] html.go: Lint RenderNode()

---
 html.go | 13 +++----------
 1 file changed, 3 insertions(+), 10 deletions(-)

diff --git a/html.go b/html.go
index 96fb655..3f3cf32 100644
--- a/html.go
+++ b/html.go
@@ -416,9 +416,8 @@ func (r *HTMLRenderer) RenderNode(w io.Writer, node *Node, entering bool) WalkSt
 			node.Literal = r.sr.Process(node.Literal)
 		}
 		r.out(w, node.Literal)
-		break
 	case Softbreak:
-		r.out(w, []byte("\n"))
+		r.out(w, []byte{'\n'})
 		// TODO: make it configurable via out(renderer.softbreak)
 	case Hardbreak:
 		r.out(w, tag("br", nil, true))
@@ -429,14 +428,12 @@ func (r *HTMLRenderer) RenderNode(w io.Writer, node *Node, entering bool) WalkSt
 		} else {
 			r.out(w, tag("/em", nil, false))
 		}
-		break
 	case Strong:
 		if entering {
 			r.out(w, tag("strong", nil, false))
 		} else {
 			r.out(w, tag("/strong", nil, false))
 		}
-		break
 	case Del:
 		if entering {
 			r.out(w, tag("del", nil, false))
@@ -525,8 +522,8 @@ func (r *HTMLRenderer) RenderNode(w io.Writer, node *Node, entering bool) WalkSt
 			// TODO: untangle this clusterfuck about when the newlines need
 			// to be added and when not.
 			if node.Prev != nil {
-				t := node.Prev.Type
-				if t == HTMLBlock || t == List || t == Paragraph || t == Header || t == CodeBlock || t == BlockQuote || t == HorizontalRule {
+				switch node.Prev.Type {
+				case HTMLBlock, List, Paragraph, Header, CodeBlock, BlockQuote, HorizontalRule:
 					r.cr(w)
 				}
 			}
@@ -540,7 +537,6 @@ func (r *HTMLRenderer) RenderNode(w io.Writer, node *Node, entering bool) WalkSt
 				r.cr(w)
 			}
 		}
-		break
 	case BlockQuote:
 		if entering {
 			r.cr(w)
@@ -549,7 +545,6 @@ func (r *HTMLRenderer) RenderNode(w io.Writer, node *Node, entering bool) WalkSt
 			r.out(w, tag("/blockquote", nil, false))
 			r.cr(w)
 		}
-		break
 	case HTMLBlock:
 		if r.Flags&SkipHTML != 0 {
 			break
@@ -581,12 +576,10 @@ func (r *HTMLRenderer) RenderNode(w io.Writer, node *Node, entering bool) WalkSt
 				r.cr(w)
 			}
 		}
-		break
 	case HorizontalRule:
 		r.cr(w)
 		r.out(w, tag("hr", attrs, r.Flags&UseXHTML != 0))
 		r.cr(w)
-		break
 	case List:
 		tagName := "ul"
 		if node.ListFlags&ListTypeOrdered != 0 {