# HG changeset patch # User reimar # Date 1253360211 0 # Node ID ee31a41c50622603d9f6a1fabb27c1cf4599c2da # Parent 272cac6de8ced54562c0085e635926dcefbfebfe Simplify color_flags calculation, remove a pointless initialization and use AV_RB16 where possible. diff -r 272cac6de8ce -r ee31a41c5062 smc.c --- a/smc.c Sat Sep 19 11:35:55 2009 +0000 +++ b/smc.c Sat Sep 19 11:36:51 2009 +0000 @@ -366,16 +366,11 @@ flags_a = xx012456, flags_b = xx89A37B */ /* build the color flags */ - color_flags_a = color_flags_b = 0; color_flags_a = - (s->buf[stream_ptr + 0] << 16) | - ((s->buf[stream_ptr + 1] & 0xF0) << 8) | - ((s->buf[stream_ptr + 2] & 0xF0) << 4) | - ((s->buf[stream_ptr + 2] & 0x0F) << 4) | - ((s->buf[stream_ptr + 3] & 0xF0) >> 4); + ((AV_RB16(s->buf + stream_ptr ) & 0xFFF0) << 8) | + (AV_RB16(s->buf + stream_ptr + 2) >> 4); color_flags_b = - (s->buf[stream_ptr + 4] << 16) | - ((s->buf[stream_ptr + 5] & 0xF0) << 8) | + ((AV_RB16(s->buf + stream_ptr + 4) & 0xFFF0) << 8) | ((s->buf[stream_ptr + 1] & 0x0F) << 8) | ((s->buf[stream_ptr + 3] & 0x0F) << 4) | (s->buf[stream_ptr + 5] & 0x0F);