altos/lisp: Make sure memmove only happens once per object. Other GC fixes
authorKeith Packard <keithp@keithp.com>
Sat, 12 Nov 2016 05:11:13 +0000 (21:11 -0800)
committerKeith Packard <keithp@keithp.com>
Mon, 20 Feb 2017 19:16:50 +0000 (11:16 -0800)
commit29c890b4599b3bbdbd09a5915ea68a63f4e0a9ac
tree791966151a186d37ba143d7a56f0f3640add54bd
parentd46ca67f93e9ecbc4d8c051c3fbdead85490b690
altos/lisp: Make sure memmove only happens once per object. Other GC fixes

The memmove may be overlapping, so make sure it happens only once by
just checking whether move_size has been set, rather than looking at
ao_lisp_moving; that doesn't get set when moving a noted cons as that
still needs to be walked at a later time.

Fix up the various looping move functions to all use the same
pattern. Atom was busted.

Signed-off-by: Keith Packard <keithp@keithp.com>
src/lisp/ao_lisp_atom.c
src/lisp/ao_lisp_cons.c
src/lisp/ao_lisp_eval.c
src/lisp/ao_lisp_frame.c
src/lisp/ao_lisp_mem.c