Review Board 1.7.16


rfc4235 - Section 4.1: Versions MUST be representable using a non-negative 32 bit integer.

Review Request #1694 - Created Jan. 25, 2012 and submitted

Alec Davis
Asterisk 1.8 and above
Reviewers
asterisk-dev
Asterisk
printf %d may print negative version numbers, if a subscription exists for long enough.

I'm not confortable with %u on 32 or 64 bit systems.

Should I perhaps use a type cast, printf("%lu\n", (unsigned long)dialogver);
Compiles
Ship it!
Posted (Jan. 26, 2012, 1:32 a.m.)
Using '%u' and a fixed 32-bit unsigned type should be fine.

https://reviewboard.asterisk.org/ runs on a server provided by Digium, Inc. and uses bandwidth donated to the open source Asterisk community by API Digital Communications in Huntsville, AL USA.
Please report problems with this site to asteriskteam@digium.com.