Skip to content

libfaad/fixed.h: Fix ISO C warning#211

Merged
fabiangreffrath merged 1 commit intoknik0:masterfrom
passgat:fix-pedantic-warning
Oct 1, 2025
Merged

libfaad/fixed.h: Fix ISO C warning#211
fabiangreffrath merged 1 commit intoknik0:masterfrom
passgat:fix-pedantic-warning

Conversation

@passgat
Copy link
Copy Markdown
Contributor

@passgat passgat commented Oct 1, 2025

Building faad2 in Buildroot raises the following warning:

In file included from buildroot/output/build/faad2-2.11.2/libfaad/common.h:255,
from buildroot/output/build/faad2-2.11.2/libfaad/bits.c:31:
buildroot/output/build/faad2-2.11.2/libfaad/fixed.h: In function ‘MUL_R’: buildroot/output/build/faad2-2.11.2/libfaad/fixed.h:163:1: warning: ISO C forbids braced-groups within expressions [-Wpedantic]
163 | ({
| ^
buildroot/output/build/faad2-2.11.2/libfaad/fixed.h:179:12: note: in expansion of macro ‘arm_mul’
179 | return arm_mul(A, B, REAL_BITS);

Building faad2 in Buildroot raises the following warning:

In file included from buildroot/output/build/faad2-2.11.2/libfaad/common.h:255,
                 from buildroot/output/build/faad2-2.11.2/libfaad/bits.c:31:
buildroot/output/build/faad2-2.11.2/libfaad/fixed.h: In function ‘MUL_R’:
buildroot/output/build/faad2-2.11.2/libfaad/fixed.h:163:1: warning: ISO C forbids braced-groups within expressions [-Wpedantic]
  163 | ({ \
      | ^
buildroot/output/build/faad2-2.11.2/libfaad/fixed.h:179:12: note: in expansion of macro ‘arm_mul’
  179 |     return arm_mul(A, B, REAL_BITS);

Signed-off-by: Dario Binacchi <dario.binacchi@amarulasolutions.com>
Copy link
Copy Markdown
Collaborator

@fabiangreffrath fabiangreffrath left a comment

Choose a reason for hiding this comment

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

Thank you!

@fabiangreffrath fabiangreffrath merged commit bc5de3a into knik0:master Oct 1, 2025
9 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants