comprehensive-rust
comprehensive-rust copied to clipboard
Simplify the solution to prefix matching
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
View this failed invocation of the CLA check for more information.
For the most up to date status, view the checks section at the bottom of the pull request.
Hi @zhouwfang, thanks a lot! This looks much better indeed :smile:
I would be happy to merge this, we just need the CLA to be signed.
Hi @mgeisler, It should be all set now. Thanks.
Thanks! Could I also get you to add a few comments to explain the solution? In particular the trick with turning the elements into Option so we can detect when the prefix iterator ends.