simsim314/LifeAPI — pull request #1 by ceebo (2014-12-13 18:25:45+00:00)
Title: From chris_c on conwaylife.com

LifeAPI looks great. I hacked around with it a bit. What do you think of these patches?

Patch 1. IMO these headers do not really belong inside LifeAPI.h. Especially comio.h seems not available on Linux.

Patch 2. I hate dealing with strings in C/C++, IMO they should only be used for input/output.

Patch 3. gcc spews a load of warnings unless char * is declared as const

Patch 4. Direct implementations of Contains and ContainsInverse. I don't know if you will like them.

Patch 5. I think these renamings make it more obvious what the function does without needing to read the documenatation. Especially ContainsInverse I think is a tricky name that I would prefer to get rid of. This is probably the most subjective patch I am submitting.

Patch 6. Add function GetBoundary. Completely untested but could be useful to easily defined the "unwanted" set in a LifeTarget.

(Oh I notice you made some changes in the last couple of hours. Hopefully this still applies cleanly?! Maybe not)