Skip to content

Commit c9119d6

Browse files
committed
Fix lint warnings
1 parent 242f88c commit c9119d6

File tree

1 file changed

+29
-25
lines changed

1 file changed

+29
-25
lines changed

types/inspect.go

Lines changed: 29 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -312,20 +312,24 @@ func (t *Type) Format(s fmt.State, v rune) {
312312
name = pkg + "." + name
313313
}
314314

315+
writeString := func(str string) {
316+
_, _ = s.Write([]byte(str))
317+
}
318+
315319
if t.Opaque() {
316320
if name == "" {
317321
name = fmt.Sprintf("<anon %s>", t.Kind())
318322
}
319323
if t.typ.Kind == coroutinev1.Kind_KIND_POINTER {
320324
name = "*" + name
321325
}
322-
s.Write([]byte(name))
326+
writeString(name)
323327
return
324328
}
325329

326330
verbose := s.Flag('+') || s.Flag('#')
327331
if name != "" && !verbose {
328-
s.Write([]byte(name))
332+
writeString(name)
329333
return
330334
}
331335

@@ -386,7 +390,7 @@ func (t *Type) Format(s fmt.State, v rune) {
386390
default:
387391
result = primitiveKind
388392
}
389-
s.Write([]byte(result))
393+
writeString(result)
390394
return
391395
}
392396

@@ -412,82 +416,82 @@ func (t *Type) Format(s fmt.State, v rune) {
412416
if name != "" {
413417
elemPrefix = fmt.Sprintf("(%s=%s", name, elemPrefix)
414418
}
415-
s.Write([]byte(elemPrefix))
419+
writeString(elemPrefix)
416420
t.Elem().Format(withoutFlags{s}, v)
417421
if name != "" {
418-
s.Write([]byte(")"))
422+
writeString(")")
419423
}
420424
return
421425
}
422426

423427
if name != "" {
424-
s.Write([]byte(fmt.Sprintf("(%s=", name)))
428+
writeString(fmt.Sprintf("(%s=", name))
425429
}
426430
switch t.typ.Kind {
427431
case coroutinev1.Kind_KIND_FUNC:
428-
s.Write([]byte("func("))
432+
writeString("func(")
429433
paramCount := t.NumParam()
430434
for i := 0; i < paramCount; i++ {
431435
if i > 0 {
432-
s.Write([]byte(", "))
436+
writeString(", ")
433437
}
434438
if i == paramCount-1 && t.Variadic() {
435-
s.Write([]byte("..."))
439+
writeString("...")
436440
}
437441
t.Param(i).Format(withoutFlags{s}, v)
438442
}
439-
s.Write([]byte(")"))
443+
writeString(")")
440444
n := t.NumResult()
441445
if n > 0 {
442-
s.Write([]byte(" "))
446+
writeString(" ")
443447
}
444448
if n > 1 {
445-
s.Write([]byte("("))
449+
writeString("(")
446450
}
447451
for i := 0; i < n; i++ {
448452
if i > 0 {
449-
s.Write([]byte(", "))
453+
writeString(", ")
450454
}
451455
t.Result(i).Format(withoutFlags{s}, v)
452456
}
453457
if n > 1 {
454-
s.Write([]byte(")"))
458+
writeString(")")
455459
}
456460
if name != "" {
457-
s.Write([]byte(")"))
461+
writeString(")")
458462
}
459463

460464
case coroutinev1.Kind_KIND_MAP:
461-
s.Write([]byte("map["))
465+
writeString("map[")
462466
t.Key().Format(withoutFlags{s}, v)
463-
s.Write([]byte("]"))
467+
writeString("]")
464468
t.Elem().Format(withoutFlags{s}, v)
465469

466470
case coroutinev1.Kind_KIND_STRUCT:
467471
n := t.NumField()
468472
if n == 0 {
469-
s.Write([]byte("struct{}"))
473+
writeString("struct{}")
470474
} else {
471-
s.Write([]byte("struct{ "))
475+
writeString("struct{ ")
472476
for i := 0; i < n; i++ {
473477
if i > 0 {
474-
s.Write([]byte("; "))
478+
writeString("; ")
475479
}
476480
f := t.Field(i)
477481
if !f.Anonymous() {
478-
s.Write([]byte(f.Name()))
479-
s.Write([]byte(" "))
482+
writeString(f.Name())
483+
writeString(" ")
480484
}
481485
f.Type().Format(withoutFlags{State: s}, v)
482486
}
483-
s.Write([]byte(" }"))
487+
writeString(" }")
484488
}
485489

486490
default:
487-
s.Write([]byte("invalid"))
491+
writeString("invalid")
488492
}
489493
if name != "" {
490-
s.Write([]byte(")"))
494+
writeString(")")
491495
}
492496
}
493497

0 commit comments

Comments
 (0)