lmi
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [lmi] [PATCH] C++ m11n: range-based for loops


From: Greg Chicares
Subject: Re: [lmi] [PATCH] C++ m11n: range-based for loops
Date: Fri, 13 Jan 2017 16:46:06 +0000
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Icedove/45.5.1

On 2017-01-13 16:31, Greg Chicares wrote:
> On 2017-01-13 15:51, Greg Chicares wrote:
> [... after applying both patches in pull/52 ...]
>> /opt/lmi/src/lmi[0]$make $coefficiency cgi_tests cli_tests              
>> Test common gateway interface:
>> make[2]: warning: -jN forced in submake: disabling jobserver mode.
>> Test command line interface:
>> boost::filesystem::is_directory: "opt": File not found.
>>   172 errors
> 
> Reverting the change to 'ce_product_name.cpp' (copied below) fixes that.
> 
> I guess boost::filestream::directory_iterator must behave like standard
> stream iterators, e.g.:
> 
>     std::istream_iterator<std::string> i(iss);
>     std::istream_iterator<std::string> const eos;
>     for(; i != eos; ++i)

Nothing here seems really satisfying:
  
http://stackoverflow.com/questions/13037490/range-based-loop-over-an-input-stream

This looks more promising:
  http://ericniebler.com/2013/11/07/input-iterators-vs-input-ranges/
| The solution to istream_iterator‘s woes will be to replace it with 
istream_range.
so I think we'll just leave these not-a-container::iterators alone
until he gets that standardized in C++23, and std::filesystem::directory_range
in some later standard.




reply via email to

[Prev in Thread] Current Thread [Next in Thread]