charlesvdv / nom-bibtex

A feature complete bibtex parser using nom
https://docs.rs/nom-bibtex
MIT License
24 stars 15 forks source link

Upgrade to Nom 5.1 #8

Closed TheZoq2 closed 4 years ago

TheZoq2 commented 4 years ago

A bit of a larger PR than the previous ones. Initially, I made this change because I wanted the improved error handling that is included in 5.1. Unfortunately, I couldn't make that work with the named macro yet.

The only major change I had to make was avoiding the deprecated ws macro. While I was at it, I also replaced u8 with str, I hope there wasn't some good reason for using u8 originally. If so, I can revert that change :)