On Sat, Jun 29, 2013 at 07:27:28AM +0200, Luca Barbato wrote:
> --- a/libavcodec/iff.c
> +++ b/libavcodec/iff.c
> @@ -223,12 +221,13 @@ static void decodeplane32(uint32_t *dst, const uint8_t
> *buf, int buf_size, int p
> static int decode_byterun(uint8_t *dst, int dst_size,
> - const uint8_t *buf, const uint8_t *const buf_end) {
> + const uint8_t *buf, const uint8_t *const buf_end)
> +{
> const uint8_t *const buf_start = buf;
> unsigned x;
> - for (x = 0; x < dst_size && buf < buf_end;) {
> + for (x = 0; x < dst_size && buf < buf_end; ) {
stray uncrustify weirdness
> @@ -261,33 +260,36 @@ static int decode_frame_ilbm(AVCodecContext *avctx,
> memset(row, 0, avctx->width << 2);
> - for (plane = 0; plane < avctx->bits_per_coded_sample && buf
> < buf_end; plane++) {
> - decodeplane32((uint32_t *) row, buf, FFMIN(s->planesize,
> buf_end - buf), plane);
> + for (plane = 0; plane < avctx->bits_per_coded_sample && buf
> < buf_end;
> + plane++) {
> + decodeplane32((uint32_t *)row, buf, FFMIN(s->planesize,
> + buf_end -
> buf), plane);
"plane" looks misplaced now. Break the line before FFMIN instead.
> @@ -317,15 +319,15 @@ static int decode_frame_byterun1(AVCodecContext *avctx,
>
> - if (avctx->codec_tag == MKTAG('I','L','B','M')) { //interleaved
> + if (avctx->codec_tag == MKTAG('I', 'L', 'B', 'M')) { //interleaved
space after //, same in other places
Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
