]> git.sesse.net Git - ffmpeg/commit
avcodec/xbmenc: Do not add last comma into output
authorJose Da Silva <digital@joescat.com>
Tue, 19 Jan 2021 05:42:57 +0000 (21:42 -0800)
committerPaul B Mahol <onemda@gmail.com>
Thu, 28 Jan 2021 14:50:09 +0000 (15:50 +0100)
commit41b8fd3a16a0c8f2de71a5338fb4f086bd857124
tree4abc54af5162ad4a92c8dfba3ed90f4228a84b40
parent9267e2ff0d5b518bcce6236d09f2941b2c1bba84
avcodec/xbmenc: Do not add last comma into output

There is a minor bug in xbm encode which adds a trailing comma at the end
of data. This isn't a big problem, but it would be nicer to be more
technically true to an array of data (by not including the last comma).

This bug fixes the output from something like this (having 4 values):
static unsigned char image_bits[] = { 0x00, 0x11, 0x22, }
to C code that looks like this instead (having 3 values):
static unsigned char image_bits[] = { 0x00, 0x11, 0x22 }
which is the intended results.
Subject: [PATCH 1/3] avcodec/xbmenc: Do not add last comma into output array

xbm outputs c arrays of data.
Including a comma at the end means there is another value to be added.
This bug fix changes something like this:
static unsigned char image_bits[] = { 0x00, 0x11, 0x22, }
to C code like this:
static unsigned char image_bits[] = { 0x00, 0x11, 0x22 }

Signed-off-by: Joe Da Silva <digital@joescat.com>
libavcodec/xbmenc.c
tests/ref/lavf/xbm