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

_______________________________________________
Unsubscribe & other changes: http://lists.boost.org/mailman/listinfo.cgi/boost

Reply via email to