From 7e0014dd1391e123d95f2537c3b2165fef7122ef Mon Sep 17 00:00:00 2001
From: Mai-Lapyst <mai-lapyst@noreply.codeberg.org>
Date: Wed, 27 Mar 2024 18:36:12 +0100
Subject: [PATCH] Fix formating & remove commented out code

---
 modules/markup/file_preview.go     | 20 ++++----------------
 modules/markup/renderer.go         |  2 +-
 services/markup/processorhelper.go | 19 -------------------
 3 files changed, 5 insertions(+), 36 deletions(-)

diff --git a/modules/markup/file_preview.go b/modules/markup/file_preview.go
index 32683c317c..95c94e0c14 100644
--- a/modules/markup/file_preview.go
+++ b/modules/markup/file_preview.go
@@ -37,7 +37,7 @@ type FilePreview struct {
 }
 
 func NewFilePreview(ctx *RenderContext, node *html.Node, locale translation.Locale) *FilePreview {
-	if (setting.FilePreviewMaxLines == 0) {
+	if setting.FilePreviewMaxLines == 0 {
 		// Feature is disabled
 		return nil
 	}
@@ -84,7 +84,6 @@ func NewFilePreview(ctx *RenderContext, node *html.Node, locale translation.Loca
 	}
 
 	lineSpecs := strings.Split(hash, "-")
-	// lineCount := len(fileContent)
 
 	commitLinkBuffer := new(bytes.Buffer)
 	err = html.Render(commitLinkBuffer, createLink(node.Data[m[0]:m[5]], commitSha[0:7], "text black"))
@@ -97,11 +96,6 @@ func NewFilePreview(ctx *RenderContext, node *html.Node, locale translation.Loca
 	if len(lineSpecs) == 1 {
 		startLine, _ = strconv.Atoi(strings.TrimPrefix(lineSpecs[0], "L"))
 		endLine = startLine
-		// if line < 1 || line > lineCount {
-		// 	return nil
-		// }
-
-		// preview.fileContent = fileContent[line-1 : line]
 		preview.subTitle = locale.Tr(
 			"markup.filepreview.line", startLine,
 			template.HTML(commitLinkBuffer.String()),
@@ -111,12 +105,6 @@ func NewFilePreview(ctx *RenderContext, node *html.Node, locale translation.Loca
 	} else {
 		startLine, _ = strconv.Atoi(strings.TrimPrefix(lineSpecs[0], "L"))
 		endLine, _ = strconv.Atoi(strings.TrimPrefix(lineSpecs[1], "L"))
-
-		// if startLine < 1 || endLine < 1 || startLine > lineCount || endLine > lineCount || endLine < startLine {
-		// 	return nil
-		// }
-
-		// preview.fileContent = fileContent[startLine-1 : endLine]
 		preview.subTitle = locale.Tr(
 			"markup.filepreview.lines", startLine, endLine,
 			template.HTML(commitLinkBuffer.String()),
@@ -125,7 +113,7 @@ func NewFilePreview(ctx *RenderContext, node *html.Node, locale translation.Loca
 		preview.lineOffset = startLine - 1
 	}
 
-	lineCount := endLine - (startLine-1)
+	lineCount := endLine - (startLine - 1)
 	if startLine < 1 || endLine < 1 || lineCount < 1 {
 		return nil
 	}
@@ -156,7 +144,7 @@ func NewFilePreview(ctx *RenderContext, node *html.Node, locale translation.Loca
 	for i := 0; i < lineCount; i++ {
 		buf, err := reader.ReadBytes('\n')
 		if err != nil {
-			break;
+			break
 		}
 		lineBuffer.Write(buf)
 	}
@@ -316,7 +304,7 @@ func (p *FilePreview) CreateHTML(locale translation.Locale) *html.Node {
 	}
 	node.AppendChild(header)
 
-	if (p.isTruncated) {
+	if p.isTruncated {
 		warning := &html.Node{
 			Type: html.ElementNode,
 			Data: atom.Div.String(),
diff --git a/modules/markup/renderer.go b/modules/markup/renderer.go
index b08c9eb230..163cd5d688 100644
--- a/modules/markup/renderer.go
+++ b/modules/markup/renderer.go
@@ -32,7 +32,7 @@ const (
 
 type ProcessorHelper struct {
 	IsUsernameMentionable func(ctx context.Context, username string) bool
-	GetRepoFileBlob    func(ctx context.Context, ownerName, repoName, commitSha, filePath string, language *string) (*git.Blob, error)
+	GetRepoFileBlob       func(ctx context.Context, ownerName, repoName, commitSha, filePath string, language *string) (*git.Blob, error)
 
 	ElementDir string // the direction of the elements, eg: "ltr", "rtl", "auto", default to no direction attribute
 }
diff --git a/services/markup/processorhelper.go b/services/markup/processorhelper.go
index 7466e962d2..ac751d0e62 100644
--- a/services/markup/processorhelper.go
+++ b/services/markup/processorhelper.go
@@ -86,25 +86,6 @@ func ProcessorHelper() *markup.ProcessorHelper {
 			}
 
 			return blob, nil
-
-			/*dataRc, err := blob.DataAsync()
-			if err != nil {
-				return nil, err
-			}
-			defer dataRc.Close()
-
-			buf, err := io.ReadAll(dataRc)
-			if err != nil {
-				log.Error("failed to completly read blob for %-v:%s. Error: %v", repo, filePath, err)
-			}
-
-			fileContent, _, err := highlight.File(blob.Name(), language, buf)
-			if err != nil {
-				log.Error("highlight.File failed, fallback to plain text: %v", err)
-				fileContent = highlight.PlainText(buf)
-			}
-
-			return fileContent, nil*/
 		},
 	}
 }