lexopt currently uses Box<dyn Iterator> but this is very restrictive. By switching to a vector's iterator we can derive common traits like Clone and get unlimited look-ahead.
This isn't strictly backward-compatible. The iterator is now exhausted when you create the parser, not during parsing. Sufficiently perverse code like this can observe it, and it might be out in the wild, so I'm bumping the version to 0.3.0. Such code can hopefully use RawArgs::as_slice() instead, especially when #15 is implemented.
Parser could implement FromIterator now, and RawArgs could provide mutable access, but I'm not sure those are useful so I left them out.
Both this change and the version bump seem like the right thing to do, but comments welcome.
lexopt currently uses
Box<dyn Iterator>but this is very restrictive. By switching to a vector's iterator we can derive common traits likeCloneand get unlimited look-ahead.This isn't strictly backward-compatible. The iterator is now exhausted when you create the parser, not during parsing. Sufficiently perverse code like this can observe it, and it might be out in the wild, so I'm bumping the version to 0.3.0. Such code can hopefully use
RawArgs::as_slice()instead, especially when #15 is implemented.Parsercould implementFromIteratornow, andRawArgscould provide mutable access, but I'm not sure those are useful so I left them out.Both this change and the version bump seem like the right thing to do, but comments welcome.
cc @nordzilla @tertsdiepraam