From a0676b73e471712a2da0791221f36132301fe89d Mon Sep 17 00:00:00 2001 From: Vic Demuzere Date: Tue, 24 Aug 2021 14:57:20 +0200 Subject: [PATCH] Remove duplicate call LastIndex This code had to loop over the entire string twice. --- main.go | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/main.go b/main.go index fe60be2..b5ac0e8 100644 --- a/main.go +++ b/main.go @@ -67,13 +67,14 @@ func assetDelivery(res http.ResponseWriter, r *http.Request) { assetName = "/index.html" } - if strings.LastIndex(assetName, ".") < 0 { + dot := strings.LastIndex(assetName, ".") + if dot < 0 { // There are no assets with no dot in it http.Error(res, "404 not found", http.StatusNotFound) return } - ext := assetName[strings.LastIndex(assetName, "."):] + ext := assetName[dot:] assetData, err := Asset(path.Join("frontend", assetName)) if err != nil { http.Error(res, "404 not found", http.StatusNotFound)