This may seem a little pedantic but I noticed a few places where we passa datum to a macro which treats the datum as a pointer. This works nowbut might not in the future (if, say, Datum were to be 8 bytes).
Thanks,
Gavin