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

host: Don't redefine inline unless __cplusplus is defined #365

Closed
jynik opened this issue Jan 22, 2015 · 1 comment
Closed

host: Don't redefine inline unless __cplusplus is defined #365

jynik opened this issue Jan 22, 2015 · 1 comment
Assignees
@jynik
Labels
Issue: Bug It's a bug and it oughta be fixed
Milestone
2015.01-rc2

Comments

@jynik
Copy link
Contributor

jynik commented Jan 22, 2015

See here:
https://github.com/Nuand/bladeRF/blob/master/host/common/include/host_config.h.in#L179

Should also check what MSVC 2013 does WRT inline now that the MS folks added some C99 support.
@jynik jynik added the Issue: Bug It's a bug and it oughta be fixed label Jan 22, 2015
@jynik jynik self-assigned this Jan 22, 2015
@jynik jynik modified the milestones: 2015.02, 2015.01-rc2 Jan 22, 2015
@jynik jynik changed the title host: Don't redefine inline unless __cpluspluse is defined host: Don't redefine inline unless __cplusplus is defined Jan 23, 2015
@jynik
Copy link
Contributor Author

jynik commented Jan 23, 2015

Closed in 4d5027b

@jynik jynik closed this as completed Jan 23, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jynik jynik

Labels
Issue: Bug It's a bug and it oughta be fixed
Projects
None yet
Milestone
2015.01-rc2
Development

No branches or pull requests
1 participant
@jynik