On Thu, Sep 7, 2023 at 8:49 PM Thomas Munro <thomas.mu...@gmail.com> wrote: > The code moved around quite a few times over several commits and quite > a lot since then, which is why I didn't go for straight revert, but > clearly the manual approach risked missing things.
It's not a big deal, obviously. > I think the > attached removes all unused 'snapshot' arguments from AM-internal > functions. Checked by compiling with clang's -Wunused-parameters, and > then searching for 'snapshot', and excluding the expected cases. This looks right to me. Thanks -- Peter Geoghegan