[FFmpeg-cvslog] r23461 - trunk/libavutil/base64.h

[Måns Rullgård]

Martin Storsj? <martin at martin.st> writes:

> On Fri, 4 Jun 2010, Reimar D?ffinger wrote:
>
>> On Fri, Jun 04, 2010 at 03:15:07AM +0200, hyc wrote:
>> > Author: hyc
>> > Date: Fri Jun  4 03:15:07 2010
>> > New Revision: 23461
>> > 
>> > Log:
>> > Add AV_BASE64_SIZE() macro
>> > 
>> > Modified:
>> >    trunk/libavutil/base64.h
>> > 
>> > Modified: trunk/libavutil/base64.h
>> > ==============================================================================
>> > --- trunk/libavutil/base64.h	Fri Jun  4 01:53:10 2010	(r23460)
>> > +++ trunk/libavutil/base64.h	Fri Jun  4 03:15:07 2010	(r23461)
>> > @@ -46,4 +46,9 @@ int av_base64_decode(uint8_t *out, const
>> >   */
>> >  char *av_base64_encode(char *out, int out_size, const uint8_t *in, int in_size);
>> >  
>> > +/**
>> > + * Calculate the output size needed to base64-encode x bytes.
>> > + */
>> > +#define AV_BASE64_SIZE(x)  (((x)+2) / 3 * 4 + 1)
>> 
>> I'd like to point out that I consider this a perfect example of
>> pointless use of a macro.  Or am I missing any advantage this has
>> over a static inline function?
>
> Can a static inline function be used for allocating statically sized 
> arrays?

No, the return value of a function is never an integer constant expression.

-- 
M?ns Rullg?rd
mans at mansr.com