[PEAK] does this look okay?

Phillip J. Eby pje at telecommunity.com
Sun Dec 7 13:59:42 EST 2003


At 12:31 PM 12/7/03 -0600, darryl wrote:
>pje,
>
>before i go any further does this still look okay?
>
>http://peak.telecommunity.com/DevCenter/ShortIntroPeakSecurity

Looks correct so far.  Go for it.

I would suggest, however, that you follow PEAK's standards for code 
indentation, as some of your example code is difficult to read.  E.g.:

    21     rules = Items(
    22           checkCanBurn = [Owner.of(Building)]
    23                   )

should look like this:

    21     rules = Items(
    22         checkCanBurn = [Owner.of(Building)]
    23     )

i.e. four-space indents always, and closing parens/brackets align with the 
indent of the line with the opening bracket.

Also, you seem to have an extra blank line at the top of your code listings.

Anyway, the content itself looks fine so far.




More information about the PEAK mailing list