[Home]

Summary:ASTERISK-06272: [patch] Converting /(root) to use *alloc() wrappers - 4
Reporter:Matthew Roth (matthew roth)Labels:
Date Opened:2006-02-07 11:24:32.000-0600Date Closed:2008-01-15 16:34:33.000-0600
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Core/General
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) root_dsp_to_file-alloc_wrappers-1.0.patch
Description:This patch updates the following files in the root source directory to use the new *alloc() wrappers:

- /dsp.c
- /enum.c
- /file.c

The following cleanup tasks were also performed on the updated code:

- malloc() + memset() consolidated to ast_calloc()
- allocations and null checks consolidated to 1 line
- sizeof calls changed to reflect the coding guidelines
- lists of #define-d numeric constants converted to enums  
- explicit void * casts removed
Comments:By: Digium Subversion (svnbot) 2008-01-15 16:34:20.000-0600

Repository: asterisk
Revision: 9259

U   trunk/dsp.c
U   trunk/enum.c
U   trunk/file.c

------------------------------------------------------------------------
r9259 | russell | 2008-01-15 16:34:19 -0600 (Tue, 15 Jan 2008) | 4 lines

various code cleanup changes including changing #define'd constants to enums,
comments to doxygen style, memory allocation to use ast_ wrappers, use calloc
instead of malloc+memset, and removing duplicated error messages (issue ASTERISK-6272)

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

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

By: Digium Subversion (svnbot) 2008-01-15 16:34:33.000-0600

Repository: asterisk
Revision: 9275

_U  team/oej/astum/
U   team/oej/astum/Makefile
U   team/oej/astum/apps/Makefile
U   team/oej/astum/apps/app_macro.c
U   team/oej/astum/cdr/Makefile
U   team/oej/astum/channels/chan_iax2.c
U   team/oej/astum/channels/chan_misdn.c
U   team/oej/astum/channels/chan_misdn_config.c
U   team/oej/astum/channels/chan_oss.c
U   team/oej/astum/channels/chan_zap.c
U   team/oej/astum/channels/misdn/Makefile
U   team/oej/astum/channels/misdn/isdn_lib.c
U   team/oej/astum/channels/misdn/isdn_lib_intern.h
U   team/oej/astum/channels/misdn/isdn_msg_parser.c
U   team/oej/astum/configs/misdn.conf.sample
U   team/oej/astum/dsp.c
U   team/oej/astum/enum.c
U   team/oej/astum/file.c
U   team/oej/astum/include/asterisk/sha1.h
U   team/oej/astum/logger.c
U   team/oej/astum/sha1.c

------------------------------------------------------------------------
r9275 | oej | 2008-01-15 16:34:33 -0600 (Tue, 15 Jan 2008) | 129 lines

Merged revisions 9150,9157,9163,9166,9186,9194,9215,9223,9225,9238-9239,9247-9248,9259-9261,9263 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
r9150 | russell | 2006-02-04 17:32:27 +0100 (Sat, 04 Feb 2006) | 2 lines

remove windows-style line endings

................
r9157 | tilghman | 2006-02-05 18:20:29 +0100 (Sun, 05 Feb 2006) | 10 lines

Merged revisions 9156 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.2

........
r9156 | tilghman | 2006-02-05 11:10:19 -0600 (Sun, 05 Feb 2006) | 2 lines

Bug 6176 - Fix race condition

........

................
r9163 | kpfleming | 2006-02-06 21:18:17 +0100 (Mon, 06 Feb 2006) | 2 lines

rename properties for forward-porting fixes so they don't conflict with developer branches

................
r9166 | kpfleming | 2006-02-06 21:30:49 +0100 (Mon, 06 Feb 2006) | 2 lines

minor cleanup

................
r9186 | crichter | 2006-02-07 14:34:59 +0100 (Tue, 07 Feb 2006) | 1 line

default values of jitterbuffer and jitterbuffer_upper_threshold should be > 0, this fixes the tv_fix warnings, because we use ast_read to transmit frames to asterisk in jitterbuffer mode, instead of queueing the audio data with ast_queue_frame.
................
r9194 | mattf | 2006-02-07 19:00:42 +0100 (Tue, 07 Feb 2006) | 2 lines

Make sure we output the APDU debug on pri debug commands

................
r9215 | crichter | 2006-02-08 17:10:21 +0100 (Wed, 08 Feb 2006) | 8 lines

* dozens of white-space removements
* some // comment removements
* debugging optimization, use port where possible
* fixed  pickup problem (pickup didn't work anymore after mqueue)
* removed some mIDSN_JOLLY defines which are not needed anymore in mqueue
* adapted the new cli.h constifications


................
r9223 | crichter | 2006-02-08 20:39:37 +0100 (Wed, 08 Feb 2006) | 1 line

L2 Signalling in PTP-TE Mode fixed
................
r9225 | crichter | 2006-02-08 20:51:53 +0100 (Wed, 08 Feb 2006) | 1 line

fix for the L2 Signalling
................
r9238 | tilghman | 2006-02-09 00:10:39 +0100 (Thu, 09 Feb 2006) | 10 lines

Merged revisions 9233 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.2

........
r9233 | tilghman | 2006-02-08 16:34:38 -0600 (Wed, 08 Feb 2006) | 2 lines

Leave it to RH/CentOS to put the freetds headers in a completely nonstandard location.

........

................
r9239 | mogorman | 2006-02-09 00:14:17 +0100 (Thu, 09 Feb 2006) | 12 lines

Merged revisions 9232 via svnmerge from
https://svn.digium.com/svn/asterisk/branches/1.2

........
r9232 | mogorman | 2006-02-08 16:12:34 -0600 (Wed, 08 Feb 2006) | 4 lines

Make logger report error,warning,notice if logger.conf
not found, also updated chan_oss to give correct
error message if its config file is not found.

........

................
r9247 | russell | 2006-02-09 02:33:50 +0100 (Thu, 09 Feb 2006) | 10 lines

Merged revisions 9246 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.2

........
r9246 | russell | 2006-02-08 20:24:55 -0500 (Wed, 08 Feb 2006) | 2 lines

reload peercontext on iax2 reload (issue ASTERISK-6279)

........

................
r9248 | russell | 2006-02-09 02:52:09 +0100 (Thu, 09 Feb 2006) | 2 lines

clarify return value (issue ASTERISK-6271)

................
r9259 | russell | 2006-02-09 03:08:04 +0100 (Thu, 09 Feb 2006) | 4 lines

various code cleanup changes including changing #define'd constants to enums,
comments to doxygen style, memory allocation to use ast_ wrappers, use calloc
instead of malloc+memset, and removing duplicated error messages (issue ASTERISK-6272)

................
r9260 | russell | 2006-02-09 03:21:57 +0100 (Thu, 09 Feb 2006) | 3 lines

remove unnecessary format check in zt_request so that we will actually accept
a requested format of ALAW (issue ASTERISK-6259, different patch)

................
r9261 | russell | 2006-02-09 03:27:30 +0100 (Thu, 09 Feb 2006) | 2 lines

remove some Makefile targets for things that don't exist in the tree

................
r9263 | russell | 2006-02-09 03:35:34 +0100 (Thu, 09 Feb 2006) | 1 line


................

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

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