"symbols".contains(c) replaced with match because it's more efficient. Unfortunately, Rust does not seem to optimize the contains-call away to something more efficient.
Some rustfmt::skip for regexes and tests (I think tests are a bit of exception because they are often a bit too long for rustfmt but breaking them makes them less readable in my opinion)
Different separators in dtypes.rs because the previous ones were implicit doc-comments
Removed println! call in parser
Some diffs may appear overly large due to moving of code up and down. I aimed to sort everything from generic to specific.
Changes:
fields
moduleIntOrChunks
is an enum nowPagination::Paragraph
Type::parse
takes chunk slice instead of vecchunk_chars
instead ofValueCharsIter
"symbols".contains(c)
replaced with match because it's more efficient. Unfortunately, Rust does not seem to optimize the contains-call away to something more efficient.rustfmt::skip
for regexes and tests (I think tests are a bit of exception because they are often a bit too long for rustfmt but breaking them makes them less readable in my opinion)dtypes.rs
because the previous ones were implicit doc-commentsprintln!
call in parserSome diffs may appear overly large due to moving of code up and down. I aimed to sort everything from generic to specific.