April 2008 - Boost - lists.preview.boost.org

Re: [boost] [1.36.0] High priority compilers for this release?
by Boris Gubenko 03 Apr '08

03 Apr '08

John Maddock wrote: > [...] Incidently, it's good to have at least one > EDG based front end in our official support list, even if it does > emulate > the MSVC bugs ;-) HP aC++ (toolset acc) is EDG based compiler. So is cxx on Tru64, but the latter is not an officially supported compiler in 1.35 (it is in 1.34). Boris

1 0

[shifted_ptr] new_sh
by Phil Bouchard 03 Apr '08

03 Apr '08

Greetings: This is a short notice to let you know I changed "make_shifted_ptee" function call to "new_sh" because it deosn't make sense to use "make_" series in that situation because "shifted_ptee" isn't a valid object. I think "new_sh" is acceptable. Regards, -Phil

1 0

GSoC Ideas: Optimizing Boost libraries for multi-core machines
by Cristianno Martins 03 Apr '08

03 Apr '08

Hello everybody, As I said in last e-mails, I'm interested in Boost projects to GSoC. Now, I have write a proposal and submit it to GSoC, but I'll really appreciate if anyone comment my idea. (I can change something wrong or strange yet) Well, I'm proposing to parallelize one or more libraries in Boost, using OpenMP to do this. Basically, I'll choose a library, review its source code, identify and parallelize the bottlenecks, and document the strategy and results. These steps will be done iteratively during the project. Any comment is welcome. Best, -- Cristianno Martins Mastering in Computer Science State University of Campinas cristianno.martins(a)students.ic.unicamp.br skype:cristiannomartins g-Talk: cristiannomartins msn: cristiannomartins(a)hotmail.com (rarely used)

7 10

[extension] Replace LoadLibrary with LoadLibraryA
by Boris 03 Apr '08

03 Apr '08

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*). Boris

5 9

CGI/FastCGI library preview
by Darren Garvey 03 Apr '08

03 Apr '08

Hi everyone, Some of you may remember the CGI library that was being worked on for last year's GSoC. It still needs *loads* of work before it could be considered ready for review, but I think it's moved on far enough since I last updated this list that some more feedback would be very useful. Pre-built docs are online: http://cgi.sf.net/doc Current source is at: *http://tinyurl.com/2bsbsf* (browser-friendly) *http://tinyurl.com/2xcye2* (check-out) ** This is alpha-grade source: functional and ugly ** The examples there have been tested on ubuntu with gcc 4.1.2 and gcc 4.2 and on windows with MSVC 9. The FastCGI examples will only work on linux for now. I've not managed to get it working with lighttpd as-is but it seems to work fine on Apache 2.2 with mod_fastcgi or mod_fcgid (and shows some healthy benchmarking results). Any and all comments/suggestions/criticisms are welcome. Kind regards, Darren

2 3

Trac open bugs - by assignee
by Marshall Clow 03 Apr '08

03 Apr '08

