Stefano Sabatini | 17 Jun 2012 12:22

[PATCH] lavc/imgconvert: remove pointless switch block from avpicture_fill()

---
 libavcodec/imgconvert.c |   10 ----------
 1 files changed, 0 insertions(+), 10 deletions(-)

diff --git a/libavcodec/imgconvert.c b/libavcodec/imgconvert.c
index 3111418..495a1d0 100644
--- a/libavcodec/imgconvert.c
+++ b/libavcodec/imgconvert.c
 <at>  <at>  -334,16 +334,6  <at>  <at>  int avpicture_layout(const AVPicture* src, enum PixelFormat pix_fmt, int width,
         }
     }

-    switch (pix_fmt) {
-    case PIX_FMT_RGB8:
-    case PIX_FMT_BGR8:
-    case PIX_FMT_RGB4_BYTE:
-    case PIX_FMT_BGR4_BYTE:
-    case PIX_FMT_GRAY8:
-        // do not include palette for these pseudo-paletted formats
-        return size;
-    }
-
     if (desc->flags & PIX_FMT_PAL) {
         uint32_t *d32 = (unsigned char *)(((size_t)dest + 3) & ~3);
         for (i = 0; i<256; i++)
--

-- 
1.7.5.4
Michael Niedermayer | 17 Jun 2012 14:38

Re: [PATCH] lavc/imgconvert: remove pointless switch block from avpicture_fill()

On Sun, Jun 17, 2012 at 12:22:13PM +0200, Stefano Sabatini wrote:
> ---
>  libavcodec/imgconvert.c |   10 ----------
>  1 files changed, 0 insertions(+), 10 deletions(-)

ok if it doesnt change the functions output

[...]

--

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

it is not once nor twice but times without number that the same ideas make
their appearance in the world. -- Aristotle
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel <at> ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Stefano Sabatini | 19 Jun 2012 01:48

Re: [PATCH] lavc/imgconvert: remove pointless switch block from avpicture_fill()

On date Sunday 2012-06-17 14:38:36 +0200, Michael Niedermayer encoded:
> On Sun, Jun 17, 2012 at 12:22:13PM +0200, Stefano Sabatini wrote:
> > ---
> >  libavcodec/imgconvert.c |   10 ----------
> >  1 files changed, 0 insertions(+), 10 deletions(-)
> 
> ok if it doesnt change the functions output

Applied.
--

-- 
FFmpeg = Free and Fast Mastodontic Perfectionist Elitarian Game

Gmane