Fix warnings

devlights · devlights · commit 6376dd688a08 · 2023-12-12T16:39:30.000+09:00
diff --git a/examples/basic/bufferop/use_as_writer.go b/examples/basic/bufferop/use_as_writer.go
@@ -7,7 +7,7 @@ import (
 	"github.com/devlights/gomy/output"
 )
 
-// UseAsReader -- bytes.Buffer を io.Writer として利用するサンプルです.
+// UseAsWriter -- bytes.Buffer を io.Writer として利用するサンプルです.
 func UseAsWriter() error {
 	// bytes.Buffer は io.Writer を実装しているので
 	// io.Readerが必要な様々な場面で利用できる
diff --git a/examples/basic/containers/generic/ring/ring.go b/examples/basic/containers/generic/ring/ring.go
@@ -3,7 +3,7 @@ package ring
 
 import "container/ring"
 
-// Ring[T] は、container/ring/Ring のジェネリック版です.
+// Ring は、container/ring/Ring のジェネリック版です.
 //
 // 標準の *ring.Ring と異なり、値の設定は SetValue() で行います。
 type Ring[T any] struct {
diff --git a/examples/basic/fileio/filesystem/listdir.go b/examples/basic/fileio/filesystem/listdir.go
@@ -7,7 +7,7 @@ import (
 	"github.com/devlights/gomy/output"
 )
 
-// ListDir -- os.DirFS() から fs.Glob() 経由で ディレクトリ 内のファイル一覧を出力するサンプルです.
+// Listdir -- os.DirFS() から fs.Glob() 経由で ディレクトリ 内のファイル一覧を出力するサンプルです.
 func Listdir() error {
 	cwd, err := os.Getwd()
 	if err != nil {
diff --git a/examples/basic/fileio/stat/examples.go b/examples/basic/fileio/stat/examples.go
@@ -15,7 +15,7 @@ func NewRegister() mapping.Register {
 }
 
 func (*register) Regist(m mapping.ExampleMapping) {
-	m["fileio_stat_mkdir_removeall"] = StatMkdirRemoveAll
+	m["fileio_stat_mkdir_removeall"] = MkdirRemoveAll
 	m["fileio_stat"] = Stat
-	m["fileio_stat_permission"] = StatPermission
+	m["fileio_stat_permission"] = Permission
 }
diff --git a/examples/basic/fileio/stat/fileio_permission.go b/examples/basic/fileio/stat/fileio_permission.go
@@ -7,8 +7,8 @@ import (
 	"github.com/devlights/gomy/output"
 )
 
-// StatPermission は、ファイルのパーミッションに関するサンプルです。
-func StatPermission() error {
+// Permission は、ファイルのパーミッションに関するサンプルです。
+func Permission() error {
 	const fpath = "examples/basic/fileio/stat/fileio_permission.go"
 
 	absPath, err := filepath.Abs(fpath)
diff --git a/examples/basic/fileio/stat/fileio_stat_mkdir_removeall.go b/examples/basic/fileio/stat/fileio_stat_mkdir_removeall.go
@@ -7,8 +7,8 @@ import (
 	"path/filepath"
 )
 
-// StatMkdirRemoveAll は、ディレクトリの存在確認と作成および削除のサンプルです.
-func StatMkdirRemoveAll() error {
+// MkdirRemoveAll は、ディレクトリの存在確認と作成および削除のサンプルです.
+func MkdirRemoveAll() error {
 	// ディレクトリパスを生成
 	dname := "try-golang-fileio03"
 	dpath := filepath.Join(os.TempDir(), dname)
diff --git a/examples/basic/goroutines/select_nilchan3.go b/examples/basic/goroutines/select_nilchan3.go
@@ -10,7 +10,7 @@ func SelectNilChan3() error {
 		gen = func(out chan<- int) {
 			defer close(out)
 			for i := 0; i < 5; i++ {
-				out <- (i + 1)
+				out <- i + 1
 			}
 		}
 		output = func(done chan<- any, in1, in2 <-chan int) {
diff --git a/examples/basic/ioop/limitread.go b/examples/basic/ioop/limitread.go
@@ -15,20 +15,20 @@ import (
 //   - https://pkg.go.dev/io@go1.19.3#LimitedReader
 func LimitRead() error {
 	const (
-		READ_SIZE = 0x04
-		BUF_SIZE  = 0xff
+		ReadSize = 0x04
+		BufSize  = 0xff
 	)
 
 	var (
 		message     = "hello world"
 		src         = bytes.NewBufferString(message)
-		limitReader = io.LimitReader(src, READ_SIZE)
+		limitReader = io.LimitReader(src, ReadSize)
 	)
 	output.Stdoutf("[LimitReader]", "original: %v\n", message)
 
 	for {
 		var (
-			buf  = make([]byte, BUF_SIZE)
+			buf  = make([]byte, BufSize)
 			size int
 			err  error
 		)
diff --git a/examples/basic/ioop/onebyteread.go b/examples/basic/ioop/onebyteread.go
@@ -15,8 +15,8 @@ import (
 //   - https://pkg.go.dev/io@go1.19.3#LimitedReader
 func OneByteRead() error {
 	const (
-		READ_SIZE = 0x01
-		BUF_SIZE  = 0xff
+		ReadSize = 0x01
+		BufSize  = 0xff
 	)
 
 	var (
@@ -26,8 +26,8 @@ func OneByteRead() error {
 
 	for {
 		var (
-			reader = io.LimitReader(src, READ_SIZE)
-			buf    = make([]byte, BUF_SIZE)
+			reader = io.LimitReader(src, ReadSize)
+			buf    = make([]byte, BufSize)
 			size   int
 			err    error
 		)
diff --git a/examples/basic/jsonop/types/customdate.go b/examples/basic/jsonop/types/customdate.go
@@ -8,7 +8,7 @@ import (
 )
 
 type (
-	// YYYY/MM/DD 形式で json.Marshal/json.Unmarshal するために利用できる構造体です.
+	// YyyyMmDd は、 YYYY/MM/DD 形式で json.Marshal/json.Unmarshal するために利用できる構造体です.
 	YyyyMmDd struct {
 		time.Time
 	}
diff --git a/examples/basic/jsonop/unmarshal_struct.go b/examples/basic/jsonop/unmarshal_struct.go
@@ -7,7 +7,7 @@ import (
 	"github.com/devlights/gomy/output"
 )
 
-// Unmarshal は、json.Unmarshal() を利用した
+// UnmarshalStruct は、json.Unmarshal() を利用したサンプルです.
 func UnmarshalStruct() error {
 	type (
 		V struct {
diff --git a/examples/basic/mutex/usechannel/usechannel.go b/examples/basic/mutex/usechannel/usechannel.go
@@ -18,14 +18,14 @@ var (
 func deposit(wg *sync.WaitGroup, v int) {
 	defer wg.Done()
 
-	balance <- (<-balance + v)
+	balance <- <-balance + v
 	countCh <- struct{}{}
 }
 
 func withdraw(wg *sync.WaitGroup, v int) {
 	defer wg.Done()
 
-	balance <- (<-balance - v)
+	balance <- <-balance - v
 	countCh <- struct{}{}
 }
 
diff --git a/examples/basic/network/split_host_port.go b/examples/basic/network/split_host_port.go
@@ -7,7 +7,7 @@ import (
 	"github.com/devlights/gomy/output"
 )
 
-// SplitJoinPort は、net.SplitHostPort のサンプルです。
+// SplitHostPort は、net.SplitHostPort のサンプルです。
 //
 // 文字列結合で 127.0.0.1:9999 という形を作っても問題はないが
 // IPv6の場合は [::1]:9999 とする必要があるため、net.JoinHostPort() を
diff --git a/examples/basic/streams/using_nopcloser.go b/examples/basic/streams/using_nopcloser.go
@@ -14,15 +14,12 @@ type (
 	}
 )
 
-// Impl check
 var _ io.ReadCloser = (*_readcloserimpl)(nil)
 
-// Impl -- io.Reader#Read
 func (r *_readcloserimpl) Read(p []byte) (int, error) {
 	return r.reader.Read(p)
 }
 
-// Impl -- io.Closer#Close
 func (r *_readcloserimpl) Close() error {
 	output.Stdoutl("[_readcloserimpl]", "Close() called")
 	return nil
diff --git a/examples/basic/strs/using_strings_clone.go b/examples/basic/strs/using_strings_clone.go
@@ -21,49 +21,49 @@ func UsingStringsClone() error {
 	//
 
 	const (
-		NUM_ITEMS = 100
+		NumItems = 100
 	)
 
 	var (
-		l = make([]string, NUM_ITEMS)
+		l = make([]string, NumItems)
 	)
 
 	// 1024バイトのランダム文字列を1000個用意
-	for i := 0; i < NUM_ITEMS; i++ {
+	for i := 0; i < NumItems; i++ {
 		var (
-			c      *exec.Cmd
-			output []byte
-			err    error
+			c   *exec.Cmd
+			o   []byte
+			err error
 		)
 
 		c = exec.Command("/bin/bash", "-c", "openssl rand -base64 1024 | tr -d '\n'")
-		if output, err = c.Output(); err != nil {
+		if o, err = c.Output(); err != nil {
 			return err
 		}
 
-		l[i] = string(output)
+		l[i] = string(o)
 	}
 
 	//
 	// 各文字列の先頭５文字分を識別子として保持しておく仕様があるとする
 	//
 
 	var (
-		store1 = make([]string, NUM_ITEMS)
-		store2 = make([]string, NUM_ITEMS)
+		store1 = make([]string, NumItems)
+		store2 = make([]string, NumItems)
 	)
 
 	// 部分文字列を取り出し、そのまま保持
 	// この場合、元文字列と部分文字列は同じメモリを共有している可能性があるため
 	// 場合によっては、５バイト分だけじゃなく、文字列全部がメモリに残ったままとなる
-	for i := 0; i < NUM_ITEMS; i++ {
+	for i := 0; i < NumItems; i++ {
 		store1[i] = l[i][:5]
 	}
 
 	// 部分文字列を取り出し、クローンしてから保持
 	// Go 1.18 で追加された strings.Clone() を利用することで、新たな割当が行われた状態で
 	// 文字列がクローンされる。なので元文字列全部がメモリに残ることはなくなる
-	for i := 0; i < NUM_ITEMS; i++ {
+	for i := 0; i < NumItems; i++ {
 		store2[i] = strings.Clone(l[i][:5])
 	}
 
diff --git a/examples/basic/syncs/use_map.go b/examples/basic/syncs/use_map.go
@@ -7,7 +7,7 @@ import (
 	"github.com/devlights/try-golang/examples/basic/syncs/maps"
 )
 
-// Map は、sync.Mapの使い方とそれをラップした型の紹介です。
+// UseMap は、sync.Mapの使い方とそれをラップした型の紹介です。
 //
 // # REFERENCES
 //   - https://hjr265.me/blog/synchronization-constructs-in-go-standard-library/
diff --git a/examples/basic/syncs/use_pool.go b/examples/basic/syncs/use_pool.go
@@ -10,8 +10,8 @@ type _PooledObject struct {
 	v int
 }
 
-func NewPooledObject(v int) *_PooledObject {
-	output.Stderrl("[New]", "Call NewPooledObject()")
+func newPooledObject(v int) *_PooledObject {
+	output.Stderrl("[New]", "Call newPooledObject()")
 	return &_PooledObject{v: v}
 }
 
@@ -54,23 +54,23 @@ func UsePool() error {
 	//
 
 	const (
-		NUM_ITEMS = 20
+		NumItems = 20
 	)
 
 	var (
 		pool = sync.Pool{
 			New: func() any {
-				return NewPooledObject(0)
+				return newPooledObject(0)
 			},
 		}
 		ch   = make(chan int)
 		done = make(chan struct{})
 		wg   = sync.WaitGroup{}
 	)
 
-	wg.Add(NUM_ITEMS)
+	wg.Add(NumItems)
 
-	for i := 0; i < NUM_ITEMS; i++ {
+	for i := 0; i < NumItems; i++ {
 		go func(i int, ch chan<- int) {
 			defer wg.Done()
 
diff --git a/examples/basic/times/time_format_timeonly.go b/examples/basic/times/time_format_timeonly.go
@@ -6,7 +6,7 @@ import (
 	"github.com/devlights/gomy/output"
 )
 
-// FormatDateTime は、Go1.20で追加された time.TimeOnly フォーマット書式についてのサンプルです.
+// FormatTimeOnly は、Go1.20で追加された time.TimeOnly フォーマット書式についてのサンプルです.
 //
 // # REFERENCES
 //   - https://pkg.go.dev/time@go1.20.2#pkg-constants
diff --git a/examples/basic/times/time_truncate_hours.go b/examples/basic/times/time_truncate_hours.go
@@ -7,7 +7,7 @@ import (
 	"github.com/devlights/gomy/times"
 )
 
-// TruncateDay -- time.Timeから時刻部分を除去するサンプルです.
+// TruncateHours -- time.Timeから時刻部分を除去するサンプルです.
 //
 // 時刻部分を除去する場合は、 time.Truncate() に 24時間 を指定する.
 //
diff --git a/examples/basic/types/define_types_easy.go b/examples/basic/types/define_types_easy.go
@@ -8,12 +8,9 @@ import (
 
 // 型定義
 type (
-	// ID
-	Id string
-	// 名前
+	Id   string
 	Name string
 
-	// ユーザ
 	User struct {
 		Id   Id
 		Name Name
diff --git a/examples/effectivego/effectivego21/effectivego_21_interface_conversion.go b/examples/effectivego/effectivego21/effectivego_21_interface_conversion.go
@@ -12,8 +12,8 @@ type (
 	}
 )
 
-// NewMyString -- 新しい myString を生成して返します.
-func NewMyString() *myString {
+// newMyString -- 新しい myString を生成して返します.
+func newMyString() *myString {
 	return &myString{}
 }
 
@@ -35,7 +35,7 @@ func InterfaceConversion() error {
 		  - キャストではない
 		  - 対象の値がインターフェースの場合のみ有効。構造体ではできない
 	*/
-	s1 := NewMyString()
+	s1 := newMyString()
 	s1.Set("helloworld")
 
 	// -------------------------------------------------------------------------
diff --git a/examples/singleapp/flag_pkg/unittest/args_test.go b/examples/singleapp/flag_pkg/unittest/args_test.go
@@ -17,10 +17,10 @@ func TestArgs(t *testing.T) {
 		{[]string{"-a"}, args.Error},
 		{[]string{"-a", "-b"}, args.Error},
 		{[]string{"-a", "-b", "-c"}, args.Error},
-		{[]string{"-a", "1"}, &args.Value{1, "", false}},
-		{[]string{"-a", "1", "-b", "hello"}, &args.Value{1, "hello", false}},
-		{[]string{"-a", "1", "-b", "hello", "-c"}, &args.Value{1, "hello", true}},
-		{[]string{"-a", "1", "-b", "hello", "-c", "true"}, &args.Value{1, "hello", true}},
+		{[]string{"-a", "1"}, &args.Value{A: 1}},
+		{[]string{"-a", "1", "-b", "hello"}, &args.Value{A: 1, B: "hello"}},
+		{[]string{"-a", "1", "-b", "hello", "-c"}, &args.Value{A: 1, B: "hello", C: true}},
+		{[]string{"-a", "1", "-b", "hello", "-c", "true"}, &args.Value{A: 1, B: "hello", C: true}},
 		{[]string{"-a", "bbb", "-b", "hello", "-c"}, args.Error},
 		{[]string{"-a", "bbb", "-b", "hello", "-c", "world"}, args.Error},
 	}
diff --git a/examples/singleapp/mod_operator_performance/main_test.go b/examples/singleapp/mod_operator_performance/main_test.go
@@ -16,33 +16,33 @@ func BenchmarkModOperatorUInt32(b *testing.B) {
 }
 
 func getPrimesInt(limit int) []int {
-	primes := []int{}
+	var primes []int
 	for x := 2; x < limit; x++ {
-		is_prime := true
+		isPrime := true
 		for y := 2; y < x; y++ {
 			if x%y == 0 {
-				is_prime = false
+				isPrime = false
 				break
 			}
 		}
-		if is_prime {
+		if isPrime {
 			primes = append(primes, x)
 		}
 	}
 	return primes
 }
 
 func getPrimesUInt32(limit uint32) []uint32 {
-	primes := []uint32{}
+	var primes []uint32
 	for x := uint32(2); x < limit; x++ {
-		is_prime := true
+		isPrime := true
 		for y := uint32(2); y < x; y++ {
 			if x%y == 0 {
-				is_prime = false
+				isPrime = false
 				break
 			}
 		}
-		if is_prime {
+		if isPrime {
 			primes = append(primes, x)
 		}
 	}
diff --git a/examples/singleapp/option_pattern/1/config/config.go b/examples/singleapp/option_pattern/1/config/config.go
diff --git a/examples/singleapp/pprof_thread_count/main.go b/examples/singleapp/pprof_thread_count/main.go
diff --git a/examples/singleapp/using_clone_method/main.go b/examples/singleapp/using_clone_method/main.go

Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,7 @@ import (`
`7`	`7`	`"github.com/devlights/gomy/output"`
`8`	`8`	`)`
`9`	`9`
`10`		`-// UseAsReader -- bytes.Buffer を io.Writer として利用するサンプルです.`
	`10`	`+// UseAsWriter -- bytes.Buffer を io.Writer として利用するサンプルです.`
`11`	`11`	`func UseAsWriter() error {`
`12`	`12`	`// bytes.Buffer は io.Writer を実装しているので`
`13`	`13`	`// io.Readerが必要な様々な場面で利用できる`
Original file line number	Diff line number	Diff line change
`@@ -15,7 +15,7 @@ func NewRegister() mapping.Register {`
`15`	`15`	`}`
`16`	`16`
`17`	`17`	`func (*register) Regist(m mapping.ExampleMapping) {`
`18`		`- m["fileio_stat_mkdir_removeall"] = StatMkdirRemoveAll`
	`18`	`+ m["fileio_stat_mkdir_removeall"] = MkdirRemoveAll`
`19`	`19`	`m["fileio_stat"] = Stat`
`20`		`- m["fileio_stat_permission"] = StatPermission`
	`20`	`+ m["fileio_stat_permission"] = Permission`
`21`	`21`	`}`
Original file line number	Diff line number	Diff line change
`@@ -7,8 +7,8 @@ import (`
`7`	`7`	`"github.com/devlights/gomy/output"`
`8`	`8`	`)`
`9`	`9`
`10`		`-// StatPermission は、ファイルのパーミッションに関するサンプルです。`
`11`		`-func StatPermission() error {`
	`10`	`+// Permission は、ファイルのパーミッションに関するサンプルです。`
	`11`	`+func Permission() error {`
`12`	`12`	`const fpath = "examples/basic/fileio/stat/fileio_permission.go"`
`13`	`13`
`14`	`14`	`absPath, err := filepath.Abs(fpath)`
Original file line number	Diff line number	Diff line change
`@@ -10,7 +10,7 @@ func SelectNilChan3() error {`
`10`	`10`	`gen = func(out chan<- int) {`
`11`	`11`	`defer close(out)`
`12`	`12`	`for i := 0; i < 5; i++ {`
`13`		`- out <- (i + 1)`
	`13`	`+ out <- i + 1`
`14`	`14`	`}`
`15`	`15`	`}`
`16`	`16`	`output = func(done chan<- any, in1, in2 <-chan int) {`
Original file line number	Diff line number	Diff line change
`@@ -8,7 +8,7 @@ import (`
`8`	`8`	`)`
`9`	`9`
`10`	`10`	`type (`
`11`		`- // YYYY/MM/DD 形式で json.Marshal/json.Unmarshal するために利用できる構造体です.`
	`11`	`+ // YyyyMmDd は、 YYYY/MM/DD 形式で json.Marshal/json.Unmarshal するために利用できる構造体です.`
`12`	`12`	`YyyyMmDd struct {`
`13`	`13`	`time.Time`
`14`	`14`	`}`