Sam,

I noticed your sequences.extras vocab. It looks like it does some
useful things that aren't in the sequences vocab, but some things
duplicate existing code. For example, reduce* is like [ ] swap
map-reduce and find-all is very similar to <enum> swap assoc-filter.
Also, reduce-r could be implemented more efficiently if you used
slices rather than subsequences of the input sequence; right now, it
takes quadratic time but it could be linear. It could use constant
stack space, rather than linear, if you implemented this by just
iterating over the sequence from back to front (this would remove the
need for slices). Are you interested in making these kinds of
cleanups?

It would be nice to not have a vocab called sequences.extras;
something with a name like this is a poorly-defined grab bag, and
things would be more clean if they were sorted into vocabularies that
had more meaningful names. There used to be a vocab called
sequences.lib but this was reorganized and deleted for that same
reason.

Dan

------------------------------------------------------------------------------
Come build with us! The BlackBerry&reg; Developer Conference in SF, CA
is the only developer event you need to attend this year. Jumpstart your
developing skills, take BlackBerry mobile applications to market and stay 
ahead of the curve. Join us from November 9&#45;12, 2009. Register now&#33;
http://p.sf.net/sfu/devconf
_______________________________________________
Factor-talk mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/factor-talk

Reply via email to