patch looks good but could add a test. I also think the i == host.length()? bit could have braces as I first misread it as host.length()?, which is of course legit C/C++, and left me feeling very confused.

Steve Loughran
added a comment - 12/Feb/10 14:00 patch looks good but could add a test. I also think the i == host.length()? bit could have braces as I first misread it as host.length()? , which is of course legit C/C++, and left me feeling very confused.