[Ffmpeg-devel] [PATCH] cosmetics - reorder bswap.h

Ramiro Ribeiro Polla ramiro
Tue Apr 24 21:35:35 CEST 2007


Hello,

Attached patches reorder bswap.h to remove code duplication.

bswap.h.returnx.diff makes 2 functions "return x;" to simplify next patch.
bswap.h.reorder.diff reorders bswap functions into bit-depth, 
special-casing inside the functions.

They remove 949 bytes from bswap.h =)

The file could then use some prettyprinting to make all asm statements 
consistent in shape, and use 4 space indentations.

Regression tests "pass" on P4 with HAVE_BYTESTREAM_H undefined. Seek 
test didn't run though, but it's not because of these patches, and I'll 
look into what triggers the error as a separate issue.

Ramiro Polla
-------------- next part --------------
A non-text attachment was scrubbed...
Name: bswap.h.returnx.diff
Type: text/x-patch
Size: 562 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070424/0a67df2d/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: bswap.h.reorder.diff
Type: text/x-patch
Size: 2983 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070424/0a67df2d/attachment-0001.bin>



More information about the ffmpeg-devel mailing list