Re: [Discuss-gnuradio] It must be defined inside the context of a function !

[Marcus Müller]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Hi Activecat,

you're right. Method calls (or function calls in general) such us
push_back can only take place in functions. That's why constructors
exist! Please refer to your favourite C++ literature for more
information. As you understood correctly, this is a C++ problem, and
not really GR-related.
Your code won't compile even after moving these calls into a function,
because you yet have to replace the <+ +> placeholders.

Then: Working in namespace gr::howto is really deprecated. To get a
working, correctly named module infrastructure fitting your
installation of GR (which hopefully is either v3.6.5 or 3.7.X), please
use gr_modtool.


Greetings,
Marcus



On 16.01.2014 08:46, Activecat wrote:
> Dear Sir,
> 
> Below code will produce compilation error:  'input_sizes' does not
> name a type. This is because input_sizes must be assigned inside a
> context of a function in c++, right ?
> 
> std::vector<int> input_sizes; 
> input_sizes.push_back(sizeof(float)); 
> input_sizes.push_back(sizeof(double)); gr_sync_block("my block",
> gr_make_io_signaturev(2, 2, input_sizes), gr_make_io_signature(1,
> 1, sizeof(float)))
> 
> Source: 
> http://gnuradio.org/redmine/projects/gnuradio/wiki/BlocksCodingGuide#IO-signatures
>
> 
> 
> My code (which produce error) is below.
> 
> [code] /* -*- c++ -*- */
> 
> #ifdef HAVE_CONFIG_H #include "config.h" #endif
> 
> #include <gr_io_signature.h> #include <vector> #include <complex> 
> #include "sample_and_hold_cc_impl.h"
> 
> namespace gr { namespace howto {
> 
> sample_and_hold_cc::sptr sample_and_hold_cc::make() { return
> gnuradio::get_initial_sptr (new sample_and_hold_cc_impl()); }
> 
> /* * The private constructor */
> 
> std::vector <int> xx; xx.push_back(25); xx.push_back( sizeof( float
> ) ); xx.push_back( sizeof( std::complex<float> ) );
> 
> sample_and_hold_cc_impl::sample_and_hold_cc_impl() :
> gr_sync_block("sample_and_hold_cc", 
> gr_make_io_signature(<+MIN_IN+>, <+MAX_IN+>, sizeof(<+ITYPE+>)), 
> gr_make_io_signature(<+MIN_OUT+>, <+MAX_OUT+>, sizeof(<+OTYPE+>))) 
> {}
> 
> /* * Our virtual destructor. */ 
> sample_and_hold_cc_impl::~sample_and_hold_cc_impl() { }
> 
> int sample_and_hold_cc_impl::work(int noutput_items, 
> gr_vector_const_void_star &input_items, gr_vector_void_star
> &output_items) { const <+ITYPE+> *in = (const <+ITYPE+> *)
> input_items[0]; <+OTYPE+> *out = (<+OTYPE+> *) output_items[0];
> 
> // Do <+signal processing+>
> 
> // Tell runtime system how many output items we produced. return
> noutput_items; }
> 
> } /* namespace howto */ } /* namespace gr */ [/code]
> 
> 
> 
> _______________________________________________ Discuss-gnuradio
> mailing list address@hidden 
> https://lists.gnu.org/mailman/listinfo/discuss-gnuradio
> 
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/

iQEcBAEBAgAGBQJS15T5AAoJEAFxB7BbsDrLV2MH/2/h8v1nkK+80GrwJCzRDJkG
cb3qh3FJ890Xp0uZfj0RX+Gpv90wPas5Xia2KAb76QUOzS00J1M88vTG6kDYUaa1
R0Ak4m2S2Ehc1ir1AvZ0z0yZi4umKHAdD8mcCjPDSC05+Qq04QtjzfMtjFNyIMXV
872n9G0JBllRyfUlC2lRPMyCkRVSSLGfzLkShu25rlm4WImdfi+0uHHjhDjTAEV6
DwWm/5JuO0wtzzx3w7kKx0kD5COMUmsBZuWUjokcB1gr7MUKA2OKa1PdIa1a8t80
D1L+6S3Ax7m2LRVl8qJK4MqrvSKzDbzJLdqpPUHZjhim0m3r/Gp5etCqxSkir4M=
=5OP/
-----END PGP SIGNATURE-----