From b0cd3b8ab9b174f11fd4db8737106be5caf9cc1c Mon Sep 17 00:00:00 2001 From: Christian Muehlhaeuser Date: Mon, 22 Jul 2019 11:35:54 +0200 Subject: [PATCH] Fixed ineffectual assignments (#7555) Don't assign values we never use. --- models/git_diff.go | 2 +- routers/user/setting/security_twofa.go | 1 - 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/models/git_diff.go b/models/git_diff.go index 2f48f1b6f..518d543e7 100644 --- a/models/git_diff.go +++ b/models/git_diff.go @@ -543,7 +543,7 @@ func ParsePatch(maxLines, maxLineCharacters, maxFiles int, reader io.Reader) (*D // Get new file. if strings.HasPrefix(line, cmdDiffHead) { - middle := -1 + var middle int // Note: In case file name is surrounded by double quotes (it happens only in git-shell). // e.g. diff --git "a/xxx" "b/xxx" diff --git a/routers/user/setting/security_twofa.go b/routers/user/setting/security_twofa.go index 6e3516dbb..5aa951a9b 100644 --- a/routers/user/setting/security_twofa.go +++ b/routers/user/setting/security_twofa.go @@ -81,7 +81,6 @@ func twofaGenerateSecretAndQr(ctx *context.Context) bool { // Filter unsafe character ':' in issuer issuer := strings.Replace(setting.AppName+" ("+setting.Domain+")", ":", "", -1) if otpKey == nil { - err = nil // clear the error, in case the URL was invalid otpKey, err = totp.Generate(totp.GenerateOpts{ SecretSize: 40, Issuer: issuer,