|
Boost : |
From: Daryle Walker (dwalker07_at_[hidden])
Date: 2003-07-19 02:37:51
In the latest post-review version, I fully disabled copying for the
array-based stream-buffer class. I also copied the revised version of
the library to
http://groups.yahoo.com/group/boost/files/more_io_revised.zip
so you don't need access to the Sandbox CVS to see the altered files.
This is the first time I've taken a look at the reviewed version and
the post-review version. The (public) changes are:
array_stream.hpp
* made basic_array_streambuf non-copyable
* completed the standard stream-buffer typedef's
* changed the default constructor to optionally take an opening mode
* added pointer & iterator constructors
* added accessors for the character read/write count and the opening
mode
* activated the stream-buffer methods to gain random-access
* packed the supporting stream templates in a big macro
iomanip.hpp
* did an "int" vs. "std::streamsize" fix
streambuf_wrapping.hpp
* changed the way to save repetition from using a common virtual base
class to using macros
array_stream.html
* added documentation for the changed parts of the *.hpp file and
expanded rationale and supporting-stream areas
iomanip.html
* expanded rationale
* minor corrections & style changes
streambuf_wrapping.html
* added example
* expanded rationale
* [noticed that I missed some changes from the *.hpp file because the
blocked-off areas from my diff-tool were so large I skimmed over them]
* completed the standard stream typedef's
* renamed the accessors for the contained stream-buffer
array_stream_test.cpp
iomanip_test.cpp
* completely changed files, transitioned from old test tools to unit
tests
Actually, I don't think the Review Manager ever gave a final answer.
Well, he and everyone else can take a quick look. It seems that some
people around here are jumpy for a release, so we should get this
review resolved soon.
Daryle
Boost list run by bdawes at acm.org, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk