You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Right now this API takes IntoIterator<Item = &'a str> and it's a pain to turn IntoIterator<Item = String> into the former. Googling suggests the solution is accepting IntoIterator<Item = AsRef<str>>, which should be backwards compatible?
The text was updated successfully, but these errors were encountered:
Unfortunately this is harder than it sounds — we specifically need to get string references with a 'a lifetime in the methods in question so that we don't have to re-allocate and copy every string argument we're given. Notice how the code also uses 'a for the other arguments here:
It would be nice to make the API work in such a way that the whole input doesn't need to be kept allocated across the runtime of the analysis, but it would require some deeper changes I think.
Right now this API takes
IntoIterator<Item = &'a str>
and it's a pain to turnIntoIterator<Item = String>
into the former. Googling suggests the solution is acceptingIntoIterator<Item = AsRef<str>>
, which should be backwards compatible?The text was updated successfully, but these errors were encountered: