From: Colin A. <col...@go...> - 2008-04-24 12:25:09
|
On 24/04/2008, Daniel Tuser <dan...@gm...> wrote: > Colin Adams wrote: > I know that the indexing clause does not conform to the Gobo styling > standards so far. And the class name should be in the same line as the class > keyword. Those are the two non-conforming things I know. Did you find more? There area a load of empty feature clauses and a dummy invariant in at least one class. Then I dislike the absence of a blank line after create and inherit. Also I dislike seeing local variables with names that might class with attributes (and so forcing unnecessary renames in descendants). I personally now write all argument names as a_*, and all local names as either a single chanaracter or l_*. I am converting my existing code on a piecemeal basis. |