search

Nullus157 / cbor-diag-rs

Support for parsing/encoding CBOR diagnostic notation and annotated hex
https://cbor.nemo157.com
Apache License 2.0
10 stars 6 forks source link

Update from nom 5 #134

Closed chrysn closed 1 year ago

chrysn commented 1 year ago

Building nom with a recent Rust raises warnings about future compatibility:

warning: the following packages contain code that will be rejected by a future version of Rust: nom v5.1.2

It appears that nom uses some macro syntax that was only accidentally valid.

I just briefly tried bumping the nom version to 7, but while I could do the change from separated_list to separated_list0 easily (nom 6's changelog says it was effectively renamed), other errors seem to require understanding what it does, for which unfortunately I don't have the time right now.

Nemo157 commented 1 year ago

Turns out I had an old update branch I forgot about https://github.com/Nullus157/cbor-diag-rs/compare/pr%C4%ABmum...Nemo157:cbor-diag-rs:nom-7, hopefully it was just waiting on that fix for over-allocation and I can get it merged soon.

fmorency commented 1 year ago

Friendly bump! :)

Nemo157 commented 1 year ago

Just published 0.1.12 with nom 7.