Re: [boost] [time_series] Review

From: Eric Niebler (eric_at_[hidden])
Date: 2007-08-08 14:42:54

Michael Fawcett wrote:
> On 8/8/07, Eric Niebler <eric_at_[hidden]> wrote:
>> Steven Watanabe wrote:
>>> time_series_facade.hpp line 280
>>>             this->sout_
>>>                 << " value = " << value << ", "
>>>                 << " offset = " << start << ", "
>>>                 << " end offset = " << stop << '\n';
>>> value shouldn't have a space before it should it?
>>
>> Probably not.
> 
> Nit-picky, but "offset" and "end offset" probably shouldn't have
> spaces before them either since the space is with the comma in the
> previous line, right?

You're right, of course, but this is really just a debug aid. A general 
ostream inserter for time series should be more configurable than this. 
I'm not sure what it should look like exactly, and I'm also not sure if 
this is a problem the time series library really needs to be solving.

-- 
Eric Niebler
Boost Consulting
www.boost-consulting.com
The Astoria Seminar ==> http://www.astoriaseminar.com