[asterisk-dev] [asterisk-commits] oej: branch 1.4 r242226 - /branches/1.4/channels/chan_sip.c
Olle E. Johansson
oej at edvina.net
Fri Jan 22 07:56:32 CST 2010
22 jan 2010 kl. 14.20 skrev Kevin P. Fleming:
> SVN commits to the Asterisk project wrote:
>> Author: oej
>> Date: Fri Jan 22 03:19:30 2010
>> New Revision: 242226
>>
>> URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=242226
>> Log:
>> Initialize notify_types to NULL
>>
>> -static struct ast_config *notify_types; /*!< The list of manual NOTIFY types we know how to send */
>> +static struct ast_config *notify_types = NULL; /*!< The list of manual NOTIFY types we know how to send */
>>
>> /*---------------------------- Forward declarations of functions in chan_sip.c */
>> /*! \note This is added to help splitting up chan_sip.c into several files
>
> This is unnecessary, as documented in the coding guidelines. File-scope
> static variables are always initialized to zero by the compiler/linker,
> as defined in the C language specification.
Ok, me bad again. Do you want me to revert it or can we let it be?
Seems like it doesn't matter to the result and the code is clearer (for us old C people) ;-)
/O
More information about the asterisk-dev
mailing list