Skip to content
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

Update lib/LWP/Simple.pm #14

Merged
merged 1 commit into from
Jul 30, 2012
Merged

Update lib/LWP/Simple.pm #14

merged 1 commit into from
Jul 30, 2012

Commits on Jul 29, 2012

  1. Update lib/LWP/Simple.pm 

    Had an IRC chat with moritz.  He thought my coding wasn't so bad but didn't fit with the surrounding code and suggested this instead.  During the discussion it also came up that recv doesn't support binary very well yet but $socket.read(nnnn) does.  It occurs to me that since HTTP headers often have a Content-length the read solution may be good in the nearish future.
    @ronaldxs
    ronaldxs committed Jul 29, 2012
    Configuration menu
    Copy the full SHA
    5c040e0 View commit details
    Browse the repository at this point in the history