[asterisk-bugs] [Asterisk 0013064]: [patch] add some const qualifiers

noreply at bugs.digium.com noreply at bugs.digium.com
Sun Jul 13 11:35:16 CDT 2008


A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=13064 
====================================================================== 
Reported By:                gknispel_proformatique
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   13064
Category:                   General
Reproducibility:            N/A
Severity:                   minor
Priority:                   normal
Status:                     new
Asterisk Version:           SVN 
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!):  
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             07-12-2008 20:11 CDT
Last Modified:              07-13-2008 11:35 CDT
====================================================================== 
Summary:                    [patch] add some const qualifiers
Description: 
The attached patch adds some const qualifiers were applicable.
It is not yet finished (the whole code base has not been completely
reviewed) but seems usable by itself without breaking the build.

====================================================================== 

---------------------------------------------------------------------- 
 Corydon76 - 07-13-08 11:35  
---------------------------------------------------------------------- 
1) Please don't add the const keyword to casts.  It does absolutely
nothing.

2) We are very cautious about altering public APIs, because it affects
backwards compatibility, especially on function pointers.  Please do not
make those changes without discussion on the -dev list.

3) Please do not remove existing casts.  They are there for good reason
(mostly for compatibility with other architectures). 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
07-13-08 11:35  Corydon76      Note Added: 0090159                          
======================================================================




More information about the asterisk-bugs mailing list