[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