http_handle_*.go: Use fmt.Fprintln, not w.Write

1 files changed, 8 insertions, 7 deletions

diff --git a/http_handle_repo_commit.go b/http_handle_repo_commit.go
index 073bfa1..1a84862 100644
--- a/http_handle_repo_commit.go
+++ b/http_handle_repo_commit.go
@@ -1,6 +1,7 @@
 package main
 
 import (
+	"fmt"
 	"net/http"
 	"strings"
 
@@ -20,23 +21,23 @@ func handle_repo_commit(w http.ResponseWriter, r *http.Request, params map[strin
 	group_name, repo_name, commit_id_specified_string := params["group_name"].(string), params["repo_name"].(string), params["commit_id"].(string)
 	repo, err := open_git_repo(r.Context(), group_name, repo_name)
 	if err != nil {
-		_, _ = w.Write([]byte("Error opening repo: " + err.Error()))
+		fmt.Fprintln(w, "Error opening repo:", err.Error())
 		return
 	}
 	commit_id_specified_string_without_suffix := strings.TrimSuffix(commit_id_specified_string, ".patch")
 	commit_id := plumbing.NewHash(commit_id_specified_string_without_suffix)
 	commit_object, err := repo.CommitObject(commit_id)
 	if err != nil {
-		_, _ = w.Write([]byte("Error getting commit object: " + err.Error()))
+		fmt.Fprintln(w, "Error getting commit object:", err.Error())
 		return
 	}
 	if commit_id_specified_string_without_suffix != commit_id_specified_string {
 		patch, err := format_patch_from_commit(commit_object)
 		if err != nil {
-			_, _ = w.Write([]byte("Error formatting patch: " + err.Error()))
+			fmt.Fprintln(w, "Error formatting patch:", err.Error())
 			return
 		}
-		_, _ = w.Write([]byte(patch))
+		fmt.Fprintln(w, patch)
 		return
 	}
 	commit_id_string := commit_object.Hash.String()
@@ -51,13 +52,13 @@ func handle_repo_commit(w http.ResponseWriter, r *http.Request, params map[strin
 
 	parent_commit_hash, patch, err := get_patch_from_commit(commit_object)
 	if err != nil {
-		_, _ = w.Write([]byte("Error getting patch from commit: " + err.Error()))
+		fmt.Fprintln(w, "Error getting patch from commit:", err.Error())
 		return
 	}
 	params["parent_commit_hash"] = parent_commit_hash.String()
 	params["patch"] = patch
 
-	// TODO: Remove unnecessary context
+	// TODO: Remove unnecessary context                                          
 	// TODO: Prepend "+"/"-"/" " instead of solely distinguishing based on color
 	usable_file_patches := make([]usable_file_patch, 0)
 	for _, file_patch := range patch.FilePatches() {
@@ -79,7 +80,7 @@ func handle_repo_commit(w http.ResponseWriter, r *http.Request, params map[strin
 
 	err = templates.ExecuteTemplate(w, "repo_commit", params)
 	if err != nil {
-		_, _ = w.Write([]byte("Error rendering template: " + err.Error()))
+		fmt.Fprintln(w, "Error rendering template:", err.Error())
 		return
 	}
 }