I'd suggest the same thing as in the other patch, i.e. not having
the large number of identical lines in the iterator.  That's just
my opinion, though.  Rest LGTM. 

Regards
 Robin

Reply via email to