[ 256 open bugs today, 245 Tuesday ] There's been an upsurge of reported bugs (probably due to increased interest in the 1.35.0 release). If any of these are yours, please take a look at them. Thanks! If you are maintaining a library, and have not logged into the trac system at <http://svn.boost.org/trac/boost/>, please do so ASAP. Until you do that, no tickets can be assigned to you, and will remain under "None". Instructions on working with tickets can be found at: <http://svn.boost.org/trac/boost/wiki/TicketWorkflow> and a mapping of user names to "real names" can be found at: <http://svn.boost.org/trac/boost/report/16> Bug summary 24 djowel 23 az_sw_dude 21 grafik 15 vladimir_prus 14 dave 13 dgregor 12 doug_gregor 12 bemandawes 11 no-maintainer 11 agurtovoy 8 turkanis 8 jsiek 8 anthonyw 6 matias 5 witt 5 nesotto 5 johnmaddock 5 igaztanaga 4 rogeeff 4 ramey 4 ebf 3 nasonov 3 kaalus 3 danielw 3 danieljames 3 None 2 shammah 2 samuel_krempp 2 pdimov 2 pavol_droba 2 jmaurer 2 hkaiser 2 aaron_windsor 1 urzuga 1 t_schwinger 1 speedsnail 1 marshall 1 garcia 1 fcacciola 1 dlwalker 1 chris_kohlhoff 1 <Blank> <Blank> 1750 Autolink library name not in sync with actually build lib None 1644 Boost.Test missing documentation file BOOST_CHECK_CLOSE_FRACTION.html None 1736 Headers containing non ASCII characters cause MS VC to issue warning 4819 in some locales None 1744 No information on building static libraries with bjam under Windows aaron_windsor 1622 Adjacency list needs to handle self-edges correctly aaron_windsor 778 top-level configure is broken agurtovoy 1001 gcc 4.1.2 segfaults on mpl/test/apply.cpp agurtovoy 1051 MPL header ordering triggers bug agurtovoy 1099 unable to iterate through a "non-explicitly-declared" mpl::set agurtovoy 1215 Boost 1.34.1_ mpl: Wrong workaround detection using MinGW32 (or.hpp_ line #33) agurtovoy 1317 has_xxx randomly broken in msvc agurtovoy 1469 [mpl] nested typedef of set has two different meanings agurtovoy 1476 always<> is not nullary. agurtovoy 1528 g++ 4.3.0 reports error because of redeclaration of next agurtovoy 1549 Boost.MPL: 'index_of' algorithm is not documented agurtovoy 1568 duplicate include guards agurtovoy 588 mpl::remove compile error with gcc 4.1.0 anthonyw 1154 tss destruction segfaults in conjunction with dlopen/dlclose anthonyw 1168 [thread] tss crashes randomly anthonyw 1268 xtime_cmp changes to deal with overflow were reverted anthonyw 1470 Boost.Thread: Compilation errors with non-__cdecl calling convention anthonyw 1657 Bug in boost::threads sample anthonyw 1713 no definitions for copy ctor and assignment operator for basic_condition_variable anthonyw 876 xtime documentation incomplete anthonyw 881 errors when build with Apache stdcxx az_sw_dude 1028 typo in boost header: date_time/time_defs.hpp az_sw_dude 1078 from_iso_string cannot read the output from to_iso_string az_sw_dude 1098 Urgent libboost-date-time1.33.1 : bugs on ia64 ? az_sw_dude 1147 date_time library contains global #define ERROR az_sw_dude 1160 warning: month_format shadows a member of this az_sw_dude 1162 warning: ticks shadows a member of this az_sw_dude 1289 posix_time::time_facet forgets to pass a_ref argument to base class az_sw_dude 1299 change in typedef stringstream_type in format_date_parser.hpp causing compilation error az_sw_dude 1306 invalid docbook in date_period.xml az_sw_dude 1384 Linking problem with date_time library on Visual C++ 2008 Express az_sw_dude 1386 date_time documentation Jamfile breaks building of boost docs az_sw_dude 1524 format_date_parser.hpp(221) : cannot convert parameter 1 az_sw_dude 1574 operator<< for boost::posix_time::ptime ignores ostream flags az_sw_dude 1674 date_time::date_formatting.hpp: incorrectly formats year using global locale az_sw_dude 1695 add correct namespace to class to fix compilation bug az_sw_dude 1726 boost/date_time/posix_time/time_formatters.hpp incorrectly formats fractional seconds using global locale az_sw_dude 1740 date_time includes <windows.h> az_sw_dude 287 wrong usage of ios_base::narrow az_sw_dude 316 date_time type conversion warning az_sw_dude 604 from_ftime incorrectly processes FILETIME filled with zeros az_sw_dude 642 operator<< for gregorian::date_duration not found az_sw_dude 889 Insane Dependencies az_sw_dude 890 date_time library supports only the current dst rule bemandawes 1378 filesystem path invalid name for hidden files (LINUX) bemandawes 1389 filesystem::ostream << boost::format generates operator ambigious error (in MSVC++) bemandawes 1458 Darwin Tiger throws std::runtime_error: locale::facet::_S_create_c_locale name not valid bemandawes 1578 bug in readdir_r_simulator() in a multi-threaded environment bemandawes 1583 Serious flaw in boost::filesystem filename handling (opensuse) bemandawes 1589 [filesystem] 'mbstate_t' : is not a member of 'std' bemandawes 1606 filesystem/src/exception.cpp doesn't build with gcc bemandawes 1613 Link Errors against file_system on OS X with Intel Version 10.1 C++ compilers bemandawes 1682 path_traits needs imbue() bemandawes 1701 boost::filesystem::remove_all - segmentation fault with wpath on Linux bemandawes 1711 Boost Serialization export facility broken on gcc 4.1_ 4.2 bemandawes 752 directory_iterator doesn't work with catch chris_kohlhoff 1703 [asio] Compiler warnings on VC8/9 + warning level 4 danieljames 1554 When BOOST_HASH_NO_EXTENSIONS is defined hash doesn't support new string types. danieljames 1555 When BOOST_HASH_NO_EXTENSIONS is defined hash doesn't support boost::system::error_code danieljames 1705 [boostbook] Spaces in reference file names for types like 'long long' danielw 1044 Argument pack inaccessable in a functions return type danielw 1054 Parameter takes an "infinite" amount of time to compile with the Intel compiler. danielw 1097 keyword.hpp generates many warnings C4180: qualifier applied to function type has no meaning; ignored dave 1010 linking boost.python on freebsd dave 1019 iterator_facade::pointer should be the return type of operator-> (DR 445) dave 1085 Python rules for compiling in Mac OS X dave 1179 [boost.python] can not export a union in VC2005sp1 dave 1181 [boost.python] can modify enum value dave 1387 Review / update docs to reflect changes dave 1460 Python classes with multiple bases are convertible only to the first class in the lists dave 1482 boost::python::throw_error_already_set should be marked noreturn dave 1516 [python] Failing VC9 regression tests dave 1599 [python] '_set_se_translator': identifier not found dave 1697 [utility][noncopyable] ADL barrier broken by msvc-9.0 dave 1730 No debug libs on Windows by default dave 865 Use of C++0x keyword as identifier dave 910 gcc strict-aliasing problems with python dgregor 1009 Improper overflow handling in shortest paths algorithms dgregor 1164 Floyd Warshall broken with unsigned edge weights dgregor 1192 [boostbook] problem when two files have the same name dgregor 1199 configure doesn't work with path containing whitespaces (MacOS X) dgregor 1310 [result_of] const-qualified function pointer doesn't work dgregor 1326 Unable to check graph isomorphism using LEDA adapter dgregor 1391 dominator_tree_test on Borland/CodeGear outputs overly long line dgregor 1395 Write Tests for Python Bindings dgregor 1630 gcc-4.3: adjacency_list.hpp includes deprecated header dgregor 1680 long long and wchar_t support in MPI dgregor 556 Bundled graph properties dgregor 732 Johnson All-Pairs needs better "no path" information dgregor 736 Const correctness violation djowel 1170 Emit Warnings instead of # symbols in the syntax highlighter djowel 1171 Quickbook title output djowel 1172 non-utf djowel 1176 Space after inline code djowel 1206 Escaping from within code? djowel 1273 CR+LF newlines in position_iterator djowel 1276 spirit problem with _GLIBCXX_DEBUG djowel 1277 Closing section like [pre with } causes Quickbok to crash djowel 1280 sections in imported source files get put inside <para> elements in docbook output djowel 1281 Quickbook produces invalid BoostBook/DocBook XML djowel 1324 tuple should have a swap() that operates element-by-element djowel 1396 wrong result_of invocation around transform_view djowel 1436 [quickbook] inline code inside headings djowel 1484 Quickbook produces illegal docbook for some docinfo fields. djowel 1500 [Spirit/Phoenix] C4512 Compiler warnings from VC8 djowel 1570 Some boost headers aren't ready for gcc 4.3 djowel 1600 duplicate include guard djowel 1601 distane of boost::tuple djowel 1607 Tuple Library Documentation Changes djowel 1610 [MSVC/STLport] [tree/common.hpp] Standard library requirements violation(incomplete type as vector parameter) djowel 1614 vector workaround needed for MSVC9 djowel 1617 spirit gcc-4.3 fix djowel 1619 [Fusion] C4512 warnings from VC9 (Visual Studio 2008) djowel 1722 [TR1/Fusion] Conflict between TR1 and Fusion headers dlwalker 613 boost/crc.hpp uses non-standard conforming syntax doug_gregor 1021 [Graph][reverse_graph] Problem using reverse_graph adapter with an adapted user defined graph doug_gregor 1076 compile error: disconnecting struct with operator()() doesn't work in VS2005 SP1 doug_gregor 375 LEDA graph adaptors do not handle hidden nodes properly doug_gregor 402 random_vertex/random_edge are unnecessarily inefficient doug_gregor 403 Document copy_component doug_gregor 447 reverse_graph and constness of property maps doug_gregor 708 Dijkstra no_init version should not require VertexListGraph doug_gregor 733 Numbered headers don't work with "preferred" syntax doug_gregor 735 Fruchterman-Reingold grid performance can be improved doug_gregor 738 Memory leaks with signal::connect? doug_gregor 829 Implicit graph not implement doug_gregor 852 Problem with Boost and GCC 4.1 ebf 1507 [variant] C4512 warnings from VC8.0 ebf 1548 [variant][docs]typo: formal arg_ operand_ should be v in times_two arglist ebf 1654 recursive variants do not handle nested variants correctly. ebf 993 Variant should use least common multiple of alignments_ rather than maximum fcacciola 1666 Visualizer for boost::optional garcia 1581 Boost.MultiArray: Resizing bug if the index is not 0-based grafik 1041 Cygwin requires PATH setting. grafik 1322 trac rejects patch attachment as spam grafik 1417 Regresson tests hang in Darwin (Tiger) grafik 1430 Regression fails under Cygwin grafik 1433 Trac's spam detection rejects too many valid messages grafik 1512 Broken link to vacpp in bjam docs. grafik 1513 Broken link to bcpp in bjam docs. grafik 1576 cannot read non-ASCII output messages grafik 1577 Some commands don't work because hStdInput == 0 grafik 1663 build.bat cannot set/reset ERRORLEVEL grafik 1731 Website 'bugs' grafik 1733 Website layout overlays on IE7 grafik 1735 [website] Wrong HTML on main documentation page. grafik 1738 Boost Jam details and download link grafik 1739 Search box and Google link grafik 1741 Review schedule out of date grafik 1742 Some bugs in http://www.boost.org/doc/libs/1_35_0 grafik 1753 Python docs: bad link to the Jamroot grafik 616 Boost Jam_ and non english directorys grafik 959 linking fails when Boost is compiled with STLport 5.1.* on Linux (names mismatch) grafik 986 Problem building Python modules on boost 1.34.0 hkaiser 1079 boost-1.34.0 fails to compile. concept checks not satisfied. hkaiser 1670 wave almost unusably slow on real-world input igaztanaga 1080 boost::interprocess win32 global file mapping issue igaztanaga 1210 interprocess::barrier hangs during second use under linux (boost 1.33.1) igaztanaga 1390 Interpocess: managed shared memory failing completely under OS X igaztanaga 1634 boost::interprocess::vector cannot contain volatile int igaztanaga 1635 Incomplete include guard in boost/intrusive jmaurer 351 Diff in state of mersenne_twister gen between GCC3.41 & CW9 jmaurer 599 uniform_01 copies engine instead of using a reference johnmaddock 1081 Empty string rejected as perl-style regex johnmaddock 1411 Fix BOOST_PARTIAL_SPECIALIZATION_EXPLICT_ARGS johnmaddock 1446 POSIX API doesn't use C locale johnmaddock 1451 error: use of C99 long long integer constant johnmaddock 1749 regex_match on filesystem iterator.leaf () leads to strange results in what[] jsiek 1531 segfault in dynamic_bitset::reference assign operator jsiek 1540 Poisson distribution very slow for large mean (and may also overflow). jsiek 1755 Boost.tokenizer is broken for wide characters - patch attached jsiek 575 Calling subgraph::global_to_local on a root graph jsiek 636 strange compiling problem for transitive_closure jsiek 815 remove_edge core dumps on self-circle jsiek 875 No iterator based constructor for adjacency_matrix jsiek 900 graphml reader namespace handling is broken kaalus 1184 property_tree assignment (operator) bug kaalus 1604 property_tree xml parser doesn't compile with gcc kaalus 1678 Boost.property_tree::read_xml does not parse UNICODE file with BOMs marshall 1252 [pool] severe overhead with unaligned sizes. matias 1406 Understanding the Tables outdated matias 1651 incorrect typedefs in examples matias 1676 Dead links on Boost.Test example documentation. matias 1707 small typos in Boost.serialization documentation matias 1727 Sample code in documentation does not compile_ has namespace errors. matias 1743 Documentation has incorrect Build & Link settings for some libraries nasonov 1220 lexical_cast requires RTTI nasonov 1704 [any] 4512 warning from VC8/9 on warning level 4 nasonov 754 boost::any - typeid comparison across shared boundaries nesotto 1629 range/iterator_range.hpp bug nesotto 1675 [range] VC8 - compiler warnings when including range/array headers nesotto 1694 NDEBUG vs. BOOST_ASSERT in iterator_range.hpp nesotto 1748 [ptr_container] compiler warnings from VC9 nesotto 1754 Illegal HTML in range docs. no-maintainer 1068 Mersenne twister disables streaming for Visual C++ no-maintainer 1222 lambda::ret<T> should declare result_type. no-maintainer 1270 uniform_smallint doesn't compile with lagged_fibonacci random number generators no-maintainer 1515 [lambda] C4512 warnings from VC8.0 no-maintainer 1544 uniform_01 w/ref template param causes ref to ref error no-maintainer 1546 Method boost::random::uniform_int::operator()(engine_ value) is missing no-maintainer 1572 [lambda] smart pointer const-ness affects its contents no-maintainer 1632 Default Interval rounding policies incomplete no-maintainer 1691 [Random] Brokem MSVC workaround no-maintainer 781 Lambda: (_1 + "y")(string("x")) Doesn't Compile no-maintainer 819 uniform_int<> with type's maximum availaible range error pavol_droba 1152 rle_example fails pavol_droba 1385 begin must be boost-qualified pdimov 1106 shared_ptr<T> should not convert to shared_ptr<U> when T* doesn't convert to U* pdimov 1719 [addressof] Can't obtain the address of an instance of an anonymous struct ramey 1285 [serialization] problem when including shared_ptr_132.hpp ramey 1708 Can't serialize class with private destructor ramey 1734 BOOST_STATIC_WARNING can't be passed a dependent expression ramey 1747 [serialization] unreferenced formal parameter warnings from VC9 rogeeff 1383 signal.h features not found on Mac OS rogeeff 1542 [test] compiler errors when building with stdcxx library rogeeff 1723 No subprocess can be started in init_unit_test_suite rogeeff 1746 Online documentation for 1.35.0 is out of date for current lib version samuel_krempp 1699 Compile warning about initialization order. samuel_krempp 859 boost::format ignores a user defined locale shammah 290 perfomance: memory cleanup for pool takes too long shammah 386 boost::pool_allocator breaks with vector of vectors speedsnail 1138 Problems with TSS and static thread library t_schwinger 1653 wave executable erroneously required for build of function_types lib turkanis 1523 [iostreams] VC8/VC9 compiler warnings turkanis 1545 [boost/rational.hpp] 'noskipws' is not a member of 'std' turkanis 1579 gzip compressor should model DualUseFilter turkanis 1580 Add static assert to prohibit instantiating a chain with mode dual_use turkanis 1649 Fix definition of JavaScript menu to avoid long chains of functions turkanis 1656 Consider implementing flush() for symmetric filters turkanis 1671 type mismatch of buffer size parameter causes overload resolution to fail turkanis 1757 Online docs have lost the menu contents in the left frame of the page urzuga 574 [boost::lambda] Compile error with libstdc++ debug mode vladimir_prus 1132 program_options documentation error. vladimir_prus 1165 more quoting problems vladimir_prus 1274 Useless error message vladimir_prus 1321 invalid docbook in tool documentation vladimir_prus 1527 [program_options] Second (and subsequent) lines in tabbed text not aligned correctly vladimir_prus 1530 Program options documentation - Overview - Syntactic Information example typo vladimir_prus 1537 no value given in 'mysql_password =' vladimir_prus 1584 cryptic bbv2 error building quickbook vladimir_prus 1587 [program_options] crashes if value part missing in external configuration file vladimir_prus 1588 [program_options] crashes if value part missing in external configuration file vladimir_prus 1645 Invalid unicode support for default_value vladimir_prus 1679 added 4-way universal binary support from mac osx (ppc_ ppc64_ i386_ x86_64) vladimir_prus 469 multitoken broken vladimir_prus 689 [program_options] Endless loop with long default arguments vladimir_prus 850 program_options strips off escaped quotes in some situations witt 1046 Boost.Python quickstart instructions mention nonexistent target witt 583 Fixes for build on IBM pSeries for AIX and Linux witt 834 Homepage regression links witt 957 The "Getting Started" page does not mention the stdlib option witt 973 zip_iterator has value_type == reference -- -- Marshall Marshall Clow Qualcomm, Inc. <mailto:mclow@qualcomm.com> The famous British scientist, Lord Kelvin, said: "When you measure what you are speaking about and express it in numbers, you know something about it, but when you cannot (or do not) measure it, when you cannot (or do not) express it in numbers, then your knowledge is of a meagre and unsatisfactory kind."

