bistromath
commented
12 years ago Give it a shot now.
On Sun, Dec 11, 2011 at 8:58 AM, kd5bwt < reply@reply.github.com
wrote:
The uhd_modes.py change I'm pretty confident in. The gr_tags stuff had an uninitialized variable that appears to be related to a type change for the tags; here's my hack to make it work, but it's clearly not the correct solution, so feel free to reject it. I get a core dump from the pmt access at air_modes_preamble.cc line 86 (pmt::pmt_symbol_to_string(tstamp.key)) because the gr_tag_t passed to tag_to_timestamp() has not been initialized.
You can merge this Pull Request by running:
git pull https://github.com/kd5bwt/gr-air-modes master
Or you can view, comment on it, or merge it online at:
https://github.com/bistromath/gr-air-modes/pull/5
-- Commit Summary --
- fix --location to parse arg properly
- fix include; hack around uninit variable
-- File Changes --
M src/lib/air_modes_preamble.cc (5) M src/lib/air_modes_slicer.cc (2) M src/python/uhd_modes.py (2)
-- Patch Links --
https://github.com/bistromath/gr-air-modes/pull/5.patch https://github.com/bistromath/gr-air-modes/pull/5.diff
Reply to this email directly or view it on GitHub: https://github.com/bistromath/gr-air-modes/pull/5
kd5bwt
The uhd_modes.py change I'm pretty confident in. The gr_tags stuff had an uninitialized variable that appears to be related to a type change for the tags; here's my hack to make it work, but it's clearly not the correct solution, so feel free to reject it. I get a core dump from the pmt access at air_modes_preamble.cc line 86 (pmt::pmt_symbol_to_string(tstamp.key)) because the gr_tag_t passed to tag_to_timestamp() has not been initialized.