New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

modernize, format, add tracing, adjust log levels #1

Open

orangecms wants to merge 1 commit into nycresistor:master from orangecms:main

orangecms commented Feb 23, 2023 •

edited

Loading

Used in: https://github.com/orangecms/vf2-loader
Demo: https://twitter.com/OrangeCMS/status/1628844843949608961

Everything else:

foo()?; instead of try!(foo()); as suggested by Clippy
add trace!() for inidividual byte transfers (that helped...)
raise some debug!() to !info()
take returned transfer counts

EDIT:
I had initially raised the maximum error count from 16 to 32, but discarded that again.

orangecms force-pushed the main branch from adf2f45 to ca9411f Compare

February 23, 2023 20:23

orangecms changed the title ~~modernize, format, raise max errors, add tracing~~ modernize, format, add tracing, adjust log levels


          modernize, format, add tracing, adjust log levels

fc71fe0

orangecms force-pushed the main branch from ca9411f to fc71fe0 Compare

February 23, 2023 20:42

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet