Boost logo

Boost :

Subject: Re: [boost] Proposed date-time patch [ Refs #2475 ]
From: Marshall Clow (mclow.lists_at_[hidden])
Date: 2011-05-03 01:12:48


On May 2, 2011, at 5:10 PM, Steven Watanabe wrote:
> On 05/02/2011 03:31 PM, Marshall Clow wrote:
>> Here's a proposed patch for ticket #2475.
>> https://svn.boost.org/trac/boost/ticket/2475
>> Anyone love it? Hate it?
>>
>>
>> Index: ../../../boost/date_time/tz_db_base.hpp
>> ===================================================================
>> --- ../../../boost/date_time/tz_db_base.hpp (revision 71667)
>> +++ ../../../boost/date_time/tz_db_base.hpp (working copy)
>> @@ -166,22 +166,26 @@
>> //! Constructs an empty database
>> tz_db_base() {}
>>
>> + //! Process csv data stream, may throw exceptions
>> + /*! May throw bad_field_count exceptions */
>> + void load_from_stream(std::ifstream &ifs)
>
> Should't this be std::istream?

D'oh!
Of course it should - thanks.

> Any particular reason you changed the brace style?

I just removed unnecessary braces - force of habit.
I can put them back if people object.

-- Marshall

Marshall Clow Idio Software <mailto:mclow.lists_at_[hidden]>

A.D. 1517: Martin Luther nails his 95 Theses to the church door and is promptly moderated down to (-1, Flamebait).
        -- Yu Suzuki


Boost list run by bdawes at acm.org, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk