commit-gnuradio
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Commit-gnuradio] r3953 - in gnuradio/branches/developers/jcorgan/hier:


From: jcorgan
Subject: [Commit-gnuradio] r3953 - in gnuradio/branches/developers/jcorgan/hier: gnuradio-core/src/lib/runtime gnuradio-core/src/python/gnuradio/gr gnuradio-examples/c++/dialtone gnuradio-examples/c++/usrp_wfm_rcv gnuradio-examples/python/audio
Date: Tue, 7 Nov 2006 20:11:35 -0700 (MST)

Author: jcorgan
Date: 2006-11-07 20:11:35 -0700 (Tue, 07 Nov 2006)
New Revision: 3953

Modified:
   
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/lib/runtime/gr_basic_block.cc
   
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/lib/runtime/gr_hier_block2.cc
   
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/lib/runtime/gr_hier_block2.h
   
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/lib/runtime/gr_hier_block2.i
   
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/lib/runtime/gr_hier_block2_impl.cc
   
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/lib/runtime/gr_hier_block2_impl.h
   
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/lib/runtime/gr_runtime.cc
   
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/lib/runtime/gr_runtime.h
   
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/lib/runtime/gr_runtime_impl.cc
   
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/lib/runtime/gr_runtime_impl.h
   
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/lib/runtime/qa_gr_hier_block2.cc
   
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/lib/runtime/qa_gr_hier_block2.h
   
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/python/gnuradio/gr/qa_hier_block2.py
   
gnuradio/branches/developers/jcorgan/hier/gnuradio-examples/c++/dialtone/dialtone.cc
   
gnuradio/branches/developers/jcorgan/hier/gnuradio-examples/c++/dialtone/dialtone.h
   
gnuradio/branches/developers/jcorgan/hier/gnuradio-examples/c++/usrp_wfm_rcv/usrp_wfm_rcv.cc
   
gnuradio/branches/developers/jcorgan/hier/gnuradio-examples/c++/usrp_wfm_rcv/usrp_wfm_rcv.h
   
gnuradio/branches/developers/jcorgan/hier/gnuradio-examples/python/audio/dial_tone2.py
Log:
Work in progress.  Clean-up and stylistic changes, elimination of 
'update_connections' technique.

Modified: 
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/lib/runtime/gr_basic_block.cc
===================================================================
--- 
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/lib/runtime/gr_basic_block.cc
   2006-11-08 01:57:18 UTC (rev 3952)
+++ 
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/lib/runtime/gr_basic_block.cc
   2006-11-08 03:11:35 UTC (rev 3953)
@@ -30,7 +30,8 @@
 static long s_next_id = 0;
 static long s_ncurrently_allocated = 0;
 
-long gr_basic_block_ncurrently_allocated()
+long 
+gr_basic_block_ncurrently_allocated()
 {
     return s_ncurrently_allocated;
 }
@@ -51,7 +52,8 @@
     s_ncurrently_allocated--;
 }
 
-gr_basic_block_sptr gr_basic_block::basic_block()
+gr_basic_block_sptr 
+gr_basic_block::basic_block()
 {
     return shared_from_this();
 }

Modified: 
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/lib/runtime/gr_hier_block2.cc
===================================================================
--- 
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/lib/runtime/gr_hier_block2.cc
   2006-11-08 01:57:18 UTC (rev 3952)
+++ 
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/lib/runtime/gr_hier_block2.cc
   2006-11-08 03:11:35 UTC (rev 3953)
@@ -49,35 +49,38 @@
     delete d_impl;
 }
 
-void gr_hier_block2::define_component(const std::string name, 
gr_basic_block_sptr basic_block)
+void 
+gr_hier_block2::define_component(const std::string name, gr_basic_block_sptr 
basic_block)
 {
     d_impl->define_component(name, basic_block);
 }
 
-void gr_hier_block2::connect(const std::string src_name, int src_port, const 
std::string dst_name, int dst_port)
+void 
+gr_hier_block2::connect(const std::string src_name, int src_port, 
+                        const std::string dst_name, int dst_port)
 {
     gr_io_signature_sptr src_io_signature;
     gr_io_signature_sptr dst_io_signature;
 
     // Check against our *input_signature* if we're wiring from one of our 
external inputs
-    if (src_name != "self-in") {
-       gr_basic_block_sptr src_block = d_impl->lookup_component(src_name);
+    if (src_name == "self") 
+       src_io_signature = input_signature();
+    else {
+       gr_basic_block_sptr src_block = d_impl->get_block_by_name(src_name);
        if (!src_block)
            throw std::invalid_argument("Undefined src name");
        src_io_signature = src_block->output_signature();
     }
-    else
-       src_io_signature = input_signature();
     
     // Check against our *output_signature* if we're wiring to one of our 
external outputs
-    if (dst_name != "self-out") {
-       gr_basic_block_sptr dst_block = d_impl->lookup_component(dst_name);
+    if (dst_name == "self") 
+       dst_io_signature = output_signature();
+    else {
+       gr_basic_block_sptr dst_block = d_impl->get_block_by_name(dst_name);
        if (!dst_block)
            throw std::invalid_argument("Undefined dst name");
        dst_io_signature = dst_block->input_signature();
     }
-    else
-       dst_io_signature = output_signature();
     
     // Check port numbers are valid
     d_impl->check_valid_port(src_io_signature, src_port);
@@ -86,17 +89,11 @@
     // Check destination port not already in use
     d_impl->check_dst_not_used(dst_name, dst_port);
 
-    // Make sure endpoints have the same stream size
-    if (src_io_signature->sizeof_stream_item(src_port) !=
-        dst_io_signature->sizeof_stream_item(dst_port))
-       throw std::invalid_argument("type mismatch");
+    // Check endpoint types match
+    d_impl->check_type_match(src_io_signature, src_port,
+                            dst_io_signature, dst_port);
 
-    gr_connection connection(gr_endpoint(src_name, src_port), 
gr_endpoint(dst_name, dst_port));
+    gr_connection connection(gr_endpoint(src_name, src_port), 
+                             gr_endpoint(dst_name, dst_port));
     d_impl->d_connections.push_back(connection);
 }
-
-// Stub implementation, accept any topology, create no subcomponents
-bool gr_hier_block2::update_connections(int ninputs, int noutputs)
-{
-    return true;
-}

Modified: 
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/lib/runtime/gr_hier_block2.h
===================================================================
--- 
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/lib/runtime/gr_hier_block2.h
    2006-11-08 01:57:18 UTC (rev 3952)
+++ 
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/lib/runtime/gr_hier_block2.h
    2006-11-08 03:11:35 UTC (rev 3953)
@@ -22,9 +22,6 @@
 #ifndef INCLUDED_GR_HIER_BLOCK2_H
 #define INCLUDED_GR_HIER_BLOCK2_H
 
-// Define to 0 to eliminate debugging
-#define GR_HIER_BLOCK2_DEBUG 1
-
 #include <gr_basic_block.h>
 
 /*!
@@ -57,22 +54,18 @@
                   gr_io_signature_sptr input_signature,
                   gr_io_signature_sptr output_signature);
 
-    virtual bool update_connections(int ninputs, int noutputs);
-
 public:
     ~gr_hier_block2();
 
     void define_component(const std::string name, gr_basic_block_sptr 
basic_block);
-    void connect(const std::string from_name, int from_port, 
-                 const std::string to_name, int to_port);
+    void connect(const std::string src_name, int src_port, 
+                 const std::string dst_name, int dst_port);
 };
 
-#if GR_HIER_BLOCK2_DEBUG
 inline std::ostream &operator << (std::ostream &os, gr_basic_block_sptr 
basic_block)
 {
     os << basic_block->name() << "(" << basic_block->unique_id() << ")";
     return os;
 }
-#endif /* GR_HIER_BLOCK2_DEBUG */
 
 #endif /* INCLUDED_GR_HIER_BLOCK2_H */

Modified: 
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/lib/runtime/gr_hier_block2.i
===================================================================
--- 
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/lib/runtime/gr_hier_block2.i
    2006-11-08 01:57:18 UTC (rev 3952)
+++ 
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/lib/runtime/gr_hier_block2.i
    2006-11-08 03:11:35 UTC (rev 3953)
@@ -49,7 +49,7 @@
     // Add a named block to the container
     void define_component(const std::string name, gr_basic_block_sptr 
basic_block)
        throw (std::invalid_argument);
-    void connect(const std::string from_name, int from_port,
-                const std::string to_name, int to_port)
+    void connect(const std::string src_name, int src_port,
+                const std::string dst_name, int dst_port)
        throw (std::invalid_argument);
 };

Modified: 
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/lib/runtime/gr_hier_block2_impl.cc
===================================================================
--- 
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/lib/runtime/gr_hier_block2_impl.cc
      2006-11-08 01:57:18 UTC (rev 3952)
+++ 
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/lib/runtime/gr_hier_block2_impl.cc
      2006-11-08 03:11:35 UTC (rev 3953)
@@ -36,21 +36,24 @@
 {
 }
 
-void gr_hier_block2_impl::define_component(const std::string &name, 
gr_basic_block_sptr basic_block)
+void 
+gr_hier_block2_impl::define_component(const std::string &name, 
+                                      gr_basic_block_sptr basic_block)
 {
-    if (name == "self-in" || name == "self-out")
+    if (name == "self")
        throw std::invalid_argument("name is reserved");
 
-    if (lookup_component(name))
-       throw std::invalid_argument("Name already in use");
+    if (get_block_by_name(name))
+       throw std::invalid_argument("name already in use");
        
-    if (lookup_component(basic_block) != "")
-       throw std::invalid_argument("Component already defined");
+    if (get_name_by_block(basic_block) != "")
+       throw std::invalid_argument("component already defined");
 
     d_components.push_back(gr_hier_component(name, basic_block));
 }
 
-gr_basic_block_sptr gr_hier_block2_impl::lookup_component(const std::string 
&name)
+gr_basic_block_sptr 
+gr_hier_block2_impl::get_block_by_name(const std::string &name)
 {
     for (std::vector<gr_hier_component>::const_iterator p = 
d_components.begin();
         p != d_components.end(); p++) {
@@ -61,7 +64,8 @@
     return gr_basic_block_sptr();
 }
 
-std::string gr_hier_block2_impl::lookup_component(gr_basic_block_sptr 
basic_block)
+std::string 
+gr_hier_block2_impl::get_name_by_block(gr_basic_block_sptr basic_block)
 {
     for (std::vector<gr_hier_component>::const_iterator p = 
d_components.begin();
         p != d_components.end(); p++) {
@@ -72,7 +76,8 @@
     return std::string("");
 }
 
-void gr_hier_block2_impl::check_valid_port(gr_io_signature_sptr sig, int port)
+void 
+gr_hier_block2_impl::check_valid_port(gr_io_signature_sptr sig, int port)
 {
     if (port < 0)
        throw std::invalid_argument("port number must not be negative");
@@ -81,7 +86,8 @@
        throw std::invalid_argument("port number exceeds max streams");
 }
 
-void gr_hier_block2_impl::check_dst_not_used(const std::string name, int port)
+void 
+gr_hier_block2_impl::check_dst_not_used(const std::string name, int port)
 {
     for (std::vector<gr_connection>::const_iterator p = d_connections.begin();
         p != d_connections.end(); p++) {
@@ -90,3 +96,12 @@
            throw std::invalid_argument("destination port in use");
     }
 }
+
+void 
+gr_hier_block2_impl::check_type_match(gr_io_signature_sptr src_sig, int 
src_port,
+                                     gr_io_signature_sptr dst_sig, int 
dst_port)
+{
+    if (src_sig->sizeof_stream_item(src_port) !=
+        dst_sig->sizeof_stream_item(dst_port))
+       throw std::invalid_argument("type mismatch");
+}

Modified: 
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/lib/runtime/gr_hier_block2_impl.h
===================================================================
--- 
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/lib/runtime/gr_hier_block2_impl.h
       2006-11-08 01:57:18 UTC (rev 3952)
+++ 
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/lib/runtime/gr_hier_block2_impl.h
       2006-11-08 03:11:35 UTC (rev 3953)
@@ -22,12 +22,10 @@
 #ifndef INCLUDED_GR_HIER_BLOCK2_IMPL_H
 #define INCLUDED_GR_HIER_BLOCK2_IMPL_H
 
-// Define to 0 to eliminate debugging
-#define GR_HIER_BLOCK2_IMPL_DEBUG 1
-
 #include <gr_basic_block.h>
 #include <stdexcept>
 #include <iostream>
+#include <boost/utility.hpp>
 
 // This beats the hell out of using pairs
 class gr_hier_component
@@ -72,30 +70,27 @@
     gr_endpoint dst() const { return d_dst; }
 };
 
-class gr_hier_block2_impl
+class gr_hier_block2_impl : boost::noncopyable
 {
 private:
     friend class gr_hier_block2;
     gr_hier_block2_impl();
 
-    // Not implemented, making private makes class noncopyable
-    gr_hier_block2_impl(const gr_hier_block2_impl &rhs);
-    gr_hier_block2_impl &operator=(const gr_hier_block2_impl &rhs);
-
     std::vector<gr_hier_component> d_components;
     std::vector<gr_connection> d_connections;
     
     void define_component(const std::string &name, gr_basic_block_sptr 
basic_block);
-    gr_basic_block_sptr lookup_component(const std::string &name);
-    std::string lookup_component(gr_basic_block_sptr basic_block);
+    gr_basic_block_sptr get_block_by_name(const std::string &name);
+    std::string get_name_by_block(gr_basic_block_sptr basic_block);
     void check_valid_port(gr_io_signature_sptr sig, int port);
     void check_dst_not_used(const std::string name, int port);
-                    
+    void check_type_match(gr_io_signature_sptr src_sig, int src_port,
+                         gr_io_signature_sptr dst_sig, int dst_port);
+                            
 public:
     ~gr_hier_block2_impl();
 };
 
-#if GR_HIER_BLOCK2_IMPL_DEBUG
 inline std::ostream &operator << (std::ostream &os, const gr_endpoint endpoint)
 {
     os << endpoint.name() << ":" << endpoint.port();
@@ -107,6 +102,5 @@
     os << connection.src() << "->" << connection.dst();
     return os;
 }
-#endif /* GR_HIER_BLOCK2_IMPL_DEBUG */
 
 #endif /* INCLUDED_GR_HIER_BLOCK2_H */

Modified: 
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/lib/runtime/gr_runtime.cc
===================================================================
--- 
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/lib/runtime/gr_runtime.cc
       2006-11-08 01:57:18 UTC (rev 3952)
+++ 
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/lib/runtime/gr_runtime.cc
       2006-11-08 03:11:35 UTC (rev 3953)
@@ -28,7 +28,8 @@
 #include <gr_runtime_impl.h>
 #include <iostream.h>
 
-gr_runtime_sptr gr_make_runtime(gr_hier_block2_sptr top_block)
+gr_runtime_sptr 
+gr_make_runtime(gr_hier_block2_sptr top_block)
 {
     return gr_runtime_sptr(new gr_runtime(top_block));
 }
@@ -43,22 +44,26 @@
     delete d_impl;
 }
 
-void gr_runtime::start()
+void 
+gr_runtime::start()
 {
     d_impl->start();
 }
 
-void gr_runtime::stop()
+void 
+gr_runtime::stop()
 {
     d_impl->stop();
 }
 
-void gr_runtime::wait()
+void 
+gr_runtime::wait()
 {
     d_impl->wait();
 }
 
-void gr_runtime::run()
+void 
+gr_runtime::run()
 {
     start();
     wait();

Modified: 
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/lib/runtime/gr_runtime.h
===================================================================
--- 
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/lib/runtime/gr_runtime.h
        2006-11-08 01:57:18 UTC (rev 3952)
+++ 
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/lib/runtime/gr_runtime.h
        2006-11-08 03:11:35 UTC (rev 3953)
@@ -23,9 +23,6 @@
 #ifndef INCLUDED_GR_RUNTIME_H
 #define INCLUDED_GR_RUNTIME_H
 
-// Define to 0 to eliminate runtime debugging
-#define GR_RUNTIME_DEBUG 1
-
 #include <gr_runtime_types.h>
 
 class gr_runtime_impl;

Modified: 
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/lib/runtime/gr_runtime_impl.cc
===================================================================
--- 
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/lib/runtime/gr_runtime_impl.cc
  2006-11-08 01:57:18 UTC (rev 3952)
+++ 
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/lib/runtime/gr_runtime_impl.cc
  2006-11-08 03:11:35 UTC (rev 3953)
@@ -24,6 +24,8 @@
 #include "config.h"
 #endif
 
+#define GR_RUNTIME_IMPL_DEBUG 1
+
 #include <gr_runtime_impl.h>
 #include <stdexcept>
 #include <iostream>
@@ -32,17 +34,14 @@
 d_top_block(top_block),
 d_running(false)
 {
-    if (GR_RUNTIME_IMPL_DEBUG)
-       std::cout << "gr_runtime_impl::gr_runtime_impl()" << std::endl;
 }
   
 gr_runtime_impl::~gr_runtime_impl()
 {
-    if (GR_RUNTIME_IMPL_DEBUG)
-       std::cout << "gr_runtime_impl::~gr_runtime_impl()" << std::endl;
 }
 
-void gr_runtime_impl::start()
+void 
+gr_runtime_impl::start()
 {
     if (GR_RUNTIME_IMPL_DEBUG)
        std::cout << "gr_runtime_impl::start()" << std::endl;
@@ -55,7 +54,8 @@
     // Start
 }
 
-void gr_runtime_impl::stop()
+void 
+gr_runtime_impl::stop()
 {
     if (GR_RUNTIME_IMPL_DEBUG)
        std::cout << "gr_runtime_impl::stop()" << std::endl;
@@ -67,7 +67,8 @@
     d_running = false;    
 }
 
-void gr_runtime_impl::wait()
+void 
+gr_runtime_impl::wait()
 {
     if (GR_RUNTIME_IMPL_DEBUG)
        std::cout << "gr_runtime_impl::wait()" << std::endl;

Modified: 
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/lib/runtime/gr_runtime_impl.h
===================================================================
--- 
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/lib/runtime/gr_runtime_impl.h
   2006-11-08 01:57:18 UTC (rev 3952)
+++ 
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/lib/runtime/gr_runtime_impl.h
   2006-11-08 03:11:35 UTC (rev 3953)
@@ -25,9 +25,6 @@
 
 #include <gr_runtime_types.h>
 
-// Define to 0 to eliminate runtime debugging
-#define GR_RUNTIME_IMPL_DEBUG 1
-
 class gr_runtime_impl
 {
 private:

Modified: 
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/lib/runtime/qa_gr_hier_block2.cc
===================================================================
--- 
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/lib/runtime/qa_gr_hier_block2.cc
        2006-11-08 01:57:18 UTC (rev 3952)
+++ 
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/lib/runtime/qa_gr_hier_block2.cc
        2006-11-08 03:11:35 UTC (rev 3953)
@@ -1,6 +1,6 @@
 /* -*- c++ -*- */
 /*
- * Copyright 2004 Free Software Foundation, Inc.
+ * Copyright 2006 Free Software Foundation, Inc.
  * 
  * This file is part of GNU Radio
  * 
@@ -30,53 +30,6 @@
 #include <gr_null_source.h>
 #include <gr_null_sink.h>
 
-// gr_hier_block2 functionality only available from within derived class, so 
make one
-class gr_test_block;
-typedef boost::shared_ptr<gr_test_block> gr_test_block_sptr;
-gr_test_block_sptr gr_make_test_block();
-
-class gr_test_block : public gr_hier_block2
-{
-private:
-    gr_test_block();
-    friend gr_test_block_sptr gr_make_test_block();
-
-    bool update_connections(int inputs, int outputs);
-    
-public:
-    ~gr_test_block();
-};
-
-gr_test_block_sptr gr_make_test_block()
-{
-    return gr_test_block_sptr(new gr_test_block());
-}
-
-gr_test_block::gr_test_block()
-  : gr_hier_block2("test_block",
-                  gr_make_io_signature(1, 1, sizeof(int)),
-                  gr_make_io_signature(1, 1, sizeof(int)))
-{
-    define_component("source", gr_make_null_source(sizeof(int)));
-    define_component("sink", gr_make_null_sink(sizeof(int)));
-}
-
-bool gr_test_block::update_connections(int inputs, int outputs)
-{
-    if (inputs != 1 || outputs != 1)
-       return false;
-
-    connect("self-in", 0, "source", 0);
-    connect("source", 0, "sink", 0);
-    connect("sink", 0, "self-out", 0);
-       
-    return true;
-}
-
-gr_test_block::~gr_test_block()
-{
-}
-
 void qa_gr_hier_block2::test_make()
 {
     gr_hier_block2_sptr src1(gr_make_hier_block2("test",
@@ -91,16 +44,3 @@
     CPPUNIT_ASSERT_EQUAL(sizeof(int), 
                         src1->output_signature()->sizeof_stream_item(0));
 }
-
-void qa_gr_hier_block2::test_derived()
-{
-    gr_test_block_sptr blk1(gr_make_test_block());
-    
-    CPPUNIT_ASSERT(blk1);
-    CPPUNIT_ASSERT_EQUAL(std::string("test_block"), blk1->name());
-    CPPUNIT_ASSERT_EQUAL(1, blk1->input_signature()->max_streams());
-    CPPUNIT_ASSERT_EQUAL(1, blk1->output_signature()->min_streams());
-    CPPUNIT_ASSERT_EQUAL(1, blk1->output_signature()->max_streams());
-    CPPUNIT_ASSERT_EQUAL(sizeof(int), 
-                        blk1->output_signature()->sizeof_stream_item(0));
-}

Modified: 
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/lib/runtime/qa_gr_hier_block2.h
===================================================================
--- 
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/lib/runtime/qa_gr_hier_block2.h
 2006-11-08 01:57:18 UTC (rev 3952)
+++ 
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/lib/runtime/qa_gr_hier_block2.h
 2006-11-08 03:11:35 UTC (rev 3953)
@@ -32,7 +32,6 @@
     CPPUNIT_TEST_SUITE(qa_gr_hier_block2);
 
     CPPUNIT_TEST(test_make);
-    CPPUNIT_TEST(test_derived);
     
     CPPUNIT_TEST_SUITE_END();
 

Modified: 
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/python/gnuradio/gr/qa_hier_block2.py
===================================================================
--- 
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/python/gnuradio/gr/qa_hier_block2.py
    2006-11-08 01:57:18 UTC (rev 3952)
+++ 
gnuradio/branches/developers/jcorgan/hier/gnuradio-core/src/python/gnuradio/gr/qa_hier_block2.py
    2006-11-08 03:11:35 UTC (rev 3953)
@@ -37,15 +37,8 @@
                                gr.io_signature(1,1,gr.sizeof_int), 
                                gr.io_signature(1,1,gr.sizeof_int))
        src1 = gr.nop(gr.sizeof_int)
-       self.assertRaises(ValueError, lambda: 
hblock.define_component("self-in", src1))
+       self.assertRaises(ValueError, lambda: hblock.define_component("self", 
src1))
 
-    def test_004_define_component_reserved_output(self):
-       hblock = gr.hier_block2("test_block", 
-                               gr.io_signature(1,1,gr.sizeof_int), 
-                               gr.io_signature(1,1,gr.sizeof_int))
-       src1 = gr.nop(gr.sizeof_int)
-       self.assertRaises(ValueError, lambda: 
hblock.define_component("self-out", src1))
-
     def test_005_define_component_name_in_use(self):
        hblock = gr.hier_block2("test_block", 
                                gr.io_signature(1,1,gr.sizeof_int), 
@@ -85,7 +78,7 @@
 
        nop1 = gr.nop(gr.sizeof_int)
        hblock.define_component("nop1", nop1)
-       hblock.connect("self-in", 0, "nop1", 0)
+       hblock.connect("self", 0, "nop1", 0)
 
     def test_009_connect_output(self):
        hblock = gr.hier_block2("test_block", 
@@ -94,7 +87,7 @@
 
        nop1 = gr.nop(gr.sizeof_int)
        hblock.define_component("nop1", nop1)
-       hblock.connect("nop1", 0, "self-out", 0)
+       hblock.connect("nop1", 0, "self", 0)
 
     def test_010_connect_unknown_src(self):
        hblock = gr.hier_block2("test_block", 
@@ -103,7 +96,7 @@
 
        nop1 = gr.nop(gr.sizeof_int)
        hblock.define_component("nop1", nop1)
-       self.assertRaises(ValueError, lambda: hblock.connect("nop2", 0, 
"self-out", 0))
+       self.assertRaises(ValueError, lambda: hblock.connect("nop2", 0, "self", 
0))
     
     def test_011_connect_unknown_dst(self):
        hblock = gr.hier_block2("test_block", 
@@ -112,7 +105,7 @@
 
        nop1 = gr.nop(gr.sizeof_int)
        hblock.define_component("nop1", nop1)
-       self.assertRaises(ValueError, lambda: hblock.connect("self-in", 0, 
"nop2", 0))
+       self.assertRaises(ValueError, lambda: hblock.connect("self", 0, "nop2", 
0))
 
     def test_012_connect_invalid_src_port_neg(self):
        hblock = gr.hier_block2("test_block", 
@@ -121,7 +114,7 @@
 
        nop1 = gr.nop(gr.sizeof_int)
        hblock.define_component("nop1", nop1)
-       self.assertRaises(ValueError, lambda: hblock.connect("self-in", -1, 
"nop1", 0))
+       self.assertRaises(ValueError, lambda: hblock.connect("self", -1, 
"nop1", 0))
 
     def test_013_connect_invalid_src_port_exceeds(self):
        hblock = gr.hier_block2("test_block", 
@@ -130,7 +123,7 @@
 
        nop1 = gr.nop(gr.sizeof_int)
        hblock.define_component("nop1", nop1)
-       self.assertRaises(ValueError, lambda: hblock.connect("self-in", 1, 
"nop1", 0))
+       self.assertRaises(ValueError, lambda: hblock.connect("self", 1, "nop1", 
0))
     
     def test_014_connect_invalid_dst_port_neg(self):
        hblock = gr.hier_block2("test_block", 
@@ -139,7 +132,7 @@
 
        nop1 = gr.nop(gr.sizeof_int)
        hblock.define_component("nop1", nop1)
-       self.assertRaises(ValueError, lambda: hblock.connect("self-in", -1, 
"nop1", 0))
+       self.assertRaises(ValueError, lambda: hblock.connect("self", -1, 
"nop1", 0))
 
     def test_015_connect_invalid_dst_port_exceeds(self):
        hblock = gr.hier_block2("test_block", 
@@ -148,7 +141,7 @@
 
        nop1 = gr.nop(gr.sizeof_int)
        hblock.define_component("nop1", nop1)
-       self.assertRaises(ValueError, lambda: hblock.connect("self-in", 1, 
"nop1", 0))
+       self.assertRaises(ValueError, lambda: hblock.connect("self", 1, "nop1", 
0))
 
     def test_016_connect_dst_port_in_use(self):
        hblock = gr.hier_block2("test_block", 
@@ -157,8 +150,8 @@
 
        nop1 = gr.nop(gr.sizeof_int)
        hblock.define_component("nop1", nop1)
-       hblock.connect("nop1", 0, "self-out", 0);
-       self.assertRaises(ValueError, lambda: hblock.connect("nop1", 0, 
"self-out", 0))
+       hblock.connect("nop1", 0, "self", 0);
+       self.assertRaises(ValueError, lambda: hblock.connect("nop1", 0, "self", 
0))
     
     def test_017_connect_type_mismatch(self):
        hblock = gr.hier_block2("test_block", 
@@ -167,7 +160,7 @@
 
        nop1 = gr.nop(gr.sizeof_char)
        hblock.define_component("nop1", nop1)
-       self.assertRaises(ValueError, lambda: hblock.connect("nop1", 0, 
"self-out", 0))
+       self.assertRaises(ValueError, lambda: hblock.connect("nop1", 0, "self", 
0))
 
 if __name__ == "__main__":
     gr_unittest.main()

Modified: 
gnuradio/branches/developers/jcorgan/hier/gnuradio-examples/c++/dialtone/dialtone.cc
===================================================================
--- 
gnuradio/branches/developers/jcorgan/hier/gnuradio-examples/c++/dialtone/dialtone.cc
        2006-11-08 01:57:18 UTC (rev 3952)
+++ 
gnuradio/branches/developers/jcorgan/hier/gnuradio-examples/c++/dialtone/dialtone.cc
        2006-11-08 03:11:35 UTC (rev 3953)
@@ -39,17 +39,7 @@
     define_component("source0", gr_make_sig_source_f(48000, GR_SIN_WAVE, 350, 
0.5));
     define_component("source1", gr_make_sig_source_f(48000, GR_SIN_WAVE, 440, 
0.5));
     define_component("sink", audio_alsa_make_sink(48000));
-}
 
-// Called by runtime whenever connectivity changes, at least once
-// on startup.
-bool dialtone::update_connections(int ninputs, int noutputs)
-{
-    if (ninputs || noutputs) // Top-level blocks have no external connections
-       return false;   
-
     connect("source0", 0, "sink", 0);
     connect("source1", 0, "sink", 1);    
-
-    return true;
-}    
+}

Modified: 
gnuradio/branches/developers/jcorgan/hier/gnuradio-examples/c++/dialtone/dialtone.h
===================================================================
--- 
gnuradio/branches/developers/jcorgan/hier/gnuradio-examples/c++/dialtone/dialtone.h
 2006-11-08 01:57:18 UTC (rev 3952)
+++ 
gnuradio/branches/developers/jcorgan/hier/gnuradio-examples/c++/dialtone/dialtone.h
 2006-11-08 03:11:35 UTC (rev 3953)
@@ -30,6 +30,4 @@
 private:
     dialtone();
     friend dialtone_sptr make_dialtone();
-
-    bool update_connections(int inputs, int outputs);
 };

Modified: 
gnuradio/branches/developers/jcorgan/hier/gnuradio-examples/c++/usrp_wfm_rcv/usrp_wfm_rcv.cc
===================================================================
--- 
gnuradio/branches/developers/jcorgan/hier/gnuradio-examples/c++/usrp_wfm_rcv/usrp_wfm_rcv.cc
        2006-11-08 01:57:18 UTC (rev 3952)
+++ 
gnuradio/branches/developers/jcorgan/hier/gnuradio-examples/c++/usrp_wfm_rcv/usrp_wfm_rcv.cc
        2006-11-08 03:11:35 UTC (rev 3953)
@@ -57,17 +57,9 @@
                                          USRP_FPGA_FILENAME,
                                          USRP_FIRMWARE_FILENAME));
     define_component("quad", gr_make_quadrature_demod_cf(demod_gain));
-    define_component("null", gr_make_null_sink(sizeof(gr_complex)));
+    define_component("null", gr_make_null_sink(sizeof(float)));
     define_component("sink", audio_alsa_make_sink(48000));
-}
 
-bool usrp_wfm_rcv::update_connections(int ninputs, int noutputs)
-{
-    if (ninputs || noutputs) // Top level blocks shouldn't have any inputs or 
outputs
-       return false;
-           
     connect("src", 0, "quad", 0);
     connect("quad", 0, "null", 0);
-
-    return true;
 }

Modified: 
gnuradio/branches/developers/jcorgan/hier/gnuradio-examples/c++/usrp_wfm_rcv/usrp_wfm_rcv.h
===================================================================
--- 
gnuradio/branches/developers/jcorgan/hier/gnuradio-examples/c++/usrp_wfm_rcv/usrp_wfm_rcv.h
 2006-11-08 01:57:18 UTC (rev 3952)
+++ 
gnuradio/branches/developers/jcorgan/hier/gnuradio-examples/c++/usrp_wfm_rcv/usrp_wfm_rcv.h
 2006-11-08 03:11:35 UTC (rev 3953)
@@ -30,6 +30,4 @@
 private:
     usrp_wfm_rcv();
     friend usrp_wfm_rcv_sptr usrp_make_wfm_rcv();
-
-    bool update_connections(int ninputs, int noutputs);
 };

Modified: 
gnuradio/branches/developers/jcorgan/hier/gnuradio-examples/python/audio/dial_tone2.py
===================================================================
--- 
gnuradio/branches/developers/jcorgan/hier/gnuradio-examples/python/audio/dial_tone2.py
      2006-11-08 01:57:18 UTC (rev 3952)
+++ 
gnuradio/branches/developers/jcorgan/hier/gnuradio-examples/python/audio/dial_tone2.py
      2006-11-08 03:11:35 UTC (rev 3953)
@@ -48,14 +48,8 @@
        self.define_component("source0", gr.sig_source_f (sample_rate, 
gr.GR_SIN_WAVE, 350, ampl))
        self.define_component("source1", gr.sig_source_f (sample_rate, 
gr.GR_SIN_WAVE, 440, ampl))
        self.define_component("dest", audio.sink(sample_rate, 
options.audio_output))
-
-    def update_connections(self, ninputs, noutputs):
-       if ninputs or noutputs: # Top level blocks shouldn't have inputs or 
outputs
-           return False
-
        self.connect("source0", 0, "dest", 0)   
        self.connect("source1", 0, "dest", 1)
-       return True
        
 if __name__ == '__main__':
     top_block = dial_tone()





reply via email to

[Prev in Thread] Current Thread [Next in Thread]