>>>>> "Rich" == Richard Lowe <richlowe at richlowe.net> writes:
>> - in cdm_pbchk(), if there are any guidelines about the order that >> the checks are done in, it would be good to write them down in the >> comments. Rich> I'm not actually sure, honestly. I think I prefer problems with Rich> your code, then problems with your workspace, but I can't back Rich> that up with reasoning. (I think the original ordering may have Rich> been taken from wx) Well, you could always add a comment like # The current order puts the code checks first, followed by the # workspace checks, but that's not set in stone. (Up to you if you want to.) mike