RE: [boost] Reminder: Tokenizer formal review

From: Gary Powell (Gary.Powell_at_[hidden])
Date: 2001-06-01 10:57:27

Ok, one last comment.... ;>

 IMO, you should drop the two member functions "assign" which take a
container, then just grab the .begin(), and .end() iterators.  Tokenizer is
more of a std::algorithm, instead of a container, and by emphasizing that
fact by operating on iterators, makes that clearer.

  But that's just an opinion. (You could go the other way, as we did with
VTL, and store the container, (or a reference to it, depending on the
parameters) In which case tokenizer looks more like a container than an
algorithm.  VTL, has the concept of a range_container, so that  for a range,
you would nest the two containers.  I'm not saying one approach is better
than another, but just you might want to think about it.

  Yours,
 -gary-

gary.powell_at_[hidden]