Re: [boost] Boost.Atomic on Windows, patch

Subject: Re: [boost] Boost.Atomic on Windows, patch
From: Daniel Larimer (dlarimer_at_[hidden])
Date: 2012-08-02 10:14:31

On Aug 2, 2012, at 9:31 AM, Václav Zeman <vhaisman_at_[hidden]> wrote:

> Hi.
> 
> I have noticed that boost/atomic/detail/generic-cas.hpp is directly
> including Windows.h header. I consider this problematic for WinSock
> using applications as Windows.h automatically pulls in Winsock.h, the
> WinSock 1 header. This is problematic because it requires users that
> want to use WinSock 2 to order their include of WinSock2.h before
> inclusion of Boost.Atomic. At this point in time anybody serious about
> sockets on Windows really do not want to use WinSock 1 but version 2
> instead. So I suggest the attached patch.

        Thanks for the patch.  I really hate those WinSock2 include dependencies!   I have incorporated it into my own vendor branch of atomic.

> -- 
> VZ
> <boost.atomic.windows.patch.txt>
> _______________________________________________
> Unsubscribe & other changes: http://lists.boost.org/mailman/listinfo.cgi/boost