On Sun, Nov 23, 2008 at 2:19 AM, Eduardo Cavazos <[EMAIL PROTECTED]> wrote:
> :: [deep-cleave-quots] ( SEQ -- quot )
>
>  [let | QUOTS  [ SEQ [ quotation?                       ] deep-filter ]
>         RECIPE [ SEQ [ dup quotation? [ drop , ] [ ] if ] deep-map    ] |
>
>    [ QUOTS cleave RECIPE bake ] ] ;
>
> MACRO: deep-cleave-quots ( seq -- quot ) [deep-cleave-quots] ;

I find this code hard to read. The unnecessary word definition and
white space is distracting ,and I find uppercase local names are hard
on the eyes when they're interspersed with the body like that. Also I
think 'when' conveys more meaning than '[ ] if'.

Slava

-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
Factor-talk mailing list
Factor-talk@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/factor-talk

Reply via email to