Page 548, FIG-16.2 has the following line:
SetOrientationAllowed((r&&%1000)>>3), (r&&100)>>2,(r&&10)>>1,r&&1)
I believe it should read
SetOrientationAllowed((r&&%1000)>>3), (r&&%100)>>2,(r&&%10)>>1,r&&%1)
(The last "%" is not necessary, but I would suggest to put it for consistency)
On a related note, the same code in FIG-16.2 assumes that 'color' is an integer of the form 0x??????, where the first pair ?? stand for red, the second pair ?? for green, and the third pair ?? for blue. I have done some testing, and it seems that the code is not compiled in this order with some functions like
DrawBox, so use rather the
MakeColor function to be safe.
Pages 628 through 630: I suggest you replace every "
CARDWIDTH" by "
CARDHEIGHT". This would make the code consistent with the name of this declared constant.