-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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.
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
Changed util.fiddling.hexdump_iter
to take a file object instead of…
#695
Conversation
… a string. This massively speeds up `phd <file> | less` for huge files.
I think that @idolf doesn't want any extra branches. In the future, create the PR from a fork. For the purpose of back-ports, does this have any functional changes? Or can it just be rolled into the "next release"? |
Since this is against |
@zachriggle, OK, I'll do PR's from a fork in the future. Thanks for merging. |
I am backporting this to stable now, as this is a regression fix. |
… a string. This massively speeds up `phd <file> | less` for huge files. (Gallopsled#695)
This was cherry-picked to 3.0.2 so realistically that's the milestone |
This code wasnt updated when the code was changed to operate on file descriptors. Additionally, there was a functional regression, that the first line would be skipped. Fixes: Gallopsled#836 Caused by: Gallopsled#695 (d65991d)
… a string. This massively speeds up
phd <file> | less
for huge files.