>>>>> "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

Reply via email to