mirror of
https://github.com/FFmpeg/FFmpeg.git
synced 2024-11-26 19:01:44 +02:00
d6fc031caf
PSEUDOPAL pixel formats are not paletted, but carried a palette with the intention of allowing code to treat unpaletted formats as paletted. The palette simply mapped the byte values to the resulting RGB values, making it some sort of LUT for RGB conversion. It was used for 1 byte formats only: RGB4_BYTE, BGR4_BYTE, RGB8, BGR8, GRAY8. The first 4 are awfully obscure, used only by some ancient bitmap formats. The last one, GRAY8, is more common, but its treatment is grossly incorrect. It considers full range GRAY8 only, so GRAY8 coming from typical Y video planes was not mapped to the correct RGB values. This cannot be fixed, because AVFrame.color_range can be freely changed at runtime, and there is nothing to ensure the pseudo palette is updated. Also, nothing actually used the PSEUDOPAL palette data, except xwdenc (trivially changed in the previous commit). All other code had to treat it as a special case, just to ignore or to propagate palette data. In conclusion, this was just a very strange old mechnaism that has no real justification to exist anymore (although it may have been nice and useful in the past). Now it's an artifact that makes the API harder to use: API users who allocate their own pixel data have to be aware that they need to allocate the palette, or FFmpeg will crash on them in _some_ situations. On top of this, there was no API to allocate the pseuo palette outside of av_frame_get_buffer(). This patch not only deprecates AV_PIX_FMT_FLAG_PSEUDOPAL, but also makes the pseudo palette optional. Nothing accesses it anymore, though if it's set, it's propagated. It's still allocated and initialized for compatibility with API users that rely on this feature. But new API users do not need to allocate it. This was an explicit goal of this patch. Most changes replace AV_PIX_FMT_FLAG_PSEUDOPAL with FF_PSEUDOPAL. I first tried #ifdefing all code, but it was a mess. The FF_PSEUDOPAL macro reduces the mess, and still allows defining FF_API_PSEUDOPAL to 0. Passes FATE with FF_API_PSEUDOPAL enabled and disabled. In addition, FATE passes with FF_API_PSEUDOPAL set to 1, but with allocation functions manually changed to not allocating a palette.
136 lines
4.0 KiB
C
136 lines
4.0 KiB
C
/*
|
|
* Copyright (c) 2009 Stefano Sabatini
|
|
*
|
|
* This file is part of FFmpeg.
|
|
*
|
|
* FFmpeg is free software; you can redistribute it and/or
|
|
* modify it under the terms of the GNU Lesser General Public
|
|
* License as published by the Free Software Foundation; either
|
|
* version 2.1 of the License, or (at your option) any later version.
|
|
*
|
|
* FFmpeg 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
|
|
* Lesser General Public License for more details.
|
|
*
|
|
* You should have received a copy of the GNU Lesser General Public
|
|
* License along with FFmpeg; if not, write to the Free Software
|
|
* Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
|
|
*/
|
|
|
|
/**
|
|
* @file
|
|
* pixdesc test filter
|
|
*/
|
|
|
|
#include "libavutil/common.h"
|
|
#include "libavutil/pixdesc.h"
|
|
#include "avfilter.h"
|
|
#include "internal.h"
|
|
#include "video.h"
|
|
|
|
typedef struct PixdescTestContext {
|
|
const AVPixFmtDescriptor *pix_desc;
|
|
uint16_t *line;
|
|
} PixdescTestContext;
|
|
|
|
static av_cold void uninit(AVFilterContext *ctx)
|
|
{
|
|
PixdescTestContext *priv = ctx->priv;
|
|
av_freep(&priv->line);
|
|
}
|
|
|
|
static int config_props(AVFilterLink *inlink)
|
|
{
|
|
PixdescTestContext *priv = inlink->dst->priv;
|
|
|
|
priv->pix_desc = av_pix_fmt_desc_get(inlink->format);
|
|
|
|
av_freep(&priv->line);
|
|
if (!(priv->line = av_malloc_array(sizeof(*priv->line), inlink->w)))
|
|
return AVERROR(ENOMEM);
|
|
|
|
return 0;
|
|
}
|
|
|
|
static int filter_frame(AVFilterLink *inlink, AVFrame *in)
|
|
{
|
|
PixdescTestContext *priv = inlink->dst->priv;
|
|
AVFilterLink *outlink = inlink->dst->outputs[0];
|
|
AVFrame *out;
|
|
int i, c, w = inlink->w, h = inlink->h;
|
|
const int cw = AV_CEIL_RSHIFT(w, priv->pix_desc->log2_chroma_w);
|
|
const int ch = AV_CEIL_RSHIFT(h, priv->pix_desc->log2_chroma_h);
|
|
|
|
out = ff_get_video_buffer(outlink, outlink->w, outlink->h);
|
|
if (!out) {
|
|
av_frame_free(&in);
|
|
return AVERROR(ENOMEM);
|
|
}
|
|
|
|
av_frame_copy_props(out, in);
|
|
|
|
for (i = 0; i < 4; i++) {
|
|
const int h1 = i == 1 || i == 2 ? ch : h;
|
|
if (out->data[i]) {
|
|
uint8_t *data = out->data[i] +
|
|
(out->linesize[i] > 0 ? 0 : out->linesize[i] * (h1-1));
|
|
memset(data, 0, FFABS(out->linesize[i]) * h1);
|
|
}
|
|
}
|
|
|
|
/* copy palette */
|
|
if (priv->pix_desc->flags & AV_PIX_FMT_FLAG_PAL ||
|
|
((priv->pix_desc->flags & FF_PSEUDOPAL) && out->data[1] && in->data[1]))
|
|
memcpy(out->data[1], in->data[1], AVPALETTE_SIZE);
|
|
|
|
for (c = 0; c < priv->pix_desc->nb_components; c++) {
|
|
const int w1 = c == 1 || c == 2 ? cw : w;
|
|
const int h1 = c == 1 || c == 2 ? ch : h;
|
|
|
|
for (i = 0; i < h1; i++) {
|
|
av_read_image_line(priv->line,
|
|
(void*)in->data,
|
|
in->linesize,
|
|
priv->pix_desc,
|
|
0, i, c, w1, 0);
|
|
|
|
av_write_image_line(priv->line,
|
|
out->data,
|
|
out->linesize,
|
|
priv->pix_desc,
|
|
0, i, c, w1);
|
|
}
|
|
}
|
|
|
|
av_frame_free(&in);
|
|
return ff_filter_frame(outlink, out);
|
|
}
|
|
|
|
static const AVFilterPad avfilter_vf_pixdesctest_inputs[] = {
|
|
{
|
|
.name = "default",
|
|
.type = AVMEDIA_TYPE_VIDEO,
|
|
.filter_frame = filter_frame,
|
|
.config_props = config_props,
|
|
},
|
|
{ NULL }
|
|
};
|
|
|
|
static const AVFilterPad avfilter_vf_pixdesctest_outputs[] = {
|
|
{
|
|
.name = "default",
|
|
.type = AVMEDIA_TYPE_VIDEO,
|
|
},
|
|
{ NULL }
|
|
};
|
|
|
|
AVFilter ff_vf_pixdesctest = {
|
|
.name = "pixdesctest",
|
|
.description = NULL_IF_CONFIG_SMALL("Test pixel format definitions."),
|
|
.priv_size = sizeof(PixdescTestContext),
|
|
.uninit = uninit,
|
|
.inputs = avfilter_vf_pixdesctest_inputs,
|
|
.outputs = avfilter_vf_pixdesctest_outputs,
|
|
};
|