Re: [boost] [extension] Replace LoadLibrary with LoadLibraryA

From: Jeremy Pack (rostovpack_at_[hidden])
Date: 2008-04-01 18:51:21

Boris,

I've already got the first issue in my queue to fix. I'll add the second one
as well. For the sake of Windows users, I may allow either wide or standard
characters for the relevant function calls. Any thoughts?

Boost.Extension has NOT yet been reviewed. I have a list of about 20 bugs
like this one, as well as a couple of functionality improvements that need
to go in before a review. I also have quite a bit of C++ cleanup to do. As
some of you have noticed, my Boost.Preprocessor code does NOT make for easy
debugging. Expect primarily implementation, rather than interface, changes.

For future bug reports for this and Boost.Reflection, you'll get the fastest
response from me by e-mailing me directly. However, if you feel it is
something that needs more discussion (ie, a feature request) then this list
is probably appropriate.

Thanks!

Jeremy

http://boost-extension.blogspot.com

On Tue, Apr 1, 2008 at 3:16 PM, Boris <boriss_at_[hidden]> wrote:

> On Tue, 01 Apr 2008 23:28:12 +0200, Boris <boriss_at_[hidden]> wrote:
>
> > Is this the right place to discuss (Boost.)Extension (using a
> parenthesis
> > as I'm not sure if this library has been accepted or even been
> reviewed)?
> >
> > In file extension/impl/library_impl.hpp, line 37, LoadLibrary needs to
> be
> > replaced with LoadLibraryA. Otherwise if you use Unicode you get a
> > compiler error (as LoadLibrary is then defined as LoadLibraryW which
> > expects a const wchar_t* and not a const char*).
>
> Another problem I ran into: In extension/impl/library_impl.hpp, line 22, a
> macro called WINDOWS_LEAN_AND_MEAN is checked (and possibly defined). This
> must be WIN32_LEAN_AND_MEAN. This is important as Boost.Asio is confused
> if winsock.h is included by another library first (that's how I ran into
> it :).
>
> Boris
>
> _______________________________________________
> Unsubscribe & other changes:
> http://lists.boost.org/mailman/listinfo.cgi/boost
>