discuss-gnuradio
[Top][All Lists]
Advanced

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

Re: [Discuss-gnuradio] New structural change


From: Ben Hilburn
Subject: Re: [Discuss-gnuradio] New structural change
Date: Mon, 19 Dec 2011 10:46:57 -0800

I also like the structure change, and I think Michael's suggestion for handling the migration is perfect.

Cheers,
Ben


On Sun, Dec 18, 2011 at 5:30 PM, Michael Dickens <address@hidden> wrote:
On Dec 18, 2011, at 3:41 PM, Tom Rondeau wrote:
> The differences, though, coming in the naming scheme and installation
> method. Instead of having a gr_<name>.h file installed into
> $prefix/include/gnuradio, it's just <name>.h that is installed into
> $prefix/include/gnuradio/blocks. Likewise, the headers in gr-filter are
> installed into $prefix/include/gnuradio/filter.

I like this change in header include structure.  Most of the projects I work on do it this way, and it makes a lot of sense.  Because this will be a change that impacts just about everyone doing C++ development using GNU Radio, I think -if possible- it would be good to keep the "gr_<name>.h" headers around, but all they do is 2 things:

1) print out a #warning about being deprecated, and to instead use "blocks/<name>.h" -- or, whatever makes sense for such a message; and

2) #include "blocks/<name>.h" so that the user's code still compiles as before

That way, we benefit from the new header include structure, but aren't totally screwed if we've forgotten to use it.  And, somewhere down the road after an appropriate amount of time, the "gr_<name>.h" headers can be removed entirely, if desired, because everyone has been well warned.  I hope something along this line can be done.

- MLD


_______________________________________________
Discuss-gnuradio mailing list
address@hidden
https://lists.gnu.org/mailman/listinfo/discuss-gnuradio


reply via email to

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