Le 25/08/2026 à 13:51, Paul Richard Thomas a écrit :
Hi Mikael,

You beat me to it - thanks!

I was going to make the following remark about patch #1:

I couldn't help but wonder whether or not we might, one day, need deep
copy in/copy out? I cannot imagine for what it would be needed but, if
the answer to the question is affirmative should we implement it now
and eliminate the assertion?

Well, with the current state of affairs, the assertion seems to be safe, that is it does not trigger. This means the copy out implementation would be dead code. Given how complicated the function already is, I would rather not add dead, untested code to it.
I'm fine with just the assertion and the explaining comment.  :-)

Reply via email to