Fix #133 #139

Closed
wants to merge 2 commits into
from

Projects

None yet

2 participants

@cmr
cmr commented Dec 4, 2012

No description provided.

@bnoordhuis
Node.js Foundation member

LGTM provided it works. Can you sign the CLA?

Note to self: see also #138.

@cmr
cmr commented Dec 5, 2012

Tests pass, and I stepped through it to make sure it was doing the right stuff (I had a hell of a time figuring out what to do with a struct url_test). dump_url also dumps the right stuff now.

Already signed the CLA, assuming that's the same one in CONTRIBUTIONS.

@bnoordhuis
Node.js Foundation member

Thanks Corey, landed in 1c7f8ca.

@bnoordhuis bnoordhuis closed this Dec 5, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment