[Home]

Summary:ASTERISK-04013: [patch] Changes of %i for %d
Reporter:Clod Patry (junky)Labels:
Date Opened:2005-04-28 21:32:44Date Closed:2008-01-15 15:32:38.000-0600
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Core/General
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) i_4_d_rev3.txt
Description:Like discussed today in the conf with kpflemming, i've changed all %i for %d for all int.

If a %i was really needed, i've changed it to %d, so it may be a good idea to check it carefully.
But like we all think, there's no need for %i in the current code, i think we can go with it.


****** ADDITIONAL INFORMATION ******

Disclaimer on file.
Comments:By: Kevin P. Fleming (kpfleming) 2005-04-29 10:26:49

Your patch contains other unrelated changes; please post a version that contains _only_ the %i -> %d conversions.
By: Kevin P. Fleming (kpfleming) 2005-04-29 12:10:53

Committed to CVS, thanks!
By: Digium Subversion (svnbot) 2008-01-15 15:32:38.000-0600

Repository: asterisk
Revision: 5533

U   trunk/acl.c
U   trunk/app.c
U   trunk/apps/app_dial.c
U   trunk/apps/app_disa.c
U   trunk/apps/app_groupcount.c
U   trunk/apps/app_meetme.c
U   trunk/apps/app_osplookup.c
U   trunk/apps/app_read.c
U   trunk/apps/app_readfile.c
U   trunk/apps/app_realtime.c
U   trunk/apps/app_rpt.c
U   trunk/apps/app_voicemail.c
U   trunk/cdr/cdr_pgsql.c
U   trunk/cdr/cdr_tds.c
U   trunk/channel.c
U   trunk/channels/chan_h323.c
U   trunk/channels/chan_iax2.c
U   trunk/channels/chan_mgcp.c
U   trunk/channels/chan_phone.c
U   trunk/channels/chan_sip.c
U   trunk/channels/chan_skinny.c
U   trunk/channels/chan_zap.c
U   trunk/channels/iax2-provision.c
U   trunk/cli.c
U   trunk/manager.c
U   trunk/pbx/pbx_config.c
U   trunk/pbx/pbx_dundi.c
U   trunk/pbx/pbx_loopback.c
U   trunk/pbx.c
U   trunk/res/res_features.c
U   trunk/res/res_osp.c

------------------------------------------------------------------------
r5533 | kpfleming | 2008-01-15 15:32:38 -0600 (Tue, 15 Jan 2008) | 2 lines

don't use '0' at all, since we have no current use cases that need non base-10 parsing (bug ASTERISK-4013)

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=5533