JvanKatwijk / dab-cmdline

DAB decoding library with example of its use
GNU General Public License v2.0
57 stars 29 forks source link

Problem with main.cpp building example-3 #63

Closed montgomeryb closed 5 years ago

montgomeryb commented 5 years ago

Hello, excellent project, thank you. In trying to build example-3, I ran in to a problem:

src/dab-cmdline/example-3/main.cpp: In function ‘int main(int, char**)’:
src/dab-cmdline/example-3/main.cpp:324:11: error: ‘theGain’ was not declared in this scope
           theGain = atoi (optarg);

After looking at some of the code, I added a definition in to main.cpp:

int             deviceGain      = 45;   // scale = 0 .. 100
int             theGain         = deviceGain;

This seemed to resolve the issue. Regards, Bryan

JvanKatwijk commented 5 years ago

Hello

Yes I am afraid some of the main programs became a little messy by incrementally supporting devices, each with different settings for gain. I myself am using example 2, that is reasonably OK. I'll give it a try next week - have to make some other changes as well - Thanks for the message

best jan

Op vr 9 aug. 2019 om 14:57 schreef montgomeryb notifications@github.com:

Hello, excellent project, thank you. In trying to build example-3, I ran in to a problem: [ 4%] Building CXX object CMakeFiles/dab-rtlsdr-3.dir/main.cpp.o src/dab-cmdline/example-3/main.cpp: In function ‘int main(int, char**)’: src/dab-cmdline/example-3/main.cpp:324:11: error: ‘theGain’ was not declared in this scope theGain = atoi (optarg);

After looking at some of the code, I added a definition in to main.cpp: int deviceGain = 45; // scale = 0 .. 100 int theGain = deviceGain; This seemed to resolve the issue. Regards, Bryan

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/JvanKatwijk/dab-cmdline/issues/63?email_source=notifications&email_token=ACCPHQFW37RQP2DVAMYXKGDQDVST7A5CNFSM4IKTYYS2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HEMR5QQ, or mute the thread https://github.com/notifications/unsubscribe-auth/ACCPHQD46JOTUFJ7Y2OW43DQDVST7ANCNFSM4IKTYYSQ .

-- Jan van Katwijk

JvanKatwijk commented 5 years ago

I rewrote (part of the) main program for example 3, such that the command options are the same as for example 3 (of course with the exception of the setting of the audio card). Furthermore now the LimeSDR is part of the configuration

montgomeryb commented 5 years ago

Hello, I just pulled the latest version. I did have a problem compiling it that took me a while to find out why .... I did not have the en_us.UTF8 locale defined so it gave me: dab_cmdline example III, Copyright 2017 J van Katwijk, Lazy Chair Computing terminate called after throwing an instance of 'std::runtime_error' what(): locale::facet::_S_create_c_locale name not valid Aborted

After doing some research, I configured the locale and the program runs fine. Now, I just need to get a consistent DAB signal with my setup.

Thanks for the update. My end goal at the moment is to hope to be able to stream a DAB channel to icecast, but again I first need to improve my reception.

Bryan.

On Sun, Aug 11, 2019 at 2:24 AM JvanKatwijk notifications@github.com wrote:

I rewrote (part of the) main program for example 3, such that the command options are the same as for example 3 (of course with the exception of the setting of the audio card). Furthermore now the LimeSDR is part of the configuration

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/JvanKatwijk/dab-cmdline/issues/63?email_source=notifications&email_token=AAMY45PONQFFYLDJK2ZPAD3QD6V75A5CNFSM4IKTYYS2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4A24JI#issuecomment-520203813, or mute the thread https://github.com/notifications/unsubscribe-auth/AAMY45LOKNVMU762CSUBFDTQD6V75ANCNFSM4IKTYYSQ .

JvanKatwijk commented 5 years ago

Yes, I think the locale lines should be changed to just calling in the "local" locale std::setlocale (LC_ALL, "");

jan

Op zo 11 aug. 2019 om 14:50 schreef montgomeryb notifications@github.com:

Hello, I just pulled the latest version. I did have a problem compiling it that took me a while to find out why .... I did not have the en_us.UTF8 locale defined so it gave me: dab_cmdline example III, Copyright 2017 J van Katwijk, Lazy Chair Computing terminate called after throwing an instance of 'std::runtime_error' what(): locale::facet::_S_create_c_locale name not valid Aborted

After doing some research, I configured the locale and the program runs fine. Now, I just need to get a consistent DAB signal with my setup.

Thanks for the update. My end goal at the moment is to hope to be able to stream a DAB channel to icecast, but again I first need to improve my reception.

Bryan.

On Sun, Aug 11, 2019 at 2:24 AM JvanKatwijk notifications@github.com wrote:

I rewrote (part of the) main program for example 3, such that the command options are the same as for example 3 (of course with the exception of the setting of the audio card). Furthermore now the LimeSDR is part of the configuration

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub < https://github.com/JvanKatwijk/dab-cmdline/issues/63?email_source=notifications&email_token=AAMY45PONQFFYLDJK2ZPAD3QD6V75A5CNFSM4IKTYYS2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4A24JI#issuecomment-520203813 , or mute the thread < https://github.com/notifications/unsubscribe-auth/AAMY45LOKNVMU762CSUBFDTQD6V75ANCNFSM4IKTYYSQ

.

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/JvanKatwijk/dab-cmdline/issues/63?email_source=notifications&email_token=ACCPHQGFBCGOFCNULQG2NDLQEADIHA5CNFSM4IKTYYS2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4BAJ2A#issuecomment-520226024, or mute the thread https://github.com/notifications/unsubscribe-auth/ACCPHQG6D42SBCPKSRZN4ADQEADIHANCNFSM4IKTYYSQ .

-- Jan van Katwijk