Re: Patch removed from CVS

On Wed, Jan 28, 2009 at 8:58 PM, Nikos Balkanas <[email protected]> wrote:
> Hi all,
>
> Attached patch to gwlib/octstr.c provides support for %ll and %llu in
> octstr_format. To do that I introduced a new format->type: 'L' for long long
> variables. This is a prerequisite patch for #460.
>
> Please vote, and if possible test (I have tested it only with my demo
> program, not kannel).
>
> BR,
> Nikos

Why default cases are casted to (short) and (unsigned short) respectively?

Also format->prec is already long, so you don't need:

+            n = (long) format->prec;

M