[FFmpeg-devel] [PATCH] lavfi: remove palette libmpcodecs wrapper

Paul B Mahol onemda at gmail.com
Fri Oct 26 20:02:26 CEST 2012


The filter is useless, swscale should be used instead.

Signed-off-by: Paul B Mahol <onemda at gmail.com>
---
 doc/filters.texi                     |   1 -
 libavfilter/Makefile                 |   1 -
 libavfilter/libmpcodecs/vf_palette.c | 236 -----------------------------------
 libavfilter/version.h                |   2 +-
 libavfilter/vf_mp.c                  |   2 -
 5 files changed, 1 insertion(+), 241 deletions(-)
 delete mode 100644 libavfilter/libmpcodecs/vf_palette.c

diff --git a/doc/filters.texi b/doc/filters.texi
index c4e29c3..0d31157 100644
--- a/doc/filters.texi
+++ b/doc/filters.texi
@@ -2598,7 +2598,6 @@ The list of the currently supported filters follows:
 @item mcdeint
 @item noise
 @item ow
- at item palette
 @item perspective
 @item phase
 @item pp7
diff --git a/libavfilter/Makefile b/libavfilter/Makefile
index 3618f10..15f6ce8 100644
--- a/libavfilter/Makefile
+++ b/libavfilter/Makefile
@@ -181,7 +181,6 @@ OBJS-$(CONFIG_MP_FILTER) += libmpcodecs/vf_kerndeint.o
 OBJS-$(CONFIG_MP_FILTER) += libmpcodecs/vf_mcdeint.o
 OBJS-$(CONFIG_MP_FILTER) += libmpcodecs/vf_noise.o
 OBJS-$(CONFIG_MP_FILTER) += libmpcodecs/vf_ow.o
-OBJS-$(CONFIG_MP_FILTER) += libmpcodecs/vf_palette.o
 OBJS-$(CONFIG_MP_FILTER) += libmpcodecs/vf_perspective.o
 OBJS-$(CONFIG_MP_FILTER) += libmpcodecs/vf_phase.o
 OBJS-$(CONFIG_MP_FILTER) += libmpcodecs/vf_pp.o
