DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUGĀ·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=43588>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED ANDĀ·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=43588





------- Additional Comments From [EMAIL PROTECTED]  2007-10-31 05:15 -------
(In reply to comment #6)
> Can we change the patch?
> See comment at STATUS File:
> -1: funkman InetAddress.getLocalHost() != localhost - it should be
>             InetAddress.getByName("localhost").getHostAddress()

I will not claim to be an expert in IPv4 or IPv6.  Granted, after tracing the
actual InetAddress source code for both getLocalHost() and getByName(), I
believe the original patch should stay, keeping getLocalHost().  If anyone
disagrees with my choice, and feels that getByName("localhost") is better, I
would have to further recommend that we use : getAllByName("localhost")[0],
which is a shortcut to the new proposal.  If I am completely incorrect, I
apologize - please correct me in a separate email so that I have a better
understanding.

New Choices/My Votes :
1.) +1  - Keep : InetAddress.getLocalHost() and apply current proposed patch
2.) -1  - Change to new proposal : InetAddress.getByName("localhost")
3.) +1? - Change to new proposal 'shortcut' :
InetAddress.getAllByName("localhost")[0]

(In reply to comment #4)
> ignore my previous comment, I'm a retard, didn't look into it enough

Filip, did you get the same results I did?  Why did you agree with me?


-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to