-
Notifications
You must be signed in to change notification settings - Fork 16
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
move from lhttpc #17
Comments
Out of curiosity, what was the actual issue with lhttpc? Anything that didn't work well with it that required a move, or is it outdated or something? (Since a patch with 53 additions and 21 deletions seem like a step in the wrong direction from my point of view - unless the extra code adds some valuable feature that was missing.) |
from https://github.com/esl/lhttpc "This project is not supported anymore." Took 1.5 years to merge the simple esl/lhttpc#43, meanwhile we were running on some fork (not a proper lhttpc-next fork, but the fork that had this commit alone)... The numbers are deceiving though, because the committer (me) chose the least-changes path (not out of being lazy :) but because I wanted #22 to be done before touching more the tests). Similarly, at least 6 lines are just about list-binary conversions, so one could argue "if only KATT was using binaries from the start.." |
Fair enough :) |
to
The text was updated successfully, but these errors were encountered: