Do not make many assumption on the dimension of the slices and just
try to decode additional lines if there is enough data left.
Decodes all the samples kindly provided by ultramage.
---
I tried to figure for a bit if there is something in the bitstream that could
give the actual number of lines but doesn't seem the case.
The solution is rather ugly, but works quite ok.
I'll rework later the structure of this codec a little further.
libavcodec/dxtory.c | 267 +++++++++++++++++++++++++++-------------------------
1 file changed, 140 insertions(+), 127 deletions(-)
diff --git a/libavcodec/dxtory.c b/libavcodec/dxtory.c
index c474499..542098c 100644
--- a/libavcodec/dxtory.c
+++ b/libavcodec/dxtory.c
@@ -233,7 +233,7 @@ static int load_buffer(AVCodecContext *avctx,
return AVERROR_INVALIDDATA;
}
- if (!*nslices || avctx->height % *nslices) {
+ if (!*nslices) {
avpriv_request_sample(avctx, "%d slices for %dx%d", *nslices,
avctx->width, avctx->height);
return AVERROR_PATCHWELCOME;
@@ -260,60 +260,84 @@ static inline uint8_t decode_sym_565(GetBitContext *gb,
uint8_t lru[8],
return val;
}
-typedef int (*decode_slice_func)(GetBitContext *gb, int width, int height,
- uint8_t *dst, int stride);
+typedef int (*decode_slice_func)(GetBitContext *gb, AVFrame *frame,
+ int line, int height, uint8_t lru[3][8]);
+
+typedef void (*setup_lru_func)(uint8_t lru[3][8]);
static int dxtory_decode_v2(AVCodecContext *avctx, AVFrame *pic,
const uint8_t *src, int src_size,
decode_slice_func decode_slice,
- enum AVPixelFormat fmt)
+ setup_lru_func setup_lru,
+ enum AVPixelFormat fmt, int block)
{
GetByteContext gb;
GetBitContext gb2;
- int nslices, slice, slice_height;
+ int nslices, slice, slice_height, ref_slice_height, line, mask;
uint32_t off, slice_size;
- uint8_t *dst;
+ uint8_t lru[3][8];
int ret;
+ int height = avctx->height;
ret = load_buffer(avctx, src, src_size, &gb, &nslices, &off);
if (ret < 0)
return ret;
- slice_height = avctx->height / nslices;
+ mask = block - 1;
+ line = 0;
+
+ slice_height = ref_slice_height = (avctx->height / nslices) & ~mask;
avctx->pix_fmt = fmt;
if ((ret = ff_get_buffer(avctx, pic, 0)) < 0)
return ret;
- dst = pic->data[0];
for (slice = 0; slice < nslices; slice++) {
slice_size = bytestream2_get_le32(&gb);
+ setup_lru(lru);
+
ret = check_slice_size(avctx, src, src_size, slice_size, off);
if (ret < 0)
return ret;
init_get_bits(&gb2, src + off + 16, (slice_size - 16) * 8);
- decode_slice(&gb2, avctx->width, slice_height, dst,
- pic->linesize[0]);
+ decode_slice(&gb2, pic, line, slice_height, lru);
+
+ line += slice_height;
+
+ while (get_bits_left(&gb2) > 16 &&
+ avctx->coded_height >= line + block) {
+ decode_slice(&gb2, pic, line, block, lru);
+
+ if (get_bits_left(&gb2) >= 0)
+ line += block;
+ }
- dst += pic->linesize[0] * slice_height;
off += slice_size;
}
+ if (avctx->height - line) {
+ av_log(avctx, AV_LOG_WARNING,
+ "Not enough slice data available, "
+ "cropping the frame by %d pixels\n",
+ avctx->height - line);
+ avctx->height = line;
+ }
+
return 0;
}
av_always_inline
-static int dx2_decode_slice_5x5(GetBitContext *gb, int width, int height,
- uint8_t *dst, int stride, int is_565)
+static int dx2_decode_slice_5x5(GetBitContext *gb, AVFrame *frame,
+ int line, int height, uint8_t lru[3][8],
+ int is_565)
{
int x, y;
int r, g, b;
- uint8_t lru[3][8];
+ int width = frame->width;
+ int stride = frame->linesize[0];
+ uint8_t *dst = frame->data[0] + stride * line;
- memcpy(lru[0], def_lru_555, 8 * sizeof(*def_lru));
- memcpy(lru[1], is_565 ? def_lru_565 : def_lru_555, 8 * sizeof(*def_lru));
- memcpy(lru[2], def_lru_555, 8 * sizeof(*def_lru));
for (y = 0; y < height; y++) {
for (x = 0; x < width; x++) {
@@ -331,16 +355,32 @@ static int dx2_decode_slice_5x5(GetBitContext *gb, int
width, int height,
return 0;
}
-static int dx2_decode_slice_555(GetBitContext *gb, int width, int height,
- uint8_t *dst, int stride)
+static void setup_lru_555(uint8_t lru[3][8])
+{
+ memcpy(lru[0], def_lru_555, 8 * sizeof(*def_lru));
+ memcpy(lru[1], def_lru_555, 8 * sizeof(*def_lru));
+ memcpy(lru[2], def_lru_555, 8 * sizeof(*def_lru));
+}
+
+static void setup_lru_565(uint8_t lru[3][8])
{
- return dx2_decode_slice_5x5(gb, width, height, dst, stride, 0);
+ memcpy(lru[0], def_lru_555, 8 * sizeof(*def_lru));
+ memcpy(lru[1], def_lru_565, 8 * sizeof(*def_lru));
+ memcpy(lru[2], def_lru_555, 8 * sizeof(*def_lru));
}
-static int dx2_decode_slice_565(GetBitContext *gb, int width, int height,
- uint8_t *dst, int stride)
+static int dx2_decode_slice_555(GetBitContext *gb, AVFrame *frame,
+ int line, int height,
+ uint8_t lru[3][8])
{
- return dx2_decode_slice_5x5(gb, width, height, dst, stride, 1);
+ return dx2_decode_slice_5x5(gb, frame, line, height, lru, 0);
+}
+
+static int dx2_decode_slice_565(GetBitContext *gb, AVFrame *frame,
+ int line, int height,
+ uint8_t lru[3][8])
+{
+ return dx2_decode_slice_5x5(gb, frame, line, height, lru, 1);
}
static int dxtory_decode_v2_565(AVCodecContext *avctx, AVFrame *pic,
@@ -349,20 +389,24 @@ static int dxtory_decode_v2_565(AVCodecContext *avctx,
AVFrame *pic,
enum AVPixelFormat fmt = AV_PIX_FMT_RGB24;
if (is_565)
return dxtory_decode_v2(avctx, pic, src, src_size,
- dx2_decode_slice_565, fmt);
+ dx2_decode_slice_565,
+ setup_lru_565,
+ fmt, 1 << 0);
else
return dxtory_decode_v2(avctx, pic, src, src_size,
- dx2_decode_slice_555, fmt);
+ dx2_decode_slice_555,
+ setup_lru_555,
+ fmt, 1 << 0);
}
-static int dx2_decode_slice_rgb(GetBitContext *gb, int width, int height,
- uint8_t *dst, int stride)
+static int dx2_decode_slice_rgb(GetBitContext *gb, AVFrame *frame,
+ int line, int height,
+ uint8_t lru[3][8])
{
- int x, y, i;
- uint8_t lru[3][8];
-
- for (i = 0; i < 3; i++)
- memcpy(lru[i], def_lru, 8 * sizeof(*def_lru));
+ int x, y;
+ int width = frame->width;
+ int stride = frame->linesize[0];
+ uint8_t *dst = frame->data[0] + stride * line;
for (y = 0; y < height; y++) {
for (x = 0; x < width; x++) {
@@ -377,29 +421,37 @@ static int dx2_decode_slice_rgb(GetBitContext *gb, int
width, int height,
return 0;
}
+static void default_setup_lru(uint8_t lru[3][8])
+{
+ int i;
+
+ for (i = 0; i < 3; i++)
+ memcpy(lru[i], def_lru, 8 * sizeof(*def_lru));
+}
+
static int dxtory_decode_v2_rgb(AVCodecContext *avctx, AVFrame *pic,
const uint8_t *src, int src_size)
{
return dxtory_decode_v2(avctx, pic, src, src_size,
dx2_decode_slice_rgb,
- AV_PIX_FMT_BGR24);
+ default_setup_lru,
+ AV_PIX_FMT_BGR24, 1 << 0);
}
-typedef int (*decode_slice_component_func)(GetBitContext *gb,
- int width, int height,
- uint8_t *Y, uint8_t *U, uint8_t *V,
- int ystride, int ustride,
- int vstride);
-
-static int dx2_decode_slice_410(GetBitContext *gb, int width, int height,
- uint8_t *Y, uint8_t *U, uint8_t *V,
- int ystride, int ustride, int vstride)
+static int dx2_decode_slice_410(GetBitContext *gb, AVFrame *frame,
+ int line, int height,
+ uint8_t lru[3][8])
{
int x, y, i, j;
- uint8_t lru[3][8];
+ int width = frame->width;
- for (i = 0; i < 3; i++)
- memcpy(lru[i], def_lru, 8 * sizeof(*def_lru));
+ int ystride = frame->linesize[0];
+ int ustride = frame->linesize[1];
+ int vstride = frame->linesize[2];
+
+ uint8_t *Y = frame->data[0] + ystride * line;
+ uint8_t *U = frame->data[1] + (ustride >> 2) * line;
+ uint8_t *V = frame->data[2] + (vstride >> 2) * line;
for (y = 0; y < height; y += 4) {
for (x = 0; x < width; x += 4) {
@@ -418,83 +470,33 @@ static int dx2_decode_slice_410(GetBitContext *gb, int
width, int height,
return 0;
}
-static int dxtory_decode_v2_component(AVCodecContext *avctx, AVFrame *pic,
- const uint8_t *src, int src_size,
- decode_slice_component_func decode_slice,
- enum AVPixelFormat fmt,
- int mask, int shift)
-{
- GetByteContext gb;
- GetBitContext gb2;
- int nslices, slice, slice_height, ref_slice_height;
- int cur_y, next_y;
- uint32_t off, slice_size;
- uint8_t *Y, *U, *V;
- int ret;
-
- ret = load_buffer(avctx, src, src_size, &gb, &nslices, &off);
- if (ret < 0)
- return ret;
-
- ref_slice_height = avctx->height / nslices;
- if ((avctx->width & mask) || (avctx->height & mask)) {
- avpriv_request_sample(avctx, "Frame dimensions %dx%d",
- avctx->width, avctx->height);
- }
-
- avctx->pix_fmt = fmt;
- if ((ret = ff_get_buffer(avctx, pic, 0)) < 0)
- return ret;
-
- Y = pic->data[0];
- U = pic->data[1];
- V = pic->data[2];
-
- cur_y = 0;
- next_y = ref_slice_height;
- for (slice = 0; slice < nslices; slice++) {
- slice_size = bytestream2_get_le32(&gb);
- slice_height = (next_y & ~mask) - (cur_y & ~mask);
-
- ret = check_slice_size(avctx, src, src_size, slice_size, off);
- if (ret < 0)
- return ret;
-
- init_get_bits(&gb2, src + off + 16, (slice_size - 16) * 8);
- decode_slice(&gb2, avctx->width, slice_height, Y, U, V,
- pic->linesize[0], pic->linesize[1],
- pic->linesize[2]);
-
- Y += pic->linesize[0] * slice_height;
- U += pic->linesize[1] * (slice_height >> shift);
- V += pic->linesize[2] * (slice_height >> shift);
- off += slice_size;
- cur_y = next_y;
- next_y += ref_slice_height;
- }
-
- return 0;
-}
static int dxtory_decode_v2_410(AVCodecContext *avctx, AVFrame *pic,
const uint8_t *src, int src_size)
{
- return dxtory_decode_v2_component(avctx, pic, src, src_size,
- dx2_decode_slice_410,
- AV_PIX_FMT_YUV410P, 3, 2);
+ return dxtory_decode_v2(avctx, pic, src, src_size,
+ dx2_decode_slice_410,
+ default_setup_lru,
+ AV_PIX_FMT_YUV410P, 1 << 2);
}
-static int dx2_decode_slice_420(GetBitContext *gb, int width, int height,
- uint8_t *Y, uint8_t *U, uint8_t *V,
- int ystride, int ustride, int vstride)
+static int dx2_decode_slice_420(GetBitContext *gb, AVFrame *frame,
+ int line, int height,
+ uint8_t lru[3][8])
{
- int x, y, i;
- uint8_t lru[3][8];
+ int x, y;
- for (i = 0; i < 3; i++)
- memcpy(lru[i], def_lru, 8 * sizeof(*def_lru));
+ int width = frame->width;
+
+ int ystride = frame->linesize[0];
+ int ustride = frame->linesize[1];
+ int vstride = frame->linesize[2];
+
+ uint8_t *Y = frame->data[0] + ystride * line;
+ uint8_t *U = frame->data[1] + (ustride >> 1) * line;
+ uint8_t *V = frame->data[2] + (vstride >> 1) * line;
- for (y = 0; y < height; y+=2) {
+ for (y = 0; y < height; y += 2) {
for (x = 0; x < width; x += 2) {
Y[x + 0 + 0 * ystride] = decode_sym(gb, lru[0]);
Y[x + 1 + 0 * ystride] = decode_sym(gb, lru[0]);
@@ -515,28 +517,38 @@ static int dx2_decode_slice_420(GetBitContext *gb, int
width, int height,
static int dxtory_decode_v2_420(AVCodecContext *avctx, AVFrame *pic,
const uint8_t *src, int src_size)
{
- return dxtory_decode_v2_component(avctx, pic, src, src_size,
- dx2_decode_slice_420,
- AV_PIX_FMT_YUV420P, 1, 1);
+ return dxtory_decode_v2(avctx, pic, src, src_size,
+ dx2_decode_slice_420,
+ default_setup_lru,
+ AV_PIX_FMT_YUV420P, 1 << 1);
}
-static int dx2_decode_slice_444(GetBitContext *gb, int width, int height,
- uint8_t *dst, int stride)
+static int dx2_decode_slice_444(GetBitContext *gb, AVFrame *frame,
+ int line, int height,
+ uint8_t lru[3][8])
{
- int x, y, i;
- uint8_t lru[3][8];
+ int x, y;
- for (i = 0; i < 3; i++)
- memcpy(lru[i], def_lru, 8 * sizeof(*def_lru));
+ int width = frame->width;
+
+ int ystride = frame->linesize[0];
+ int ustride = frame->linesize[1];
+ int vstride = frame->linesize[2];
+
+ uint8_t *Y = frame->data[0] + ystride * line;
+ uint8_t *U = frame->data[1] + ustride * line;
+ uint8_t *V = frame->data[2] + vstride * line;
for (y = 0; y < height; y++) {
for (x = 0; x < width; x++) {
- dst[x * 3 + 0] = decode_sym(gb, lru[0]);
- dst[x * 3 + 1] = decode_sym(gb, lru[1]) ^ 0x80;
- dst[x * 3 + 2] = decode_sym(gb, lru[2]) ^ 0x80;
+ Y[x] = decode_sym(gb, lru[0]);
+ U[x] = decode_sym(gb, lru[1]) ^ 0x80;
+ V[x] = decode_sym(gb, lru[2]) ^ 0x80;
}
- dst += stride;
+ Y += ystride;
+ U += ustride;
+ V += vstride;
}
return 0;
@@ -547,7 +559,8 @@ static int dxtory_decode_v2_444(AVCodecContext *avctx,
AVFrame *pic,
{
return dxtory_decode_v2(avctx, pic, src, src_size,
dx2_decode_slice_444,
- AV_PIX_FMT_YUV444P);
+ default_setup_lru,
+ AV_PIX_FMT_YUV444P, 1 << 0);
}
static int decode_frame(AVCodecContext *avctx, void *data, int *got_frame,
--
2.5.0
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