diff --git a/libavfilter/libmpcodecs/vf_palette.c b/libavfilter/libmpcodecs/vf_palette.c
deleted file mode 100644
index 543b6c7..0000000
--- a/libavfilter/libmpcodecs/vf_palette.c
+++ /dev/null
@@ -1,236 +0,0 @@
-/*
- * This file is part of MPlayer.
- *
- * MPlayer is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * MPlayer is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License along
- * with MPlayer; if not, write to the Free Software Foundation, Inc.,
- * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
- */
-
-#include <stdio.h>
-#include <stdlib.h>
-#include <inttypes.h>
-
-#include "config.h"
-#include "mp_msg.h"
-#include "help_mp.h"
-
-#include "img_format.h"
-#include "mp_image.h"
-#include "vf.h"
-#include "mpbswap.h"
-
-#include "libswscale/swscale.h"
-
-#include "libavutil/avstring.h"
-
-//===========================================================================//
-
-// commented out 16 and 15 bit output support, because the conversion
-// routines are incorrrect.  they assume the palette to be of the same
-// depth as the output, which is incorrect. --Joey
-
-static const unsigned int bgr_list[]={
-    IMGFMT_BGR32,
-    IMGFMT_BGR24,
-//    IMGFMT_BGR16,
-//    IMGFMT_BGR15,
-    0
-};
-static const unsigned int rgb_list[]={
-    IMGFMT_RGB32,
-    IMGFMT_RGB24,
-//    IMGFMT_RGB16,
-//    IMGFMT_RGB15,
-    0
-};
-
-/**
- * Palette is assumed to contain BGR16, see rgb32to16 to convert the palette.
- */
-static void palette8torgb16(const uint8_t *src, uint8_t *dst, long num_pixels, const uint8_t *palette)
-{
-    long i;
-    for (i=0; i<num_pixels; i++)
-        ((uint16_t *)dst)[i] = ((const uint16_t *)palette)[src[i]];
-}
-
-static void palette8tobgr16(const uint8_t *src, uint8_t *dst, long num_pixels, const uint8_t *palette)
-{
-    long i;
-    for (i=0; i<num_pixels; i++)
-        ((uint16_t *)dst)[i] = bswap_16(((const uint16_t *)palette)[src[i]]);
-}
-
-static unsigned int gray_pal[256];
-
-static unsigned int find_best(struct vf_instance *vf, unsigned int fmt){
-    unsigned int best=0;
-    int ret;
-    const unsigned int* p;
-    if(fmt==IMGFMT_BGR8) p=bgr_list;
-    else if(fmt==IMGFMT_RGB8) p=rgb_list;
-    else return 0;
-    while(*p){
-        ret=vf->next->query_format(vf->next,*p);
-        mp_msg(MSGT_VFILTER,MSGL_DBG2,"[%s] query(%s) -> %d\n",vf->info->name,vo_format_name(*p),ret&3);
-        if(ret&VFCAP_CSP_SUPPORTED_BY_HW){ best=*p; break;} // no conversion -> bingo!
-        if(ret&VFCAP_CSP_SUPPORTED && !best) best=*p; // best with conversion
-        ++p;
-    }
-    return best;
-}
-
-//===========================================================================//
-
-struct vf_priv_s {
-    unsigned int fmt;
-    int pal_msg;
-};
-
-static int config(struct vf_instance *vf,
-        int width, int height, int d_width, int d_height,
-        unsigned int flags, unsigned int outfmt){
-    if (!vf->priv->fmt)
-        vf->priv->fmt=find_best(vf,outfmt);
-    if(!vf->priv->fmt){
-        // no matching fmt, so force one...
-        if(outfmt==IMGFMT_RGB8) vf->priv->fmt=IMGFMT_RGB32;
-        else if(outfmt==IMGFMT_BGR8) vf->priv->fmt=IMGFMT_BGR32;
-        else return 0;
-    }
-    return vf_next_config(vf,width,height,d_width,d_height,flags,vf->priv->fmt);
-}
-
-static int put_image(struct vf_instance *vf, mp_image_t *mpi, double pts){
-    mp_image_t *dmpi;
-    uint8_t *old_palette = mpi->planes[1];
-
-    // hope we'll get DR buffer:
-    dmpi=vf_get_image(vf->next,vf->priv->fmt,
-        MP_IMGTYPE_TEMP, MP_IMGFLAG_ACCEPT_STRIDE,
-        mpi->w, mpi->h);
-
-    if (!mpi->planes[1])
-    {
-        if(!vf->priv->pal_msg){
-            mp_msg(MSGT_VFILTER,MSGL_V,"[%s] no palette given, assuming builtin grayscale one\n",vf->info->name);
-            vf->priv->pal_msg=1;
-        }
-        mpi->planes[1] = (unsigned char*)gray_pal;
-    }
-
-    if(mpi->w==mpi->stride[0] && dmpi->w*(dmpi->bpp>>3)==dmpi->stride[0]){
-        // no stride conversion needed
-        switch(IMGFMT_RGB_DEPTH(dmpi->imgfmt)){
-        case 15:
-        case 16:
-            if (IMGFMT_IS_BGR(dmpi->imgfmt))
-                palette8tobgr16(mpi->planes[0],dmpi->planes[0],mpi->h*mpi->w,mpi->planes[1]);
-            else
-                palette8torgb16(mpi->planes[0],dmpi->planes[0],mpi->h*mpi->w,mpi->planes[1]);
-            break;
-        case 24:
-            if (IMGFMT_IS_BGR(dmpi->imgfmt))
-                sws_convertPalette8ToPacked24(mpi->planes[0],dmpi->planes[0],mpi->h*mpi->w,mpi->planes[1]);
-            else
-                sws_convertPalette8ToPacked24(mpi->planes[0],dmpi->planes[0],mpi->h*mpi->w,mpi->planes[1]);
-            break;
-        case 32:
-            if (IMGFMT_IS_BGR(dmpi->imgfmt))
-                sws_convertPalette8ToPacked32(mpi->planes[0],dmpi->planes[0],mpi->h*mpi->w,mpi->planes[1]);
-            else
-                sws_convertPalette8ToPacked32(mpi->planes[0],dmpi->planes[0],mpi->h*mpi->w,mpi->planes[1]);
-            break;
-        }
-    } else {
-        int y;
-        for(y=0;y<mpi->h;y++){
-            unsigned char* src=mpi->planes[0]+y*mpi->stride[0];
-            unsigned char* dst=dmpi->planes[0]+y*dmpi->stride[0];
-            switch(IMGFMT_RGB_DEPTH(dmpi->imgfmt)){
-            case 15:
-            case 16:
-                if (IMGFMT_IS_BGR(dmpi->imgfmt))
-                    palette8tobgr16(src,dst,mpi->w,mpi->planes[1]);
-                else
-                    palette8torgb16(src,dst,mpi->w,mpi->planes[1]);
-                break;
-            case 24:
-                if (IMGFMT_IS_BGR(dmpi->imgfmt))
-                    sws_convertPalette8ToPacked24(src,dst,mpi->w,mpi->planes[1]);
-                else
-                    sws_convertPalette8ToPacked24(src,dst,mpi->w,mpi->planes[1]);
-                break;
-            case 32:
-                if (IMGFMT_IS_BGR(dmpi->imgfmt))
-                    sws_convertPalette8ToPacked32(src,dst,mpi->w,mpi->planes[1]);
-                else
-                    sws_convertPalette8ToPacked32(src,dst,mpi->w,mpi->planes[1]);
-                break;
-            }
-        }
-    }
-    mpi->planes[1] = old_palette;
-
-    return vf_next_put_image(vf,dmpi, pts);
-}
-
-//===========================================================================//
-
-static int query_format(struct vf_instance *vf, unsigned int fmt){
-    int best=find_best(vf,fmt);
-    if(!best) return 0; // no match
-    return vf->next->query_format(vf->next,best);
-}
-
-static void uninit(vf_instance_t *vf) {
-  free(vf->priv);
-}
-
-static int vf_open(vf_instance_t *vf, char *args){
-    unsigned int i;
-    vf->config=config;
-    vf->uninit=uninit;
-    vf->put_image=put_image;
-    vf->query_format=query_format;
-    vf->priv=malloc(sizeof(struct vf_priv_s));
-    memset(vf->priv, 0, sizeof(struct vf_priv_s));
-    for(i=0;i<256;i++) gray_pal[i]=0x01010101*i;
-    if (args)
-    {
-        if (!av_strcasecmp(args,"rgb15")) vf->priv->fmt=IMGFMT_RGB15; else
-        if (!av_strcasecmp(args,"rgb16")) vf->priv->fmt=IMGFMT_RGB16; else
-        if (!av_strcasecmp(args,"rgb24")) vf->priv->fmt=IMGFMT_RGB24; else
-        if (!av_strcasecmp(args,"rgb32")) vf->priv->fmt=IMGFMT_RGB32; else
-        if (!av_strcasecmp(args,"bgr15")) vf->priv->fmt=IMGFMT_BGR15; else
-        if (!av_strcasecmp(args,"bgr16")) vf->priv->fmt=IMGFMT_BGR16; else
-        if (!av_strcasecmp(args,"bgr24")) vf->priv->fmt=IMGFMT_BGR24; else
-        if (!av_strcasecmp(args,"bgr32")) vf->priv->fmt=IMGFMT_BGR32; else
-        {
-            mp_msg(MSGT_VFILTER, MSGL_WARN, MSGTR_MPCODECS_UnknownFormatName, args);
-            return 0;
-        }
-    }
-    return 1;
-}
-
-const vf_info_t vf_info_palette = {
-    "8bpp indexed (using palette) -> BGR 15/16/24/32 conversion",
-    "palette",
-    "A'rpi & Alex",
-    "",
-    vf_open,
-    NULL
-};
-
-//===========================================================================//
diff --git a/libavfilter/version.h b/libavfilter/version.h
index 7b55903..3ce13c1 100644
--- a/libavfilter/version.h
+++ b/libavfilter/version.h
@@ -30,7 +30,7 @@
 
 #define LIBAVFILTER_VERSION_MAJOR  3
 #define LIBAVFILTER_VERSION_MINOR  20
