[FFmpeg-devel] [PATCH] provide AV_BASE64_SIZE() macro

Michael Niedermayer michaelni
Thu Jun 3 23:44:32 CEST 2010


On Thu, Jun 03, 2010 at 10:39:52PM +0100, M?ns Rullg?rd wrote:
> Michael Niedermayer <michaelni at gmx.at> writes:
> 
> > On Thu, Jun 03, 2010 at 01:50:06PM -0700, Howard Chu wrote:
> >> Just for convenience...
> >
> > IMHO this is making the code harder to understand
> 
> How so?  It's an obscure calculation repeated multiple times.  That is
> exactly the kind of situation functions and macros were invented for.

to me the calculation is clear, the macro name is less clear to me.

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

Breaking DRM is a little like attempting to break through a door even
though the window is wide open and the only thing in the house is a bunch
of things you dont want and which you would get tomorrow for free anyway
-------------- 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/20100603/a1e34c1d/attachment.pgp>



More information about the ffmpeg-devel mailing list