File: gocritic.go

package info (click to toggle)
gitaly 16.11.6%2Bds1-5
  • links: PTS, VCS
  • area: main
  • in suites: forky, sid
  • size: 1,623,960 kB
  • sloc: cpp: 33,732; java: 20,579; sh: 5,372; cs: 3,973; javascript: 3,426; python: 3,053; makefile: 2,303; ansic: 1,729; php: 1,455; asm: 1,217; xml: 838; ruby: 452; sql: 431; yacc: 22; sed: 1
file content (48 lines) | stat: -rw-r--r-- 1,092 bytes parent folder | download | duplicates (2)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
//golangcitest:args -Egocritic
//golangcitest:config_path testdata/configs/gocritic.yml
package testdata

import (
	"flag"
	"log"
	"strings"
)

var _ = *flag.Bool("global1", false, "") // want `flagDeref: immediate deref in \*flag.Bool\(.global1., false, ..\) is most likely an error; consider using flag\.BoolVar`

type size1 struct {
	a bool
}

type size2 struct {
	size1
	b bool
}

func gocriticRangeValCopySize1(ss []size1) {
	for _, s := range ss {
		log.Print(s)
	}
}

func gocriticRangeValCopySize2(ss []size2) {
	for _, s := range ss { // want "rangeValCopy: each iteration copies 2 bytes.*"
		log.Print(s)
	}
}

func gocriticStringSimplify() {
	s := "Most of the time, travellers worry about their luggage."
	s = strings.Replace(s, ",", "", -1) // want "ruleguard: this Replace call can be simplified.*"
	log.Print(s)
}

func gocriticDup(x bool) {
	if x && x { // want "ruleguard: suspicious identical LHS and RHS.*"
		log.Print("x is true")
	}
}

func gocriticRuleWrapperFunc() {
	strings.Replace("abcabc", "a", "d", -1) // want "ruleguard: this Replace call can be simplified.*"
}