http://d.puremagic.com/issues/show_bug.cgi?id=5401

           Summary: std.socket updates and boost license
           Product: D
           Version: D2
          Platform: Other
        OS/Version: All
            Status: NEW
          Severity: enhancement
          Priority: P2
         Component: Phobos
        AssignedTo: nobody@puremagic.com
        ReportedBy: chris@dprogramming.com


--- Comment #0 from Chris Miller <chris@dprogramming.com> 2011-01-02 22:59:59 PST ---
The following changes have been made to std.socket:
 - License changed to Boost Software License - Version 1.0
 - public constructor added for Socket that accepts an existing socket_t and
its AddressFamily.
 - createAddress protected method added to Socket, returns a new Address object
for the current address family; can be overridden to support other addresses.
 - Address class methods name and nameLen are now public.
 - Remove std.traits Select!, use ptrdiff_t instead.
 - Updated unittests; they were marked as broken and acted in various ways, but
really they depend on the environment and network.

These changes were inspired by this thread: http://www.digitalmars.com/webnews/newsgroups.php?art_group=digitalmars.D&article_id=115983

Also updated htmlget.d sample to compile with the latest D2.

Thanks to those who have made various changes and fixes to std.socket since my last contribution.

The code can be found at the following URL (due to bugzilla's "original
material" agreement).
http://www.dprogramming.com/socket14.zip

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------

http://d.puremagic.com/issues/show_bug.cgi?id=5401


Andrei Alexandrescu <andrei@metalanguage.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |andrei@metalanguage.com


--- Comment #1 from Andrei Alexandrescu <andrei@metalanguage.com> 2011-01-03 08:52:27 PST ---
Thanks, Chris! Since you went through the trouble of doing the actual work, any chance you could pass it through a formal review process modeled after http://www.boost.org/community/reviews.html? What you mainly have to do is enhance the documentation, generate HTML so people can see it, and act on feedback from people on the digitalmars.d newsgroup. Let us know - thanks!

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------

http://d.puremagic.com/issues/show_bug.cgi?id=5401


Walter Bright <bugzilla@digitalmars.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |bugzilla@digitalmars.com


--- Comment #2 from Walter Bright <bugzilla@digitalmars.com> 2011-01-03 12:17:59 PST ---
I also want to specifically thank Chris for changing the license to Boost.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------

http://d.puremagic.com/issues/show_bug.cgi?id=5401


Daniel Gibson <metalcaedes@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |metalcaedes@gmail.com


--- Comment #3 from Daniel Gibson <metalcaedes@gmail.com> 2011-02-14 23:42:04 PST ---
This is great, thanks!
When this code is accepted http://d.puremagic.com/issues/show_bug.cgi?id=4234
can be closed.

If additional work is done for a review process etc, may I suggest to add
support for IPv6 (Internet6Address?) and local addresses (AF_UNIX) as well
(also a function like socketpair() would go nice with that)?

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------

http://d.puremagic.com/issues/show_bug.cgi?id=5401


Johannes Pfau <johannespfau@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |johannespfau@gmail.com


--- Comment #4 from Johannes Pfau <johannespfau@gmail.com> 2011-02-15 09:41:02 PST ---
Another small request: Could we use gethostbyname_r instead of gethostbyname where supported? This seems to be a lot safer regarding multithreading. gethostbyname_r seems to be a gnu extension, but it's a better option for all systems with the gnu c library. Or probably the getaddrinfo(3), getnameinfo(3), gai_strerror(3) functions could be used. Those are standard and also threadsafe.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------

http://d.puremagic.com/issues/show_bug.cgi?id=5401


Masahiro Nakagawa <repeatedly@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |repeatedly@gmail.com


--- Comment #5 from Masahiro Nakagawa <repeatedly@gmail.com> 2011-02-16 08:27:57 PST ---
http://lists.puremagic.com/pipermail/phobos/2010-July/001171.html

I suggested std.socket replacement in Phobos ML. This improvement that creates Asio based new socket is still continuing with a view to event, but I don't have enough time ;(

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------

http://d.puremagic.com/issues/show_bug.cgi?id=5401


Vladimir Panteleev <thecybershadow@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |mike.casinghino@gmail.com


--- Comment #6 from Vladimir Panteleev <thecybershadow@gmail.com> 2011-08-23 22:08:08 PDT ---
*** Issue 4234 has been marked as a duplicate of this issue. ***

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------

http://d.puremagic.com/issues/show_bug.cgi?id=5401


Vladimir Panteleev <thecybershadow@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |thecybershadow@gmail.com


--- Comment #7 from Vladimir Panteleev <thecybershadow@gmail.com> 2011-08-23 23:41:01 PDT ---
(In reply to comment #1)
> formal review process

I think that's overkill, considering there are nearly no API changes and it doesn't break existing code, but maybe the intent escapes me at the moment.

I've included Chris's update as part of a larger pull request containing various std.socket changes:

https://github.com/D-Programming-Language/phobos/pull/211

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------

http://d.puremagic.com/issues/show_bug.cgi?id=5401


Jameson <beatgammit@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
                 CC|                            |beatgammit@gmail.com
         Resolution|                            |FIXED


--- Comment #8 from Jameson <beatgammit@gmail.com> 2013-05-19 22:47:38 PDT ---
This appears to be resolved. The associated pull request has been merged 2 years ago.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------