tangled.org / core
Monorepo for Tangled tangled.org
fork atom

Use rows.Close() consistently #899

merged opened by octet-stream.net targeting master from octet-stream.net/core: thombles/rows-close

rows.Close() was missed in a number of places so we're at the mercy of the GC when those resources get cleaned up.

Labels

None yet.

assignee

None yet.

Participants 2
AT URI
at://did:plc:txurc6ueald5d7462bpvzdby/sh.tangled.repo.pull/3ma7xfkkb7p22
Interdiff #0 → #1
unified split
appview/db/follow.go

This file has not been changed.

appview/db/issues.go

This file has not been changed.

appview/db/language.go

This file has not been changed.

appview/db/profile.go

This file has not been changed.

appview/db/registration.go

This file has not been changed.

appview/db/repos.go

This file has not been changed.

appview/db/star.go

This file has not been changed.

spindle/db/repos.go

This file has not been changed.

History

2 rounds 1 comment
sign up or login to add to the discussion
octet-stream.net submitted #1
diff
2 commits
expand
86f1bd67
appview/db: close rows handles consistently
f88a1147
spindle/db: close rows handles consistently
expand 1 comment
oppi.li

lgtm thank you!

pull request successfully merged
octet-stream.net submitted #0
diff
2 commits
expand
009c0151
appview/db: close rows handles consistently
0f6151f5
spindle/db: close rows handles consistently
expand 0 comments