Boost logo

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