On Mon, 6 Jan 2014, Diego Biurrun wrote:

The define does not originate from configure, so it should not
have a name that is CONFIG_-prefixed.
---

I wouldn't really call this cosmetics

libavcodec/ac3enc_fixed.c           |  2 +-
libavcodec/arm/fft_fixed_init_arm.c |  2 +-
libavcodec/cos_tablegen.c           |  2 +-
libavcodec/fft-fixed-test.c         |  2 +-
libavcodec/fft-internal.h           |  4 ++--
libavcodec/fft-test.c               | 16 ++++++++--------
libavcodec/fft.h                    |  8 ++++----
libavcodec/fft_fixed.c              |  2 +-
libavcodec/fft_float.c              |  2 +-
libavcodec/fft_template.c           |  2 +-
libavcodec/mdct_fixed.c             |  2 +-
libavcodec/mdct_float.c             |  2 +-
libavcodec/mdct_template.c          |  2 +-
13 files changed, 24 insertions(+), 24 deletions(-)

The patch itself looks ok

// Martin
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to