1 0

[thread] Interest in a locking pointer and lockable concept
by klaus triendl 03 Apr '08

03 Apr '08

Hi, is there any interest in something like a locking pointer and a type that I call "lockable" that pairs any type with a mutex? Inspired by Andrei Alexandrescu's article "volatile - Multithreaded Programmer's Best Friend" on ddj I implemented a concept similar to a locking pointer called "lock_acquirer" that collects acquisition of a mutex and a volatile cast of the locked type. It is even more threadsafe to use than a locking pointer. Additionally I made a class "lockable" that pairs a (volatile) type with a mutex type, wrapping both in a distinct type, which has first the advantage that the type to protect and the mutex are glued together, and second that it encapsulates volatile qualifier correctly. Also, the locked type in a lockable can, if wanted, by a restricted interface only be accessed by a lock_acquirer thus forcing the programmer to a very threadsafe programming style. I can post my implementation along with a simple example to the boost vault. Klaus

6 25

Regression in Multi-index, 1.35?
by Amit 03 Apr '08

03 Apr '08

Hi, I am trying to migrate my project to 1.35. The following used to compile and work with 1.34.1: bmx::multi_index_container<wstring, bmx::indexed_by<bmx::ordered_unique<bmx::identity<wstring> >, bmx::random_access<> >, boost::fast_pool_allocator<wstring> > But refuses to compile with 1.35 (I use Intel compiler 10.1 with MSVC7.1 STL). Something to do with allocator rebind. I am including some of the compiler output below. Can someone please suggest a resolution (hopefully keeping the pool allocator intact)? Thanks Amit C:\DevLibraries\Boost\include\boost-1_35_0\boost/pool/pool_alloc.hpp(130): error: reference to void is not allowed typedef value_type & reference; ^ detected during: instantiation of class "boost::fast_pool_allocator<T, UserAllocator, Mutex, NextSize> [with T=void, UserAllocator=boost::default_user_allocator_new_delete, Mutex=boost::details::pool::default_mutex, NextSize=32U]" at line 149 of "C:\Boost\include\boost-1_35_0\boost/detail/allocator_utilities.hpp" instantiation of class "boost::detail::allocator::rebinder<Allocator>::result<Type> [with Allocator=boost::fast_pool_allocator<void, boost::default_user_allocator_new_delete, boost::details::pool::default_mutex, 32U>, Type=boost::multi_index::detail::random_access_index_node_impl< boost::fast_pool_allocator<void, boost::default_user_allocator_new_delete, boost::details::pool::default_mutex, 32U>>]" at line 158 of "C:\Boost\include\boost-1_35_0\boost/detail/allocator_utilities.hpp" instantiation of class "boost::detail::allocator::compliant_allocator_rebind_to<Allocator, Type> [with Allocator=boost::fast_pool_allocator<void, boost::default_user_allocator_new_delete, boost::details::pool::default_mutex, 32U>, Type=boost::multi_index::detail::random_access_index_node_impl< boost::fast_pool_allocator<void, boost::default_user_allocator_new_delete, boost::details::pool::default_mutex, 32U>>]" at line 62 of "C:\Boost\include\boost-1_35_0\boost/mpl/eval_if.hpp" instantiation of class "boost::mpl::eval_if_c<C, F1, F2> [with C=false, F1=boost::detail::allocator::partial_std_allocator_rebind_to< boost::fast_pool_allocator<void, boost::default_user_allocator_new_delete, boost::details::pool::default_mutex, 32U>, boost::multi_index::detail::random_access_index_node_impl< boost::fast_pool_allocator<void, boost::default_user_allocator_new_delete, boost::details::pool::default_mutex, 32U>>>,

