[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