On Wed, Jan 23, 2013 at 7:08 AM, Tom Rondeau <[email protected]> wrote:


> Unfortunately, they have also deprecated Mutex::scoped_lock, so we're
> going to have to reevaluate our use of that.... (not a show-stopper by any
> means, just some changes we'll have to make).
>

It looks like we are using the gruel::scoped_lock typedef everywhere except
in the gr_file_source and gr::file_source blocks where bare
boost::mutex::scoped_lock is used.  I did a quick test of the below change
to use the recommended type and at least all the QA unit tests pass:

--- a/gruel/src/include/gruel/thread.h
+++ b/gruel/src/include/gruel/thread.h
@@ -31,7 +31,7 @@ namespace gruel {

   typedef boost::thread                    thread;
   typedef boost::mutex                     mutex;
-  typedef boost::mutex::scoped_lock        scoped_lock;
+  typedef boost::unique_lock<boost::mutex> scoped_lock;
   typedef boost::condition_variable        condition_variable;

 } /* namespace gruel */

So when the time comes it should be easy to switch over (and fix the bare
usage in the file source.)

Johnathan
_______________________________________________
Discuss-gnuradio mailing list
[email protected]
https://lists.gnu.org/mailman/listinfo/discuss-gnuradio

Reply via email to