-#define LIBAVFILTER_VERSION_MICRO 106
+#define LIBAVFILTER_VERSION_MICRO 107
 
 #define LIBAVFILTER_VERSION_INT AV_VERSION_INT(LIBAVFILTER_VERSION_MAJOR, \
                                                LIBAVFILTER_VERSION_MINOR, \
diff --git a/libavfilter/vf_mp.c b/libavfilter/vf_mp.c
index 9e3270e..7700b1f 100644
--- a/libavfilter/vf_mp.c
+++ b/libavfilter/vf_mp.c
@@ -158,7 +158,6 @@ extern const vf_info_t vf_info_mcdeint;
 extern const vf_info_t vf_info_noformat;
 extern const vf_info_t vf_info_noise;
 extern const vf_info_t vf_info_ow;
-extern const vf_info_t vf_info_palette;
 extern const vf_info_t vf_info_perspective;
 extern const vf_info_t vf_info_phase;
 extern const vf_info_t vf_info_pp7;
@@ -210,7 +209,6 @@ static const vf_info_t* const filters[]={
     &vf_info_mcdeint,
     &vf_info_noise,
     &vf_info_ow,
-    &vf_info_palette,
     &vf_info_perspective,
     &vf_info_phase,
     &vf_info_pp,
-- 
1.7.11.2



More information about the ffmpeg-devel mailing list