Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Replace all *printf's in the library with bladerf_log(), with configurable verbosity #31

Closed
jynik opened this issue Jul 25, 2013 · 2 comments
Closed

Replace all *printf's in the library with bladerf_log(), with configurable verbosity #31

jynik opened this issue Jul 25, 2013 · 2 comments
Assignees
@jynik
Labels
Issue: Enhancement Missing functionality, or a new feature request

Comments

@jynik
Copy link
Contributor

jynik commented Jul 25, 2013

Currently we have dbg_printf compile-time printfs. A more desirable approach would be to have healthy set of logging routines, and a configurable log-level.

int bladerf_log(level, ...);
int bladerf_set_log_verbosity(level);

int bladerf_set_log_output(FILE ); / Optional */
@ghost ghost assigned jynik Jul 25, 2013
@fjenner
Copy link
Contributor

fjenner commented Aug 26, 2013

I'll probably take a stab at this later in the week as a way to get my foot in the door on this project. Hopefully it's not an immediate priority right now...

@jynik
Copy link
Contributor Author

jynik commented Aug 30, 2013

Frank, would you also be interested in tackling issue #66?

jynik added a commit that referenced this issue Sep 2, 2013
@jynik
Merge pull request #74 from frank8371/issue31_squashed
cf1d772 
Added logging mechanism (Issue #31)
@Nuand Nuand closed this as completed Sep 6, 2013
rtucker added a commit that referenced this issue Aug 10, 2018
@rtucker
host: libbladeRF/bladerf2: remove RETURN_ERROR_STATUS_ARG (#31)
df36134 
it doesn't work in all situations, so it's not a particularly useful
macro
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jynik jynik

Labels
Issue: Enhancement Missing functionality, or a new feature request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jynik @fjenner