--- Log opened Tue Aug 19 00:00:15 2025
00:53 -!- kinlo [~peter@user/kinlo] has quit [Server closed connection]
00:53 -!- kinlo [~peter@user/kinlo] has joined #bitcoin-kernel
01:10 -!- Anduck [~anduck@user/Anduck] has quit [Ping timeout: 245 seconds]
01:10 -!- Anduck [~anduck@user/Anduck] has joined #bitcoin-kernel
03:25 -!- achow101 [~achow101@user/achow101] has quit [Server closed connection]
03:25 -!- achow101 [~achow101@user/achow101] has joined #bitcoin-kernel
10:56 -!- yuvicc [~yuvicc@user/yuvicc] has joined #bitcoin-kernel
11:02 -!- yuvicc [~yuvicc@user/yuvicc] has quit [Quit: yuvicc]
14:04 < stickies-v> fyi - I've wrapped up (for now) my work on implementing the ownership semantics I described earlier (https://gist.github.com/stickies-v/db3766418a0c92ba1118ccedcbc3e68e, gist updated to reflect some minor architecture changes)
14:04 < stickies-v> you can see the changes with the HEAD (which is now outdated) that I based it on via https://github.com/TheCharlatan/bitcoin/compare/39c2c5afd75e5d455ac2699dcc1c65728e1a5bc5...stickies-v:bitcoin:kernel/ownership-semantics
14:05 < TheCharlatan> nice, looking forward to reading through that.
14:06 < stickies-v> I think the best starting point to look at the changes is by looking at `bitcoinkernel_wrapper.h`, which is by far the most structured, and then work your way back from there into the `bitcoinkernel.h` and `bitcoinkernel.cpp` changes
14:08 < stickies-v> note: I really like purpeKarrot's approach in https://github.com/purpleKarrot/btck/blob/e5ea925d5cd588a60cd726e59ee17f55903f692f/src/util/type_mapping.hpp, and it would work really well with my changes, but I did not implement them yet
14:09 < stickies-v> it's a big diff - feel free to ping me with questions before spending too much time digging deep
14:13 -!- purpleKarrot [~purpleKar@2001:1620:5547:0:fafe:5eff:fe5b:42e9] has joined #bitcoin-kernel
14:22 < stickies-v> a couple of extra notes:
14:22 < stickies-v> 1) I implemented ownership semantics for a few (not all) types (btck_BlockIndex, btck_Coin, btck_ScriptPubkey, btck_TransactionOutput, btck_BlockSpentOutputs, btck_TransactionSpentOutputs) as per https://gist.github.com/stickies-v/db3766418a0c92ba1118ccedcbc3e68e
14:23 < stickies-v> 2) removed `btck_BlockPointer`, is now also just a `btck_Block` (but with #33078, the callback can be promoted to a `btck_BlockHandle`, so that's still desirable imo)
14:23 < stickies-v> 3) implemented both `btck_get_handle_block_spent_outputs` and `btck_get_block_spent_outputs` to show how this approach gives us flexibility in exposing multiple ways of doing things, based on actual need
14:24 < stickies-v> thank you for your attention to this matter, i will now stop my monologue - here if you need me!
15:36 -!- purpleKarrot [~purpleKar@2001:1620:5547:0:fafe:5eff:fe5b:42e9] has quit [Quit: purpleKarrot]
22:20 -!- purpleKarrot [~purpleKar@2001:1620:5547:0:fafe:5eff:fe5b:42e9] has joined #bitcoin-kernel
--- Log closed Wed Aug 20 00:00:17 2025