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@web.de> wrote:
On Tue, 01 Apr 2008 23:28:12 +0200, Boris <boriss@web.de> 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