[FFmpeg-devel] rmdec.c: add SIPR codec try #2
Ronald S. Bultje
rsbultje
Wed Mar 18 21:25:20 CET 2009
Hi,
I'm gonna keep trying, just so I learn to not be an idiot. If you want
me to stop, just say so and I'll commit that first version.
On Tue, Mar 17, 2009 at 11:57 PM, Kostya <kostya.shishkov at gmail.com> wrote:
> On Tue, Mar 17, 2009 at 05:34:01PM -0400, Ronald S. Bultje wrote:
>> + ? ? ? ?/* swap 4bit-nibbles of block 'i' with 'o' */
>> + ? ? ? ?if (!((i ^ o) & 1)) LOOP_ITER {
>> + ? ? ? ? ? ?buf[i >> 1] = (buf[i >> 1] & mask2) | (buf[o >> 1] & mask1);
>> + ? ? ? ? ? ?buf[o >> 1] = (buf[o >> 1] & mask2) | tmp;
>> + ? ? ? ?} LOOP_END else LOOP_ITER {
>> + ? ? ? ? ? ?int shift1 = (i & 1) ? 4 : 0, shift2 = (i & 1) ? 0 : 4;
>> + ? ? ? ? ? ?buf[i >> 1] = (buf[i >> 1] & mask2) |
>> + ? ? ? ? ? ? ? ?(((buf[o >> 1] & mask2) << shift1) >> shift2);
>> + ? ? ? ? ? ?buf[o >> 1] = (buf[o >> 1] & mask1) | ((tmp << shift2) >> shift1);
>> + ? ? ? ?} LOOP_END;
>
> No, that still looks like some suspicious condition.
> You'd better write it without macroses.
OK, I rewrote it without the (slow) loops. I tried to unroll the loops
in such a way that as little work as possible is done within the
loops. Please let me know if I'm showing improvements. :-).
Ronald
-------------- next part --------------
Index: ffmpeg-svn/libavformat/rmdec.c
===================================================================
--- ffmpeg-svn.orig/libavformat/rmdec.c 2009-03-17 16:42:54.000000000 -0400
+++ ffmpeg-svn/libavformat/rmdec.c 2009-03-18 16:20:59.000000000 -0400
@@ -48,6 +48,21 @@
int audio_pkt_cnt; ///< Output packet counter
} RMDemuxContext;
+static const unsigned char sipr_swaps[38][2] = {
+ { 0, 63 }, { 1, 22 }, { 2, 44 }, { 3, 90 },
+ { 5, 81 }, { 7, 31 }, { 8, 86 }, { 9, 58 },
+ { 10, 36 }, { 12, 68 }, { 13, 39 }, { 14, 73 },
+ { 15, 53 }, { 16, 69 }, { 17, 57 }, { 19, 88 },
+ { 20, 34 }, { 21, 71 }, { 24, 46 }, { 25, 94 },
+ { 26, 54 }, { 28, 75 }, { 29, 50 }, { 32, 70 },
+ { 33, 92 }, { 35, 74 }, { 38, 85 }, { 40, 56 },
+ { 42, 87 }, { 43, 65 }, { 45, 59 }, { 48, 79 },
+ { 49, 93 }, { 51, 89 }, { 55, 95 }, { 61, 76 },
+ { 67, 83 }, { 77, 80 }
+};
+
+static const unsigned char sipr_subpk_size[4] = { 29, 19, 37, 20 };
+
static inline void get_strl(ByteIOContext *pb, char *buf, int buf_size, int len)
{
int i;
@@ -171,19 +186,28 @@
return -1;
}
- if(sub_packet_size <= 0){
- av_log(s, AV_LOG_ERROR, "sub_packet_size is invalid\n");
- return -1;
- }
-
if (!strcmp(buf, "cook")) st->codec->codec_id = CODEC_ID_COOK;
else if (!strcmp(buf, "sipr")) st->codec->codec_id = CODEC_ID_SIPR;
else st->codec->codec_id = CODEC_ID_ATRAC3;
+
+ ast->audio_framesize = st->codec->block_align;
+ if (st->codec->codec_id == CODEC_ID_SIPR) {
+ if (flavor > 3) {
+ av_log(s, AV_LOG_ERROR, "bad SIPR file flavor %d\n",
+ flavor);
+ return -1;
+ }
+ st->codec->block_align = sipr_subpk_size[flavor];
+ } else {
+ if(sub_packet_size <= 0){
+ av_log(s, AV_LOG_ERROR, "sub_packet_size is invalid\n");
+ return -1;
+ }
+ st->codec->block_align = ast->sub_packet_size;
+ }
st->codec->extradata_size= codecdata_length;
st->codec->extradata= av_mallocz(st->codec->extradata_size + FF_INPUT_BUFFER_PADDING_SIZE);
get_buffer(pb, st->codec->extradata, st->codec->extradata_size);
- ast->audio_framesize = st->codec->block_align;
- st->codec->block_align = ast->sub_packet_size;
if(ast->audio_framesize >= UINT_MAX / sub_packet_h){
av_log(s, AV_LOG_ERROR, "rm->audio_framesize * sub_packet_h too large\n");
@@ -637,6 +661,65 @@
}
}
+/** perform 4-bit block reordering for SIPR data */
+static void
+rm_reorder_sipr_data (RMStream *ast)
+{
+ int n, bs = ast->sub_packet_h * ast->audio_framesize * 2 / 96; // nibbles per subpacket
+
+ for (n = 0; n < 38; n++) {
+ int j;
+ int i = bs * sipr_swaps[n][0];
+ int o = bs * sipr_swaps[n][1];
+ uint8_t *buf = ast->pkt.data;
+
+ /* swap 4bit-nibbles of block 'i' with 'o' */
+ if (!((i ^ o) & 1)) {
+ int bytes = (bs - (i & 1)) >> 1;
+ char t[bytes];
+ if ((i ^ bs) & 1) { /* last 4bits, if unaligned */
+ int tmp = buf[(i + bs - 1) >> 1] & 0x0F;
+ buf[(i + bs - 1) >> 1] = (buf[i >> 1] & 0xF0) |
+ (buf[(o + bs - 1) >> 1] & 0x0F);
+ buf[o >> 1] = (buf[(o + bs - 1) >> 1] & 0xF0) | tmp;
+ }
+ if (i & 1) { /* first 4 bits, if unaligned */
+ int tmp = buf[i >> 1] & 0xF0;
+ buf[i >> 1] = (buf[i >> 1] & 0x0F) | (buf[o >> 1] & 0xF0);
+ buf[o >> 1] = (buf[o >> 1] & 0x0F) | tmp;
+ i++; o++;
+ }
+ memcpy(&t, &buf[i >> 1], bytes);
+ memcpy(&buf[i >> 1], &buf[o >> 1], bytes);
+ memcpy(&buf[o >> 1], &t, bytes);
+ } else {
+ int mask1, mask2, shift1, shift2, tmp;
+
+ mask1 = (i & 1) ? 0xF0 : 0x0F;
+ mask2 = (i & 1) ? 0x0F : 0xF0;
+ shift1 = (i & 1) ? 4 : 0;
+ shift2 = (i & 1) ? 0 : 4;
+ for (j = 0; j < bs; j+=2, i+=2, o+=2) {
+ tmp = buf[i >> 1] & mask1;
+ buf[i >> 1] = (buf[i >> 1] & mask2) |
+ (((buf[o >> 1] & mask2) << shift1) >> shift2);
+ buf[o >> 1] = (buf[o >> 1] & mask1) |
+ ((tmp << shift2) >> shift1);
+ }
+ i -= j - 1; o-= j - 1;
+ tmp = mask1; mask1 = mask2; mask2 = tmp;
+ tmp = shift1; shift1 = shift2; shift2 = tmp;
+ for (j = 1; j < bs; j+=2, i+=2, o+=2) {
+ tmp = buf[i >> 1] & mask1;
+ buf[i >> 1] = (buf[i >> 1] & mask2) |
+ (((buf[o >> 1] & mask2) << shift1) >> shift2);
+ buf[o >> 1] = (buf[o >> 1] & mask1) |
+ ((tmp << shift2) >> shift1);
+ }
+ }
+ }
+}
+
int
ff_rm_parse_packet (AVFormatContext *s, ByteIOContext *pb,
AVStream *st, RMStream *ast, int len, AVPacket *pkt,
@@ -675,10 +758,16 @@
for (x = 0; x < w/sps; x++)
get_buffer(pb, ast->pkt.data+sps*(h*x+((h+1)/2)*(y&1)+(y>>1)), sps);
break;
+ case CODEC_ID_SIPR:
+ get_buffer(pb, ast->pkt.data + y * w, w);
+ break;
}
if (++(ast->sub_packet_cnt) < h)
return -1;
+ if (st->codec->codec_id == CODEC_ID_SIPR)
+ rm_reorder_sipr_data(ast);
+
ast->sub_packet_cnt = 0;
rm->audio_stream_num = st->index;
rm->audio_pkt_cnt = h * w / st->codec->block_align;
More information about the ffmpeg-devel
mailing list