On 26/09/14 09:42, Diego Biurrun wrote:
This avoids an unused variable warning on big-endian systems.
---
  libavcodec/pcm-dvd.c | 5 +++--
  1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/libavcodec/pcm-dvd.c b/libavcodec/pcm-dvd.c
index 0872d29..791df54 100644
--- a/libavcodec/pcm-dvd.c
+++ b/libavcodec/pcm-dvd.c
@@ -154,21 +154,22 @@ static void *pcm_dvd_decode_samples(AVCodecContext 
*avctx, const uint8_t *src,
      GetByteContext gb;
      int i;
      uint8_t t;
-    int samples;

      bytestream2_init(&gb, src, blocks * s->block_size);
      switch (avctx->bits_per_coded_sample) {
      case 16:
+    {

Usually we put it in front of ": "

  #if HAVE_BIGENDIAN
          bytestream2_get_buffer(&gb, dst16, blocks * s->block_size);
          dst16 += blocks * s->block_size / 2;
  #else
-        samples = blocks * avctx->channels;
+        int samples = blocks * avctx->channels;
          do {
              *dst16++ = bytestream2_get_be16u(&gb);
          } while (--samples);
  #endif
          return dst16;
+    }
      case 20:
          do {
              for (i = s->groups_per_block; i; i--) {


Beside that looks fine to me.

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

Reply via email to