📝 chore: update some comments or code style

Author: inhere

byteutil/buffer.go

@@ -16,9 +16,7 @@ type Buffer struct {
 }
 
 // NewBuffer instance
-func NewBuffer() *Buffer {
-	return &Buffer{}
-}
+func NewBuffer() *Buffer { return &Buffer{} }
 
 // PrintByte to buffer, ignore error. alias of WriteByte()
 func (b *Buffer) PrintByte(c byte) {
@@ -112,16 +110,10 @@ func (b *Buffer) ResetAndGet() string {
 }
 
 // Close buffer
-func (b *Buffer) Close() error {
-	return b.CloseErr
-}
+func (b *Buffer) Close() error { return b.CloseErr }
 
 // Flush buffer
-func (b *Buffer) Flush() error {
-	return b.FlushErr
-}
+func (b *Buffer) Flush() error { return b.FlushErr }
 
 // Sync anf flush buffer
-func (b *Buffer) Sync() error {
-	return b.SyncErr
-}
+func (b *Buffer) Sync() error { return b.SyncErr }

maputil/alias.go

@@ -3,6 +3,7 @@ package maputil
 import "fmt"
 
 // Aliases implemented a simple string alias map.
+//  - key: alias, value: real name
 type Aliases map[string]string
 
 // AddAlias to the Aliases

mathutil/convert.go

@@ -40,6 +40,11 @@ type ConvOption[T any] struct {
 	// 	- if true: will use real type try convert. default is false
 	//	- NOTE: current T type's ptr is default support.
 	HandlePtr bool
+	// StrictMode for convert value. default is false
+	//
+	// TRUE:
+	//  - to int: string, float will return error
+	StrictMode bool
 	// set custom fallback convert func for not supported type.
 	UserConvFn ToTypeFunc[T]
 }
@@ -309,7 +314,7 @@ func StrIntOr(s string, defVal int) int {
 }
 
 /*************************************************************
- * convert value to int64
+ * region convert to int64
  *************************************************************/
 
 // Int64 convert value to int64, return error on failed

strutil/check.go

@@ -95,10 +95,10 @@ func IContainsOne(s string, subs []string) bool {
 	return false
 }
 
-// ContainsAll substr(s) in the given string. alias of HasAllSubs()
+// ContainsAll given string should contain all substrings. alias of HasAllSubs()
 func ContainsAll(s string, subs []string) bool { return HasAllSubs(s, subs) }
 
-// HasAllSubs all substr in the given string.
+// HasAllSubs given string should contain all substrings
 func HasAllSubs(s string, subs []string) bool {
 	for _, sub := range subs {
 		if !strings.Contains(s, sub) {

strutil/runes.go

@@ -75,8 +75,8 @@ func Utf8Width(s string) int { return RunesWidth([]rune(s)) }
 //	str := "hi,你好"
 //
 //	len(str) // 9
-//	strutil.Utf8Width(str) // 7
-//	len([]rune(str)) = utf8.RuneCountInString(s) // 5
+//	strutil.RunesWidth(str) // 7 一个中文字占两个字符
+//	len([]rune(str)) = utf8.RuneCountInString(s) // 5 按字算
 func RunesWidth(rs []rune) (w int) {
 	if len(rs) == 0 {
 		return

testutil/envmock.go

@@ -72,6 +72,10 @@ func MockOsEnvByText(envText string, fn func()) {
 		if line = strings.TrimSpace(line); line == "" {
 			continue
 		}
+		if line[0] == '#' || strings.HasPrefix(line, "//") {
+			continue
+		}
+
 		nodes := strings.SplitN(line, "=", 2)
 		envKey := strings.TrimSpace(nodes[0])
 
@@ -85,11 +89,6 @@ func MockOsEnvByText(envText string, fn func()) {
 	MockCleanOsEnv(mp, fn)
 }
 
-// MockOsEnv by env map data. alias of MockCleanOsEnv
-func MockOsEnv(mp map[string]string, fn func()) {
-	MockCleanOsEnv(mp, fn)
-}
-
 var envGroupSet = make(map[string]map[string]string)
 
 // SetOsEnvs by map data with a group key. should call RemoveTmpEnvs after tested.
@@ -153,7 +152,10 @@ func RunOnCleanEnv(runFn func()) {
 	RevertOSEnv()
 }
 
-// MockCleanOsEnv by env map data.
+// MockOsEnv by input map data. alias of MockCleanOsEnv
+func MockOsEnv(mp map[string]string, fn func()) { MockCleanOsEnv(mp, fn) }
+
+// MockCleanOsEnv by input env map data.
 //
 // will CLEAR all old ENV data, use given a data map.
 // will RECOVER old ENV after fn run.

testutil/testutil.go

@@ -142,9 +142,7 @@ func SetTimeLocal(tl *time.Location) {
 }
 
 // SetTimeLocalUTC custom time.Local=UTC for testing.
-func SetTimeLocalUTC() {
-	SetTimeLocal(time.UTC)
-}
+func SetTimeLocalUTC() { SetTimeLocal(time.UTC) }
 
 // RestoreTimeLocal restore time.Local
 func RestoreTimeLocal() {