enbility / eebus-go

93%

LAST BUILD ON BRANCH dev

branch: SELECT

CHANGE BRANCH

x

• No branch selected
• 188-update-controlboxhems-examples-positive_failsafe_limits
• 188-update-controlboxhems-examples-to-write-negative-appl-values
• bugfix/remote-uc-removal
• chore/bump-spine-go-to-prepare-for-releases
• chore/redundant-nil-checks
• dev
• docs/testspec-coverage-comments
• feat/ca-cdsf
• feat/ca-crcsf
• feat/ca-crct
• feat/ca-crhsf
• feat/ca-crht
• feat/consistent-write-operation-gate
• feat/hvac-setpoint-features
• feat/implement-mu-mpc
• feat/ma-mdsf
• feat/ma-mdt
• feat/ma-mrcsf
• feat/ma-mrhsf
• feat/mot-usecase
• feat/mrt-usecase
• feat/rebased-mu-mpc-mgcp
• feat/va-vhan
• feature/binding-handling
• feature/examples-clarification
• feature/improve-data
• feature/improve-disconnect-handling
• feature/improved-register-handling
• feature/jsonrpc2
• feature/minor-maintenance
• feature/mu-mpc--gcp-mgcp--smartmeter-demo
• feature/shippairing-integration
• fix/cs-lpc-lpp-0-duration
• fix/eg-lpc-electricalconnection-sync
• fix/eg-lpc-lpp-missing-constraints-setup
• fix/event-firing-consistency
• fix/gcp-mgcp-valid-actor-types
• fix/ignore-usecaseavailable-eg
• fix/inactive-limit-without-value
• fix/loadcontrol-phase-parameter-description
• fix/lpc-lpp-dependencies
• fix/missing-pointer-nil-checks
• fix/ohpcf-register-event-handler
• fix/ohpcf-sequence-id
• fix/opev-oscev-heartbeat
• fix/optional-function-not-supported
• fix/persist-full-write-upstream
• fix/regenerate-mocks
• fix/report-per-phase-measurements-for-correct-phase
• fix/use-correct-eg-lpc-lpp-characteristic-types
• implement-cem-ohpcf
• implement-gcp-mgcp
• main
• ohpcf-msgcounter
• patch-3
• patch-5
• push-zxvruxpyxtuv
• result-callback-with-message-counter
• test/lpp-partial-write-parity
• wip/failsafe-limit-write-approval

Build # 36748858296

Build Type

push

github

Committed by web-flow

Commit Message

chore: remove redundant nil checks (#258)

`len()` is defined on nil slices and returns `0`, so the paired `== nil`
check is redundant. Likewise, ranging over a nil slice is a no-op, so
the guard around it adds nothing.

- `features/internal/deviceconfiguration.go` — `descriptions == nil ||
len(descriptions) == 0`
- `usecases/cs/lpc/usecase.go` — `data.LoadControlLimitData == nil ||
len(data.LoadControlLimitData) == 0`
- `usecases/cs/lpp/usecase.go` — same
- `features/internal/electricalconnection.go` — `if set.Range != nil {
for _, rangeItem := range set.Range {` guard removed

The outer `data == nil` checks are kept — those guard the pointer
dereference.

Found with an AST pass over the whole repo rather than a text search, so
the set is complete for these shapes: `x == nil || len(x) == 0` and `x
!= nil && len(x) > 0` in either operand order and including the `< 1` /
`<= 0` / `!= 0` / `>= 1` spellings, plus nil guards wrapping a `range`
or `append` on the same value.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Coverage Stats

16 of 16 new or added lines in 10 files covered. (100.0%)

9272 of 9996 relevant lines covered (92.76%)

13.94 hits per line