[FFmpeg-devel] [PATCH] Simplify table definitions in ff_data_to_hex

Michael Niedermayer michaelni
Thu Mar 25 16:53:20 CET 2010


On Thu, Mar 25, 2010 at 04:09:50PM +0200, Martin Storsj? wrote:
> Hi,
> 
> As in $subj, declare the binary->hex mapping tables using a string instead 
> of an array of chars - makes the code a bit shorter and easier to read 
> imo.
> 
> // Martin
>  utils.c |   10 ++--------
>  1 file changed, 2 insertions(+), 8 deletions(-)
> afcf262ce7839c92902ddc97398e71ca38dfa2ba  simplify_ff_data_to_hex.patch
> commit e8b619da86acaa4631b6b28b4a15630b80f49469
> Author: Martin Storsjo <martin at martin.st>
> Date:   Thu Mar 25 16:08:13 2010 +0200
> 
>     Simplify the ff_data_to_hex function

ok

note the *char suggested in the thread is not ok (iam picky on redundant
pointers)

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Observe your enemies, for they first find out your faults. -- Antisthenes
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100325/fdc02329/attachment.pgp>



More information about the ffmpeg-devel mailing list