4 4

Re: [boost] Review Request: Generalization of the Pimpl idiom
by Vladimir.Batov＠wrsa.com.au 03 Apr '08

03 Apr '08

... > > That requirement to have > > > > Derived1::Derived1(implementation*) > > > > is unreasonable as it is clearly not related to to the Derived1 *public* > > interface. On the other hand reset()-based approach works fine and scales > > fine. > > > > ... > > I understand now. This will force to add an artifial *protected* constructor > to > Derived1. Do you really think that this is a real problem for the developer > of Derived1? > If I understand your library forbids this to the developper because > pimpl_base do not have the following protected constructor. > pimpl_base (implementation* impl) : impl_(impl) {} > > I think that this will be a good adding to the pimpl_base class. Respectfully I have to disagree. The reason is that it has been long established precedent in C++ not to provide alternative mechanisms/tools/interfaces if the existing already does that. An immediate example OTOH is that an introduction of "superclass" keyword was rejected because a typedef would do the same thing. Granted, tediously and not as elegantly but still. Here IMHO it's the same story -- we are not talking about missing functionality but rather questionale interface sugar-coating which BTW requires the user to do more than he really have to ("force to add an artifial *protected* constructor"). ... > > I am not sure what that is supposed to demonstrate. You use infrastructure > > to help you to do your job. If you decide that the infrastructuere does > > not help you and discard that, you'll obviously have to re-implement > > whatever functionality from that infrastructure you managed to deploy. If > > one uses std::list and then for whatever reason goes ahead with his own > > list implementation or merely swaps it for std::vector, he'll likely to > > face the same issue. Trade-offs of the pimpl idiom are well-known -- > > another indirection and memory allocation. In the end, one can deal with > > memory-allocation by writing a custom allocator for > > pimpl<Foo>::implementation. > > If pimpl is here to hide the implementation, why it needs to extends the > interface of the initial class? > Why the user of my class should suffer from the trade-offs of the pimpl > idiom at the inteface level? Pimpl extend the interface so that the user does not have to provide those methods. Like comparison and validity check, etc. That does not come at the user's expence and, therefore, how exactly the user suffers from that escapes me. > >> Do you really think that the pimpl idiom is helping me to hide my > >> implementation. If I want to hide my implementation I need to restrict > > to > >> private inheritance. > > > > I cannot say about you but I do really think that the pimpl idiom is > > helping me to hide the implementation. Private inheritance restricts > > access but not hides the implementation. > > I was talking about private inheritance from pimpl!!! If you try deploying pimpl in applications, you'll quickly see that you'll need public inheritance. I do. Having said that, if you insist on inheriting privately, there is nothing stopping you. We are not discussing missing pimpl functionality or crooked implementation, right? > > struct Base { int k; }; > > class Derived : Base {} > > > > Here Base::k is not accessible in Derived but not hidden. More, if that > > stuff is a commercial library, I can easily circumvent that restriction by > > modifying Derived declaration. Pimpl really hides the implementation. See > > for more about the idiom > > > > 1. Guru of the Week #24. http://www.gotw.ca/gotw/024.htm > > 2. Herb Sutter. Exceptional C++ (Addison-Wesley, 1999) > > 3. J. Carolan. Constructing bullet-proof classes. In Proceedings C++ at > > Work'89 (SIGS Publications, 1989) > > 4. James O. Coplien. Advanced C++ Programming Styles and Idioms > > (Addison-Wesley, 1992) > > 5. Eric Gamma et al. Design Patterns (Addison-Wesley,1995) > > This is why I suggested to remove the null object and bool type conversion > operators for pimpl<A>::value_semantics. Maybe it will be a good idea to > give them protected acces. I admit that "This is why" escpes me. Invalid objects are all over -- nullptr, shared_ptr(), -1, etc. Pimpl extends that concept onto actual classes and gives it uniform and proper implementation. I've been using this extensively and I do indeed need that... public. If one does not need that, he won't use that. That feature won't kick in by accident. > >> Once we remove the public inheritance from pimpl, it is not eassy to > >> implement all these extra functions and operators. > > > > Then do not remove public inheritance. I am far from being convinced that > > insistence on private inheritance is justified. > > Conveinced or not this is a way to mask the pimpl functions, and it seams to > me the best way to use pimpl<A>::value_semantics. This is not the case with > pimpl<A>::pointer_semantics from which it is better to inherit publically > giving acces to the null object and the bool type conversion operators. > This is only a comment on how I will use it :-) Yes, that makes sense. Although if my usage experience is of any guidance, you'll swap private for public fairly soon. I guess, we'll see. ... > >> >> Could we inherit publicly from A and privatly from pimpl<B, > >> >> A>::value_semantics? > >> > > >> > Cannot comment on this one as I am not convinced we need that (see > > above). > >> Imagine A3ppClass is a 3pp class ( we can not modify it), and before > > using > >> the > >> your pimpl class we had: > >> > >> class MyClass : public A3ppClass {}; > >> > >> How can we use pimpl without changing the interface of MyClass, neither > >> adding nor removing any function? > >> > >> May be we can resume: do not use the pimpl library is you can not master > > the > >> complete inheritance hierarchy. ... > In any case I don't see a real benefit of using pimpl when the base class is > 3pp class and we can not change it. Well, if you build a run-time polymorphic hierarchy on top of an existing 3rd-party base class, then I tend to agree with you as you have to follow the design and strategy outlined by the base class designer. I do not feel like it is a pimpl inherent weakness. Havind said that, I'd shy away from coming to your generic conclusion based on that particular case. Unless I have to write deep run-time polymorphic hierarchies (like Widgets, etc.), I tend to cut inheritance short and I believe it is the strong current programming trend in C++. Therefore, in your case I'd try pimpl<MyClass>::implementation : public A3ppClass {} then in MyClass I'd bridged to A3ppClass only the interface that MyClass needed: MyClass::foo() { (*this)->A3ppClass::foo(); } It might not be applicable in your particular case -- no one claims to have the silver bullet. ... > >> class A : private pimpl<A>::value_semantics {...}; > >> class B : private pimpl<B>::value_semantics {...}; > >> class C : public A, public B {...}; > >> > >> May be we can resume: do not use the pimpl library if you need multiple > >> inheritance. > > > > I cannot see anything drastically wrong with the code above even though I > > probably would not do it that way. So, I'll have to disagree with your > > conclusion. > > Anyway, we will do the same, don't use pimpl with multiple inheritance. I would not say that I will not. If/when the time comes for me to deploy multiple inheritance, I cannot see myself hesitating because of the pimpl. ... > > I have to disagee as I find that view over-simplified and in fact > > incorrect. Pimpl or no pimpl in general no objects beyond value-semantics > > PODs can be safely (or without additional effort) put into shared memory. > > I hope so, and I expect that you will do what is needed to achiev this. I > tried only to help you. And it is much appreciated. > Pointers are emulated in Boost.Interprocess with the smart pointer > offet_prt. > You can yet have inheritance without virtual functions. Well, I would not insist on that without severe reservations, numerous ifs and buts. > Please, could you take a look to the Boost.Interprocess, Boost.Intrusive and > Boost.MultiIndex libraries. > Even if not standards they are portable. :-) Yep, had a quick look at boost.interprocess. Seems like it takes care of shared memory hasssle. Cool. I presume he managed it for UNIXes and Windows. Super cool. And I see boost.multiindex supporting boost.interprocess allocators. Now I get it. Still I do not feel like rushing to support the feature. :-) We'll get the bridge when we get there. I'll keep it iin mingd though. Thanks for your patience and insistence. You see, I not THAT thick after all. :-) > > Then, you'll have to provide custom allocators. The issue just > > keeps more and more complicated. Do you actually have that setting or we > > are talking hypothetically? If the former, then pimpl will not be your > > primary concern. :-) If it is the latter, then I find it too ephemeral to > > discuss effectively. > > I insist. This is not hypothesis. I have to manage a lot of data shared > between process, I use specific allocators, and synchronization mechanisms. > I wanted only to know if I can hide the implementation of this classes with > the pilm classes. That's all. OK, I see you need that. :-) It'll take me sometime to get up to speed with boost.interprocess. Worse, for various reasons I cannot use un-official boost. So, go and figure. > > If you do not have several processes accessing the same data and, > > therefore, do not care for inter-process data-access synchronization, > > inter-process vtbl pointer correction, etc., then I'd be wondering why you > > use shared memory in the first place. > > Who has said that I have no several process sharing the same data? No, you certainly did not say you did not have several processes. Have you noticed there was 'if' at the beginning of my sentence? :-) > But the data has no virtual functions. This is not a raison to don't want to > hide its implementation. Fair enough. You convinced me. Honestly, Unfortunately, or reasons beyond my influence I won't be able to extend pimpl support onto boost.interprocess in a short timeframe. Apologies. .. > Thank you very much for your time and good luck for your review. > I promise I'll try your library. Thanks for your input. Much appreciated. Thanks, Vladimir.

2 1

Removing copyright symbols from files.....
by Marshall Clow 03 Apr '08

03 Apr '08

Unless anyone objects, I am going to remove the copyright symbol "©" from source and header files (in response to support request #597). I will not touch anything that's not in a comment. ;-) It looks like about 37 header files, and 24 source files. I'll do this tonight, unless anyone objects. Example transformation: In file "boost/archive/detail/auto_link_archive.hpp", on line #12: // © Copyright Robert Ramey 2004 will become: // (c) Copyright Robert Ramey 2004 -- -- Marshall Marshall Clow Idio Software <mailto:marshall@idio.com> It is by caffeine alone I set my mind in motion. It is by the beans of Java that thoughts acquire speed, the hands acquire shaking, the shaking becomes a warning. It is by caffeine alone I set my mind in motion.

3 5