Re: [Boost-bugs] [Boost C++ Libraries] #7408: istream_range should not pull istream

Subject: Re: [Boost-bugs] [Boost C++ Libraries] #7408: istream_range should not pull istream
From: Boost C++ Libraries (noreply_at_[hidden])
Date: 2013-06-19 18:18:29


#7408: istream_range should not pull istream
-------------------------------------------------+-------------------------
  Reporter: Christopher Yeleighton | Owner: neilgroves
  <giecrilj@…> | Status: closed
      Type: Bugs | Component: range
 Milestone: To Be Determined | Severity:
   Version: Boost 1.51.0 | Optimization
Resolution: fixed | Keywords:
-------------------------------------------------+-------------------------

Comment (by nathanridge):

 Replying to [comment:6 ne01026@…]:
> Replying to [comment:3 nathanridge]:
> > Fixed in trunk. However, I'm not convinced that this accomplishes
 anything, as istream_range.hpp includes <iterator> to use
 istream_iterator, and <iterator> most likely needs to bring in <istream>
 to properly define istream_iterator (since it's a template class so its
 methods need to be defined in the header). libstdc++'s <iterator>, for
 example, brings in <istream>.
>
> {{{#!c++
> #include <iterator>
> void foo (::std ::wistream & = ::std ::wcin);
> }}}
>
> error: ‘wcin’ is not a member of ‘std’ (good!)

 That would be because 'wcin' is defined in <iostream>, not <istream>, and
 <iterator> doesn't bring in <iostream>, only <istream>. istream_range.hpp
 has never brought in <iostream>, either, so I don't see how this is
 relevant.

-- 
Ticket URL: <https://svn.boost.org/trac/boost/ticket/7408#comment:7>
Boost C++ Libraries <http://www.boost.org/>
Boost provides free peer-reviewed portable C++ source libraries.

This archive was generated by hypermail 2.1.7 : 2017-02-16 18:50:13 UTC