The inner scope re-declared `var err error` immediately before assigning from `io.ReadAll`, unnecessarily shadowing the `err` already in scope from `reader.ReadByte()`. Removing the shadow simplifies the control flow and eliminates the redundant declaration (AX Principle 1 — names should not introduce unnecessary cognitive overhead). Co-Authored-By: Charon <charon@lethean.io> |
||
|---|---|---|
| .. | ||
| packet.go | ||
| packet_test.go | ||
| reader.go | ||
| reader_test.go | ||