From b8bf57a8a133e4d68cc4a5c3044ec67d6f0f11d2 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jan=20Christian=20Gr=C3=BCnhage?= Date: Thu, 7 Jul 2022 07:37:15 +0200 Subject: [PATCH] Add review for yap v0.8.0 --- .../reviews/2022-07-package-0Oh57g.proof.crev | 29 +++++++++++++++++++ 1 file changed, 29 insertions(+) create mode 100644 YdnEoYtqvbBGv0hhENLDUYmc3tNfm5V5NIG5hCovHyM/reviews/2022-07-package-0Oh57g.proof.crev diff --git a/YdnEoYtqvbBGv0hhENLDUYmc3tNfm5V5NIG5hCovHyM/reviews/2022-07-package-0Oh57g.proof.crev b/YdnEoYtqvbBGv0hhENLDUYmc3tNfm5V5NIG5hCovHyM/reviews/2022-07-package-0Oh57g.proof.crev new file mode 100644 index 0000000..9e26435 --- /dev/null +++ b/YdnEoYtqvbBGv0hhENLDUYmc3tNfm5V5NIG5hCovHyM/reviews/2022-07-package-0Oh57g.proof.crev @@ -0,0 +1,29 @@ +----- BEGIN CREV PROOF ----- +kind: package review +version: -1 +date: "2022-07-07T07:37:15.986010721+02:00" +from: + id-type: crev + id: YdnEoYtqvbBGv0hhENLDUYmc3tNfm5V5NIG5hCovHyM + url: "https://git.jcg.re/jcgruenhage/crev-proofs.git" +package: + source: "https://crates.io" + name: yap + version: 0.8.0 + revision: 1918ec523cdca956b83e34aaa0de65d54ba6b962 + digest: o662jTvJbUJREUMUPUTeqNWJzKWwIXOOPClPZOHdQ5U +review: + thoroughness: high + understanding: high + rating: positive +alternatives: + - source: "https://crates.io" + name: nom +comment: |- + Follow-up from my previous review for version 0.7.2: The sound use of unsafe has now gotten a test case to show that it's actually working as expected with exotic characters, and the `Tokens::optional_err` function I was missing last time is something I've contributed since. Still very happy with yap, and looking forward to seeing this used more. + + For a more detailed review, read my previous coverage of 0.7.2. +----- SIGN CREV PROOF ----- +o5nfh-fDuC01CACyF4xKcgaJc8zPw_xCz2oRADWBz3VH9vPqS73vIuFc21tESo1Ca9Cvl468D62HLvhyv-TvBg +----- END CREV PROOF ----- +