Skip to content

Fix compiler warning when MIC_DUAL_ENABLED is not defined#127

Merged
oscarbailey-xmos merged 2 commits intodevelopfrom
feature/fix-compiler-warning
Jan 13, 2021
Merged

Fix compiler warning when MIC_DUAL_ENABLED is not defined#127
oscarbailey-xmos merged 2 commits intodevelopfrom
feature/fix-compiler-warning

Conversation

@oscarbailey-xmos
Copy link
Copy Markdown
Contributor

@oscarbailey-xmos oscarbailey-xmos commented Dec 8, 2020 

/home/oscar/sandboxes/usb_audio/lib_mic_array/lib_mic_array/src/mic_array_dual.xc:282:12: warning: 'dc_eliminate' defined but not used [-Wunused-function]
static int dc_eliminate(int x, int &prev_x, long long &state){
           ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

@oscarbailey-xmos oscarbailey-xmos requested review from ed-xmos, mbanth and xluciano and removed request for xluciano December 8, 2020 16:28
mbanth
mbanth approved these changes Dec 8, 2020
View reviewed changes
Copy link
Copy Markdown
Contributor

@mbanth mbanth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@oscarbailey-xmos oscarbailey-xmos merged commit 1ac7440 into develop Jan 13, 2021
@oscarbailey-xmos oscarbailey-xmos deleted the feature/fix-compiler-warning branch January 13, 2021 16:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mbanth mbanth mbanth approved these changes

@ed-xmos ed-xmos Awaiting requested review from ed-xmos

@xluciano xluciano Awaiting requested review from xluciano

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@oscarbailey-xmos @mbanth