[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.
- Re: [lmi] [PATCH] C++ m11n: range-based for loops, (continued)
- Re: [lmi] [PATCH] C++ m11n: range-based for loops, Greg Chicares, 2017/01/13
- Re: [lmi] [PATCH] C++ m11n: range-based for loops, Greg Chicares, 2017/01/13
- Re: [lmi] [PATCH] C++ m11n: range-based for loops, Vadim Zeitlin, 2017/01/13
- Re: [lmi] [PATCH] C++ m11n: range-based for loops, Greg Chicares, 2017/01/13
- Re: [lmi] [PATCH] C++ m11n: range-based for loops, Greg Chicares, 2017/01/13
- Re: [lmi] [PATCH] C++ m11n: range-based for loops, Vadim Zeitlin, 2017/01/13
- Re: [lmi] [PATCH] C++ m11n: range-based for loops, Vadim Zeitlin, 2017/01/13
- [lmi] Improving usability of automated tests [Was: [PATCH] C++ m11n: range-based for loops], Greg Chicares, 2017/01/13
- Re: [lmi] Improving usability of automated tests, Vadim Zeitlin, 2017/01/13
- Re: [lmi] [PATCH] C++ m11n: range-based for loops, Greg Chicares, 2017/01/13
- Re: [lmi] [PATCH] C++ m11n: range-based for loops,
Greg Chicares <=
Re: [lmi] [PATCH] C++ m11n: range-based for loops, Greg Chicares, 2017/01/13
Re: [lmi] [PATCH] C++ m11n: range-based for loops, Greg Chicares, 2017/01/14
- Re: [lmi] [PATCH] C++ m11n: range-based for loops, Vadim Zeitlin, 2017/01/15
- Re: [lmi] [PATCH] C++ m11n: range-based for loops, Greg Chicares, 2017/01/15
- Re: [lmi] [PATCH] C++ m11n: range-based for loops, Vadim Zeitlin, 2017/01/15
- Re: [lmi] [PATCH] C++ m11n: range-based for loops, Greg Chicares, 2017/01/16
- Re: [lmi] [PATCH] C++ m11n: range-based for loops, Vadim Zeitlin, 2017/01/16
- Re: [lmi] [PATCH] C++ m11n: range-based for loops, Greg Chicares, 2017/01/16
- Re: [lmi] [PATCH] C++ m11n: range-based for loops, Vadim Zeitlin, 2017/01/17