Boost C++ Libraries: Ticket #8914: boost/flyweight/detail/recursive_lw_mutex.hpp: Ignores all failures from pthread_* functions https://svn.boost.org/trac10/ticket/8914 <p> boost/flyweight/detail/recursive_lw_mutex.hpp ignores all failures from pthread_* functions. Functions include pthread_mutexattr_init, pthread_mutex_init, pthread_mutex_lock and pthread_mutex_unlock. </p> <p> A lock failure is usually a bad thing, and I can't come up with scenarios where a silent failure is desired. It will make a bad problem worse by corrupting data or terminating the program. </p> <p> At minimum (as a user), I would expect for Boost to use BOOST_ASSERT with an appropriate exception in debugging and diagnostic builds; and BOOST_VERIFY with an appropriate exception for release or production builds. </p> <p> Perhaps it would be a good idea to use boost/thread/pthread/mutex.hpp. It appears to be more mature and have a bit more stability. In addition, it throws lock exceptions where appropriate. </p> en-us Boost C++ Libraries /htdocs/site/boost.png https://svn.boost.org/trac10/ticket/8914 Trac 1.4.3 Joaquín M López Muñoz Sun, 11 Aug 2013 15:27:11 GMT status changed; resolution set https://svn.boost.org/trac10/ticket/8914#comment:1 https://svn.boost.org/trac10/ticket/8914#comment:1 <ul> <li><strong>status</strong> <span class="trac-field-old">new</span> → <span class="trac-field-new">closed</span> </li> <li><strong>resolution</strong> → <span class="trac-field-new">fixed</span> </li> </ul> <p> (In <a class="changeset" href="https://svn.boost.org/trac10/changeset/85302" title="fixed #8914">[85302]</a>) fixed <a class="closed ticket" href="https://svn.boost.org/trac10/ticket/8914" title="#8914: Bugs: boost/flyweight/detail/recursive_lw_mutex.hpp: Ignores all failures ... (closed: fixed)">#8914</a> </p> Ticket Joaquín M López Muñoz Sun, 11 Aug 2013 15:30:23 GMT <link>https://svn.boost.org/trac10/ticket/8914#comment:2 </link> <guid isPermaLink="false">https://svn.boost.org/trac10/ticket/8914#comment:2</guid> <description> <p> Hi Jeffrey, </p> <p> Fixed at </p> <p> <a class="ext-link" href="https://svn.boost.org/trac/boost/changeset/85302"><span class="icon">​</span>https://svn.boost.org/trac/boost/changeset/85302</a> </p> <p> I'd appreciate if you can check eveything's OK on your side. It is my understanding that any of these functions failing indicates a severe condition not recoverable even if exceptions are thrown, hence I've decided to simply add BOOST_VERIFY. </p> <p> Have you actually found a situation where boost/flyweight/detail/recursive_lw_mutex.hpp silently failed? </p> <p> Best, </p> </description> <category>Ticket</category> </item> </channel> </rss>