On 10/17/2014 07:49 AM, Anton Khirnov wrote:
Quoting Vittorio Giovara (2014-10-15 18:32:56)
From: Luca Barbato <[email protected]>

Spare a branch and make coverity less confused.

CC: [email protected]
Bug-Id: CID 732222
---
  libavresample/utils.c | 9 ++++-----
  1 file changed, 4 insertions(+), 5 deletions(-)


Yet again -- I'm not a fan of randomly shuffling code around just to
make some specific tool shut up. There is a huge number of such tools
and each one complains about different perfectly valid code. We cannot
appease them all and we should not try.


I am also not in favor of this patch.

-Justin

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

Reply via email to