From 9c4601bdf8369ed72944085e3952111cf4aeea11 Mon Sep 17 00:00:00 2001
From: 6543 <6543@obermui.de>
Date: Fri, 9 Apr 2021 09:40:34 +0200
Subject: [PATCH] Code Formats, Nits & Unused Func/Var deletions (#15286)

* _ to unused func options

* rm useless brakets

* rm trifial non used models functions

* rm dead code

* rm dead global vars

* fix routers/api/v1/repo/issue.go

* dont overload import module
---
 cmd/admin.go                             |  6 +++---
 cmd/hook.go                              |  4 ++--
 models/branches.go                       | 19 -------------------
 models/issue_label.go                    | 13 -------------
 models/lfs.go                            |  4 ++--
 models/repo_test.go                      |  2 +-
 models/user.go                           |  3 ---
 models/user_heatmap.go                   |  2 +-
 models/user_openid.go                    |  6 +-----
 modules/markup/common/linkify.go         |  4 +---
 modules/process/manager.go               |  7 ++-----
 modules/queue/bytefifo.go                |  4 ++--
 modules/queue/helper.go                  |  2 +-
 modules/queue/queue_bytefifo.go          |  4 ++--
 modules/queue/queue_disk.go              |  2 +-
 modules/queue/queue_redis.go             |  2 +-
 modules/queue/unique_queue_disk.go       |  2 +-
 modules/queue/unique_queue_redis.go      |  2 +-
 modules/references/references.go         |  2 +-
 modules/repofiles/action.go              |  2 +-
 modules/setting/setting.go               |  5 -----
 modules/storage/helper.go                |  2 +-
 modules/storage/storage.go               |  2 --
 routers/api/v1/org/team.go               |  2 +-
 routers/api/v1/repo/issue.go             |  5 ++---
 routers/api/v1/repo/pull.go              |  2 +-
 routers/events/events.go                 |  8 ++++----
 routers/org/teams.go                     |  4 ++--
 routers/repo/http.go                     |  2 +-
 routers/repo/issue.go                    |  2 --
 routers/repo/setting_protected_branch.go |  8 ++++----
 routers/user/home.go                     |  3 ---
 services/pull/merge.go                   |  2 +-
 33 files changed, 41 insertions(+), 98 deletions(-)

diff --git a/cmd/admin.go b/cmd/admin.go
index 3ddf8eddf9..31ba77877b 100644
--- a/cmd/admin.go
+++ b/cmd/admin.go
@@ -512,7 +512,7 @@ func runDeleteUser(c *cli.Context) error {
 	return models.DeleteUser(user)
 }
 
-func runRepoSyncReleases(c *cli.Context) error {
+func runRepoSyncReleases(_ *cli.Context) error {
 	if err := initDB(); err != nil {
 		return err
 	}
@@ -578,14 +578,14 @@ func getReleaseCount(id int64) (int64, error) {
 	)
 }
 
-func runRegenerateHooks(c *cli.Context) error {
+func runRegenerateHooks(_ *cli.Context) error {
 	if err := initDB(); err != nil {
 		return err
 	}
 	return repo_module.SyncRepositoryHooks(graceful.GetManager().ShutdownContext())
 }
 
-func runRegenerateKeys(c *cli.Context) error {
+func runRegenerateKeys(_ *cli.Context) error {
 	if err := initDB(); err != nil {
 		return err
 	}
diff --git a/cmd/hook.go b/cmd/hook.go
index 1fcc0a18c3..def3b636eb 100644
--- a/cmd/hook.go
+++ b/cmd/hook.go
@@ -166,7 +166,7 @@ Gitea or set your environment appropriately.`, "")
 	}
 
 	// the environment setted on serv command
-	isWiki := (os.Getenv(models.EnvRepoIsWiki) == "true")
+	isWiki := os.Getenv(models.EnvRepoIsWiki) == "true"
 	username := os.Getenv(models.EnvRepoUsername)
 	reponame := os.Getenv(models.EnvRepoName)
 	userID, _ := strconv.ParseInt(os.Getenv(models.EnvPusherID), 10, 64)
@@ -322,7 +322,7 @@ Gitea or set your environment appropriately.`, "")
 
 	// the environment setted on serv command
 	repoUser := os.Getenv(models.EnvRepoUsername)
-	isWiki := (os.Getenv(models.EnvRepoIsWiki) == "true")
+	isWiki := os.Getenv(models.EnvRepoIsWiki) == "true"
 	repoName := os.Getenv(models.EnvRepoName)
 	pusherID, _ := strconv.ParseInt(os.Getenv(models.EnvPusherID), 10, 64)
 	pusherName := os.Getenv(models.EnvPusherName)
diff --git a/models/branches.go b/models/branches.go
index 440c093095..1ac1fa49e5 100644
--- a/models/branches.go
+++ b/models/branches.go
@@ -260,12 +260,6 @@ func (protectBranch *ProtectedBranch) IsProtectedFile(patterns []glob.Glob, path
 	return r
 }
 
-// GetProtectedBranchByRepoID getting protected branch by repo ID
-func GetProtectedBranchByRepoID(repoID int64) ([]*ProtectedBranch, error) {
-	protectedBranches := make([]*ProtectedBranch, 0)
-	return protectedBranches, x.Where("repo_id = ?", repoID).Desc("updated_unix").Find(&protectedBranches)
-}
-
 // GetProtectedBranchBy getting protected branch by ID/Name
 func GetProtectedBranchBy(repoID int64, branchName string) (*ProtectedBranch, error) {
 	return getProtectedBranchBy(x, repoID, branchName)
@@ -283,19 +277,6 @@ func getProtectedBranchBy(e Engine, repoID int64, branchName string) (*Protected
 	return rel, nil
 }
 
-// GetProtectedBranchByID getting protected branch by ID
-func GetProtectedBranchByID(id int64) (*ProtectedBranch, error) {
-	rel := &ProtectedBranch{}
-	has, err := x.ID(id).Get(rel)
-	if err != nil {
-		return nil, err
-	}
-	if !has {
-		return nil, nil
-	}
-	return rel, nil
-}
-
 // WhitelistOptions represent all sorts of whitelists used for protected branches
 type WhitelistOptions struct {
 	UserIDs []int64
diff --git a/models/issue_label.go b/models/issue_label.go
index 4f082b8a6e..d1ff469236 100644
--- a/models/issue_label.go
+++ b/models/issue_label.go
@@ -510,19 +510,6 @@ func GetLabelIDsInOrgByNames(orgID int64, labelNames []string) ([]int64, error)
 		Find(&labelIDs)
 }
 
-// GetLabelIDsInOrgsByNames returns a list of labelIDs by names in one of the given
-// organization.
-// it silently ignores label names that do not belong to the organization.
-func GetLabelIDsInOrgsByNames(orgIDs []int64, labelNames []string) ([]int64, error) {
-	labelIDs := make([]int64, 0, len(labelNames))
-	return labelIDs, x.Table("label").
-		In("org_id", orgIDs).
-		In("name", labelNames).
-		Asc("name").
-		Cols("id").
-		Find(&labelIDs)
-}
-
 // GetLabelInOrgByID returns a label by ID in given organization.
 func GetLabelInOrgByID(orgID, labelID int64) (*Label, error) {
 	return getLabelInOrgByID(x, orgID, labelID)
diff --git a/models/lfs.go b/models/lfs.go
index 90f6523d4a..c8abb1cd49 100644
--- a/models/lfs.go
+++ b/models/lfs.go
@@ -131,11 +131,11 @@ func (repo *Repository) CountLFSMetaObjects() (int64, error) {
 func LFSObjectAccessible(user *User, oid string) (bool, error) {
 	if user.IsAdmin {
 		count, err := x.Count(&LFSMetaObject{Pointer: lfs.Pointer{Oid: oid}})
-		return (count > 0), err
+		return count > 0, err
 	}
 	cond := accessibleRepositoryCondition(user)
 	count, err := x.Where(cond).Join("INNER", "repository", "`lfs_meta_object`.repository_id = `repository`.id").Count(&LFSMetaObject{Pointer: lfs.Pointer{Oid: oid}})
-	return (count > 0), err
+	return count > 0, err
 }
 
 // LFSAutoAssociate auto associates accessible LFSMetaObjects
diff --git a/models/repo_test.go b/models/repo_test.go
index cd4bbcccfa..10ba2c99f8 100644
--- a/models/repo_test.go
+++ b/models/repo_test.go
@@ -75,7 +75,7 @@ func TestGetRepositoryCount(t *testing.T) {
 	assert.NoError(t, err2)
 	assert.NoError(t, err3)
 	assert.Equal(t, int64(3), count)
-	assert.Equal(t, (privateCount + publicCount), count)
+	assert.Equal(t, privateCount+publicCount, count)
 }
 
 func TestGetPublicRepositoryCount(t *testing.T) {
diff --git a/models/user.go b/models/user.go
index aacf2957e3..cdb82603cd 100644
--- a/models/user.go
+++ b/models/user.go
@@ -76,9 +76,6 @@ const (
 )
 
 var (
-	// ErrUserNotKeyOwner user does not own this key error
-	ErrUserNotKeyOwner = errors.New("User does not own this public key")
-
 	// ErrEmailNotExist e-mail does not exist error
 	ErrEmailNotExist = errors.New("E-mail does not exist")
 
diff --git a/models/user_heatmap.go b/models/user_heatmap.go
index 74678459cf..0e2767212e 100644
--- a/models/user_heatmap.go
+++ b/models/user_heatmap.go
@@ -65,7 +65,7 @@ func getUserHeatmapData(user *User, team *Team, doer *User) ([]*UserHeatmapData,
 		Select(groupBy+" AS timestamp, count(user_id) as contributions").
 		Table("action").
 		Where(cond).
-		And("created_unix > ?", (timeutil.TimeStampNow() - 31536000)).
+		And("created_unix > ?", timeutil.TimeStampNow()-31536000).
 		GroupBy(groupByName).
 		OrderBy("timestamp").
 		Find(&hdata)
diff --git a/models/user_openid.go b/models/user_openid.go
index 18cf51c6b6..597f19d77d 100644
--- a/models/user_openid.go
+++ b/models/user_openid.go
@@ -35,6 +35,7 @@ func GetUserOpenIDs(uid int64) ([]*UserOpenID, error) {
 	return openids, nil
 }
 
+// isOpenIDUsed returns true if the openid has been used.
 func isOpenIDUsed(e Engine, uri string) (bool, error) {
 	if len(uri) == 0 {
 		return true, nil
@@ -43,11 +44,6 @@ func isOpenIDUsed(e Engine, uri string) (bool, error) {
 	return e.Get(&UserOpenID{URI: uri})
 }
 
-// IsOpenIDUsed returns true if the openid has been used.
-func IsOpenIDUsed(openid string) (bool, error) {
-	return isOpenIDUsed(x, openid)
-}
-
 // NOTE: make sure openid.URI is normalized already
 func addUserOpenID(e Engine, openid *UserOpenID) error {
 	used, err := isOpenIDUsed(e, openid.URI)
diff --git a/modules/markup/common/linkify.go b/modules/markup/common/linkify.go
index 25621bf801..8a4b2a8985 100644
--- a/modules/markup/common/linkify.go
+++ b/modules/markup/common/linkify.go
@@ -122,9 +122,7 @@ func (s *linkifyParser) Parse(parent ast.Node, block text.Reader, pc parser.Cont
 			}
 		}
 	}
-	if m == nil {
-		return nil
-	}
+
 	if consumes != 0 {
 		s := segment.WithStop(segment.Start + 1)
 		ast.MergeOrAppendTextSegment(parent, s)
diff --git a/modules/process/manager.go b/modules/process/manager.go
index 9d57f4eb7b..e42e38a0f0 100644
--- a/modules/process/manager.go
+++ b/modules/process/manager.go
@@ -8,7 +8,6 @@ package process
 import (
 	"bytes"
 	"context"
-	"errors"
 	"fmt"
 	"io"
 	"os/exec"
@@ -22,10 +21,8 @@ import (
 // then we delete the singleton.
 
 var (
-	// ErrExecTimeout represent a timeout error
-	ErrExecTimeout = errors.New("Process execution timeout")
-	manager        *Manager
-	managerInit    sync.Once
+	manager     *Manager
+	managerInit sync.Once
 
 	// DefaultContext is the default context to run processing commands in
 	DefaultContext = context.Background()
diff --git a/modules/queue/bytefifo.go b/modules/queue/bytefifo.go
index 2cd0ba0b95..94478e6f05 100644
--- a/modules/queue/bytefifo.go
+++ b/modules/queue/bytefifo.go
@@ -23,7 +23,7 @@ type UniqueByteFIFO interface {
 	Has(data []byte) (bool, error)
 }
 
-var _ (ByteFIFO) = &DummyByteFIFO{}
+var _ ByteFIFO = &DummyByteFIFO{}
 
 // DummyByteFIFO represents a dummy fifo
 type DummyByteFIFO struct{}
@@ -48,7 +48,7 @@ func (*DummyByteFIFO) Len() int64 {
 	return 0
 }
 
-var _ (UniqueByteFIFO) = &DummyUniqueByteFIFO{}
+var _ UniqueByteFIFO = &DummyUniqueByteFIFO{}
 
 // DummyUniqueByteFIFO represents a dummy unique fifo
 type DummyUniqueByteFIFO struct {
diff --git a/modules/queue/helper.go b/modules/queue/helper.go
index 161c2fe8e7..e0368bce3a 100644
--- a/modules/queue/helper.go
+++ b/modules/queue/helper.go
@@ -50,7 +50,7 @@ func toConfig(exemplar, cfg interface{}) (interface{}, error) {
 		var err error
 
 		configBytes, err = json.Marshal(cfg)
-		ok = (err == nil)
+		ok = err == nil
 	}
 	if !ok {
 		// no ... we've tried hard enough at this point - throw an error!
diff --git a/modules/queue/queue_bytefifo.go b/modules/queue/queue_bytefifo.go
index fe5178ff2d..bc86078493 100644
--- a/modules/queue/queue_bytefifo.go
+++ b/modules/queue/queue_bytefifo.go
@@ -21,7 +21,7 @@ type ByteFIFOQueueConfiguration struct {
 	Name    string
 }
 
-var _ (Queue) = &ByteFIFOQueue{}
+var _ Queue = &ByteFIFOQueue{}
 
 // ByteFIFOQueue is a Queue formed from a ByteFIFO and WorkerPool
 type ByteFIFOQueue struct {
@@ -196,7 +196,7 @@ func (q *ByteFIFOQueue) IsTerminated() <-chan struct{} {
 	return q.terminated
 }
 
-var _ (UniqueQueue) = &ByteFIFOUniqueQueue{}
+var _ UniqueQueue = &ByteFIFOUniqueQueue{}
 
 // ByteFIFOUniqueQueue represents a UniqueQueue formed from a UniqueByteFifo
 type ByteFIFOUniqueQueue struct {
diff --git a/modules/queue/queue_disk.go b/modules/queue/queue_disk.go
index 88b8c414c0..6c15a8e63b 100644
--- a/modules/queue/queue_disk.go
+++ b/modules/queue/queue_disk.go
@@ -55,7 +55,7 @@ func NewLevelQueue(handle HandlerFunc, cfg, exemplar interface{}) (Queue, error)
 	return queue, nil
 }
 
-var _ (ByteFIFO) = &LevelQueueByteFIFO{}
+var _ ByteFIFO = &LevelQueueByteFIFO{}
 
 // LevelQueueByteFIFO represents a ByteFIFO formed from a LevelQueue
 type LevelQueueByteFIFO struct {
diff --git a/modules/queue/queue_redis.go b/modules/queue/queue_redis.go
index 2b1d36f0ad..af2cc30335 100644
--- a/modules/queue/queue_redis.go
+++ b/modules/queue/queue_redis.go
@@ -69,7 +69,7 @@ type redisClient interface {
 	Close() error
 }
 
-var _ (ByteFIFO) = &RedisByteFIFO{}
+var _ ByteFIFO = &RedisByteFIFO{}
 
 // RedisByteFIFO represents a ByteFIFO formed from a redisClient
 type RedisByteFIFO struct {
diff --git a/modules/queue/unique_queue_disk.go b/modules/queue/unique_queue_disk.go
index dd6ac1a538..8ec8848bc4 100644
--- a/modules/queue/unique_queue_disk.go
+++ b/modules/queue/unique_queue_disk.go
@@ -59,7 +59,7 @@ func NewLevelUniqueQueue(handle HandlerFunc, cfg, exemplar interface{}) (Queue,
 	return queue, nil
 }
 
-var _ (UniqueByteFIFO) = &LevelUniqueQueueByteFIFO{}
+var _ UniqueByteFIFO = &LevelUniqueQueueByteFIFO{}
 
 // LevelUniqueQueueByteFIFO represents a ByteFIFO formed from a LevelUniqueQueue
 type LevelUniqueQueueByteFIFO struct {
diff --git a/modules/queue/unique_queue_redis.go b/modules/queue/unique_queue_redis.go
index 96fcad1a83..20a50cc1f2 100644
--- a/modules/queue/unique_queue_redis.go
+++ b/modules/queue/unique_queue_redis.go
@@ -62,7 +62,7 @@ func NewRedisUniqueQueue(handle HandlerFunc, cfg, exemplar interface{}) (Queue,
 	return queue, nil
 }
 
-var _ (UniqueByteFIFO) = &RedisUniqueByteFIFO{}
+var _ UniqueByteFIFO = &RedisUniqueByteFIFO{}
 
 // RedisUniqueByteFIFO represents a UniqueByteFIFO formed from a redisClient
 type RedisUniqueByteFIFO struct {
diff --git a/modules/references/references.go b/modules/references/references.go
index c243f25f5d..6c0db0cf47 100644
--- a/modules/references/references.go
+++ b/modules/references/references.go
@@ -296,7 +296,7 @@ func convertFullHTMLReferencesToShortRefs(re *regexp.Regexp, contentBytes *[]byt
 
 		// our new section has length endPos - match[3]
 		// our old section has length match[9] - match[3]
-		(*contentBytes) = (*contentBytes)[:len((*contentBytes))-match[9]+endPos]
+		*contentBytes = (*contentBytes)[:len(*contentBytes)-match[9]+endPos]
 		pos = endPos
 	}
 }
diff --git a/modules/repofiles/action.go b/modules/repofiles/action.go
index 52cc89dbae..d7e3ff4525 100644
--- a/modules/repofiles/action.go
+++ b/modules/repofiles/action.go
@@ -201,7 +201,7 @@ func UpdateIssuesCommit(doer *models.User, repo *models.Repository, commits []*r
 					continue
 				}
 			}
-			close := (ref.Action == references.XRefActionCloses)
+			close := ref.Action == references.XRefActionCloses
 			if close && len(ref.TimeLog) > 0 {
 				if err := issueAddTime(refIssue, doer, c.Timestamp, ref.TimeLog); err != nil {
 					return err
diff --git a/modules/setting/setting.go b/modules/setting/setting.go
index 280987ed66..f609edba17 100644
--- a/modules/setting/setting.go
+++ b/modules/setting/setting.go
@@ -318,7 +318,6 @@ var (
 	LogRootPath        string
 	DisableRouterLog   bool
 	RouterLogLevel     log.Level
-	RouterLogMode      string
 	EnableAccessLog    bool
 	AccessLogTemplate  string
 	EnableXORMLog      bool
@@ -408,10 +407,6 @@ var (
 	IsWindows     bool
 	HasRobotsTxt  bool
 	InternalToken string // internal access token
-
-	// UILocation is the location on the UI, so that we can display the time on UI.
-	// Currently only show the default time.Local, it could be added to app.ini after UI is ready
-	UILocation = time.Local
 )
 
 // IsProd if it's a production mode
diff --git a/modules/storage/helper.go b/modules/storage/helper.go
index 46ab82aed6..4d8ba8e6d9 100644
--- a/modules/storage/helper.go
+++ b/modules/storage/helper.go
@@ -50,7 +50,7 @@ func toConfig(exemplar, cfg interface{}) (interface{}, error) {
 		var err error
 
 		configBytes, err = json.Marshal(cfg)
-		ok = (err == nil)
+		ok = err == nil
 	}
 	if !ok {
 		// no ... we've tried hard enough at this point - throw an error!
diff --git a/modules/storage/storage.go b/modules/storage/storage.go
index 65f8978e5a..984f154db4 100644
--- a/modules/storage/storage.go
+++ b/modules/storage/storage.go
@@ -19,8 +19,6 @@ import (
 var (
 	// ErrURLNotSupported represents url is not supported
 	ErrURLNotSupported = errors.New("url method not supported")
-	// ErrIterateObjectsNotSupported represents IterateObjects not supported
-	ErrIterateObjectsNotSupported = errors.New("iterateObjects method not supported")
 )
 
 // ErrInvalidConfiguration is called when there is invalid configuration for a storage
diff --git a/routers/api/v1/org/team.go b/routers/api/v1/org/team.go
index 99bbd9eefe..bc86bbcbe3 100644
--- a/routers/api/v1/org/team.go
+++ b/routers/api/v1/org/team.go
@@ -660,7 +660,7 @@ func SearchTeam(ctx *context.APIContext) {
 		UserID:      ctx.User.ID,
 		Keyword:     strings.TrimSpace(ctx.Query("q")),
 		OrgID:       ctx.Org.Organization.ID,
-		IncludeDesc: (ctx.Query("include_desc") == "" || ctx.QueryBool("include_desc")),
+		IncludeDesc: ctx.Query("include_desc") == "" || ctx.QueryBool("include_desc"),
 		ListOptions: listOptions,
 	}
 
diff --git a/routers/api/v1/repo/issue.go b/routers/api/v1/repo/issue.go
index 683a2a43b7..6b46dc0fef 100644
--- a/routers/api/v1/repo/issue.go
+++ b/routers/api/v1/repo/issue.go
@@ -141,7 +141,6 @@ func SearchIssues(ctx *context.APIContext) {
 		keyword = ""
 	}
 	var issueIDs []int64
-	var labelIDs []int64
 	if len(keyword) > 0 && len(repoIDs) > 0 {
 		if issueIDs, err = issue_indexer.SearchIssuesByKeyword(repoIDs, keyword); err != nil {
 			ctx.Error(http.StatusInternalServerError, "SearchIssuesByKeyword", err)
@@ -176,7 +175,7 @@ func SearchIssues(ctx *context.APIContext) {
 
 	// Only fetch the issues if we either don't have a keyword or the search returned issues
 	// This would otherwise return all issues if no issues were found by the search.
-	if len(keyword) == 0 || len(issueIDs) > 0 || len(labelIDs) > 0 {
+	if len(keyword) == 0 || len(issueIDs) > 0 || len(includedLabelNames) > 0 {
 		issuesOpt := &models.IssuesOptions{
 			ListOptions: models.ListOptions{
 				Page:     ctx.QueryInt("page"),
@@ -675,7 +674,7 @@ func EditIssue(ctx *context.APIContext) {
 		}
 	}
 	if form.State != nil {
-		issue.IsClosed = (api.StateClosed == api.StateType(*form.State))
+		issue.IsClosed = api.StateClosed == api.StateType(*form.State)
 	}
 	statusChangeComment, titleChanged, err := models.UpdateIssueByAPI(issue, ctx.User)
 	if err != nil {
diff --git a/routers/api/v1/repo/pull.go b/routers/api/v1/repo/pull.go
index 2d16e801db..eff998ee99 100644
--- a/routers/api/v1/repo/pull.go
+++ b/routers/api/v1/repo/pull.go
@@ -580,7 +580,7 @@ func EditPullRequest(ctx *context.APIContext) {
 	}
 
 	if form.State != nil {
-		issue.IsClosed = (api.StateClosed == api.StateType(*form.State))
+		issue.IsClosed = api.StateClosed == api.StateType(*form.State)
 	}
 	statusChangeComment, titleChanged, err := models.UpdateIssueByAPI(issue, ctx.User)
 	if err != nil {
diff --git a/routers/events/events.go b/routers/events/events.go
index 7542f5681a..2c1034038f 100644
--- a/routers/events/events.go
+++ b/routers/events/events.go
@@ -32,10 +32,10 @@ func Events(ctx *context.Context) {
 
 	if !ctx.IsSigned {
 		// Return unauthorized status event
-		event := (&eventsource.Event{
+		event := &eventsource.Event{
 			Name: "close",
 			Data: "unauthorized",
-		})
+		}
 		_, _ = event.WriteTo(ctx)
 		ctx.Resp.Flush()
 		return
@@ -137,10 +137,10 @@ loop:
 					break loop
 				}
 				// Replace the event - we don't want to expose the session ID to the user
-				event = (&eventsource.Event{
+				event = &eventsource.Event{
 					Name: "logout",
 					Data: "elsewhere",
-				})
+				}
 			}
 
 			_, err := event.WriteTo(ctx.Resp)
diff --git a/routers/org/teams.go b/routers/org/teams.go
index 520ded33b4..e612cd767c 100644
--- a/routers/org/teams.go
+++ b/routers/org/teams.go
@@ -193,7 +193,7 @@ func NewTeamPost(ctx *context.Context) {
 	ctx.Data["PageIsOrgTeams"] = true
 	ctx.Data["PageIsOrgTeamsNew"] = true
 	ctx.Data["Units"] = models.Units
-	var includesAllRepositories = (form.RepoAccess == "all")
+	var includesAllRepositories = form.RepoAccess == "all"
 
 	t := &models.Team{
 		OrgID:                   ctx.Org.Organization.ID,
@@ -286,7 +286,7 @@ func EditTeamPost(ctx *context.Context) {
 
 	isAuthChanged := false
 	isIncludeAllChanged := false
-	var includesAllRepositories = (form.RepoAccess == "all")
+	var includesAllRepositories = form.RepoAccess == "all"
 	if !t.IsOwnerTeam() {
 		// Validate permission level.
 		auth := models.ParseAccessMode(form.Permission)
diff --git a/routers/repo/http.go b/routers/repo/http.go
index 0377979e8b..2b79dddbbd 100644
--- a/routers/repo/http.go
+++ b/routers/repo/http.go
@@ -91,7 +91,7 @@ func httpBase(ctx *context.Context) (h *serviceHandler) {
 		strings.HasSuffix(ctx.Req.URL.Path, "git-upload-archive") {
 		isPull = true
 	} else {
-		isPull = (ctx.Req.Method == "GET")
+		isPull = ctx.Req.Method == "GET"
 	}
 
 	var accessMode models.AccessMode
diff --git a/routers/repo/issue.go b/routers/repo/issue.go
index 4475e35f63..da3772ef5a 100644
--- a/routers/repo/issue.go
+++ b/routers/repo/issue.go
@@ -52,8 +52,6 @@ const (
 )
 
 var (
-	// ErrTooManyFiles upload too many files
-	ErrTooManyFiles = errors.New("Maximum number of files to upload exceeded")
 	// IssueTemplateCandidates issue templates
 	IssueTemplateCandidates = []string{
 		"ISSUE_TEMPLATE.md",
diff --git a/routers/repo/setting_protected_branch.go b/routers/repo/setting_protected_branch.go
index c395a394c2..fba2c095cf 100644
--- a/routers/repo/setting_protected_branch.go
+++ b/routers/repo/setting_protected_branch.go
@@ -130,16 +130,16 @@ func SettingsProtectedBranch(c *context.Context) {
 	c.Data["merge_whitelist_users"] = strings.Join(base.Int64sToStrings(protectBranch.MergeWhitelistUserIDs), ",")
 	c.Data["approvals_whitelist_users"] = strings.Join(base.Int64sToStrings(protectBranch.ApprovalsWhitelistUserIDs), ",")
 	contexts, _ := models.FindRepoRecentCommitStatusContexts(c.Repo.Repository.ID, 7*24*time.Hour) // Find last week status check contexts
-	for _, context := range protectBranch.StatusCheckContexts {
+	for _, ctx := range protectBranch.StatusCheckContexts {
 		var found bool
-		for _, ctx := range contexts {
-			if ctx == context {
+		for i := range contexts {
+			if contexts[i] == ctx {
 				found = true
 				break
 			}
 		}
 		if !found {
-			contexts = append(contexts, context)
+			contexts = append(contexts, ctx)
 		}
 	}
 
diff --git a/routers/user/home.go b/routers/user/home.go
index 3436a44bae..431ffdde8e 100644
--- a/routers/user/home.go
+++ b/routers/user/home.go
@@ -770,9 +770,6 @@ func getActiveTeamOrOrgRepoIds(ctxUser *models.User, team *models.Team, unitType
 
 	if team != nil {
 		env = ctxUser.AccessibleTeamReposEnv(team)
-		if err != nil {
-			return nil, fmt.Errorf("AccessibleTeamReposEnv: %v", err)
-		}
 	} else {
 		env, err = ctxUser.AccessibleReposEnv(ctxUser.ID)
 		if err != nil {
diff --git a/services/pull/merge.go b/services/pull/merge.go
index bbe631f68c..518ffa4849 100644
--- a/services/pull/merge.go
+++ b/services/pull/merge.go
@@ -98,7 +98,7 @@ func Merge(pr *models.PullRequest, doer *models.User, baseGitRepo *git.Repositor
 		if err = ref.Issue.LoadRepo(); err != nil {
 			return err
 		}
-		close := (ref.RefAction == references.XRefActionCloses)
+		close := ref.RefAction == references.XRefActionCloses
 		if close != ref.Issue.IsClosed {
 			if err = issue_service.ChangeStatus(ref.Issue, doer, close); err != nil {
 				return err