Re: [bug-gnulib] Re: base64

[Stepan Kasal]

Hi Jim,

On Thu, Nov 25, 2004 at 03:25:34PM +0100, Jim Meyering wrote:
> There should be parentheses around each RHS:
> 
>   #define BASE64_CEILING_DIV(n, k) ((n + (k-1)) / k)
>   #define BASE64_LENGTH(inlen) (BASE64_CEILING_DIV(inlen, 3) * 4)

Thank you for correcting me.  To make BASE64_CEILING_DIV generally usable,
one would have to do:

    #define BASE64_CEILING_DIV(n, k) (((n) + ((k)-1)) / (k))
    #define BASE64_LENGTH(inlen) (BASE64_CEILING_DIV(inlen, 3) * 4)

but perhaps we can settle with:

    #define _BASE64_CEILING_DIV(n, k) ((n + (k-1)) / k)
    #define BASE64_LENGTH(inlen) (_BASE64_CEILING_DIV(inlen, 3) * 4)

This is identical to your version, only the name of the aux macro is
changed.

Cheers,
        Stepan