class iterator doesn't work with stream_iterator about utfcpp HOT 5 CLOSED

I would like to have a look at it today. I have already spotted multiple calls to operator* moves the actual iterator by one - so if you print(*iter); multiple times and use "abcdef" as your string, you see the difference.

from utfcpp.

The compilation problem is simple: unchecked iterator constructor takes only one argument, not three. Instead of:

for(utf8_iterator iter(it, it, eos)

do

for(utf8_iterator iter(it)

Looking at the rest...

from utfcpp.

unchecked::iterator compiles. checked::iterator doesn't.
The problem comes from "operator<" which doesn't exist for std::istream_iterator

from utfcpp.

Not a bug. The iterator class adapts bi-directional iterators and istream_itrator is an input iterator.

from utfcpp.

The class utf8::unchecked::iterator doesn't say so. See my PR #59 for a bug example.
The utf8::iterator needs a bi-directional iterator because of the range-checking. Remove the range-checking and it will fail.

from utfcpp.