Re: [Boost-users] [boost] Boost.DLL formal review is ongoing

Subject: Re: [Boost-users] [boost] Boost.DLL formal review is ongoing
From: Ralf Globisch (rglobisch_at_[hidden])
Date: 2015-07-03 15:14:34

Hi,

Firstly, thanks Anthony for submitting this useful library to boost, and to Vladimir for managing the review.

- Should the library be accepted?

Yes. I had been meaning to test the library for some time, and with the review reminder finally
made some time to do so. My vote of yes is based on my (limited) positive experience with the 
library, as well as having followed past related discussions on the mailing list, in which many 
questions are addressed. (http://lists.boost.org/Archives/boost/2014/09/216650.php)

I think boost.DLL should be accepted unless someone has a serious/valid reservation
as to why this library should not to be accepted.

- How useful is it?
Very useful. It enabled me to add a test-plugin mechanism to an existing application 
(Windows and Linux) in a matter of minutes. 

- What's your evaluation of
   - Design
Looks good to me. Simple to use, with good integration into rest of boost (e.g. file_system).

I like the fact that the library is header only and minimal effort was required to integrate
plug-in like functionality into an existing application.

   - Implementation
I'm not knowledgeable enough to comment on this. 

   - Documentations
Good, the tutorials made it fairly easy to get going and covered my common usage scenarios
Code is well commented.

- How much effort did you put into your evaluation?
About three hours worth of effort, reading through documentation and source code, and using 
library to create a very simple plug-in mechanism in an existing application.
Tested basic plug-in functionality on Windows and Linux.

- Did you attempt to use the library? On what systems and compilers?
Yes

-Linux Ubuntu 14.10 with gcc 4.9.1 with boost 1.57

-Window 7 64-bit with VS 2013 community edition with boost 1.58. Initially made the mistake of not checking the boost version (which had been 1.53 and caused lots of compilation errors.) 

Minor typo in documentation:
Just above the "Full Sources" for tutorial 4 "If we want o make a traditional" should be "If we want to make a traditional" 

Regards,
Ralf

>>> Vladimir Prus <vladimir.prus_at_[hidden]> 07/03/15 3:52 PM >>>

The formal review of Boost.DLL library by Antony Polukhin is nearing the end of the first week.
As of today, over 100 people read through the library's tutorial, suggesting quite some interest.
However, no reviews were submitted yet. Please take the time to post your thoughts - even if you
don't have time to do a full review, or try the library, comments on design and interfaces are
still very valuable.

The summary of the library features and review checklists are reproduced below.

Boost.DLL is a C++98 library for comfortable work with DLL and DSO. Library
provides a portable across platforms way to:

    - load libraries at runtime
    - import and export any native functions and variables
    - make alias names for C++ mangled functions and symbols
    - query libraries/objects and executables for sections and exported
    symbols
    - self loading and self querying
    - getting program and module location by exported symbol

The documentation can be found at:

     http://apolukhin.github.io/Boost.DLL/index.html

and the source can be obtained at:

     https://github.com/apolukhin/Boost.DLL

Please post your reviews on the mailing list and if possible, answer the
following questions:

- Should the library be accepted?
- How useful is it?
- What's your evaluation of
   - Design
   - Implementation
   - Documentations
   - Tests
- How much effort did you put into your evaluation?
- Did you attempt to use the library? On what systems and compilers?

Thanks,
Volodya

_______________________________________________
Unsubscribe & other changes: http://lists.boost.org/mailman/listinfo.cgi/boost

_______________________________________________
Unsubscribe & other changes: http://lists.boost.org/mailman/listinfo.cgi/boost

-- 
This message is subject to the CSIR's copyright terms and conditions, e-mail legal notice, and implemented Open Document Format (ODF) standard. 
The full disclaimer details can be found at http://www.csir.co.za/disclaimer.html.
This message has been scanned for viruses and dangerous content by MailScanner, 
and is believed to be clean.
Please consider the environment before printing this email.
-- 
This message is subject to the CSIR's copyright terms and conditions, e-mail legal notice, and implemented Open Document Format (ODF) standard. 
The full disclaimer details can be found at http://www.csir.co.za/disclaimer.html.
This message has been scanned for viruses and dangerous content by MailScanner, 
and is believed to be clean.
Please consider the environment before printing this email.