change Store.Insert() to take arguments by pointer

This increases memory overhead slightly, but when converting Gorp uses
into Oblast, I saw that there is a legitimate risk of just ignoring the
result value and continuing with the unupdated records. So according to
the design priorities that I myself laid down, I'm sacrificing some
performance for safety of use.

1 files changed, 5 insertions, 1 deletions

diff --git a/TODO.md b/TODO.md
index 57b81a1..f427337 100644
--- a/TODO.md
+++ b/TODO.md
@@ -1,2 +1,6 @@
-- TODO: Store.Insert should take its arguments as pointers; the risk of continuing to use the un-updated records is too great
+<!--
+SPDX-FileCopyrightText: 2026 Stefan Majewsky <majewsky@gmx.net>
+SPDX-License-Identifier: Apache-2.0
+-->
+
 - TODO: consider adding an upsert, e.g. `func (Store[R]) InsertOrUpdate(db Handle, records ...*R) error`, that chooses based on whether any auto fields is non-zero