[asterisk-bugs] [Asterisk 0013480]: [patch] remove zap/dahdi-related code duplications with some #define-s
Asterisk Bug Tracker
noreply at bugs.digium.com
Thu Dec 18 14:42:05 CST 2008
A NOTE has been added to this issue.
======================================================================
http://bugs.digium.com/view.php?id=13480
======================================================================
Reported By: tzafrir
Assigned To: jpeeler
======================================================================
Project: Asterisk
Issue ID: 13480
Category: General
Reproducibility: N/A
Severity: tweak
Priority: normal
Status: assigned
Asterisk Version: SVN
SVN Branch (only for SVN checkouts, not tarball releases): 1.4
SVN Revision (number only!): 142358
Disclaimer on File?: N/A
Request Review:
======================================================================
Date Submitted: 2008-09-14 18:32 CDT
Last Modified: 2008-12-18 14:42 CST
======================================================================
Summary: [patch] remove zap/dahdi-related code duplications
with some #define-s
Description:
The following patch adds 2 different #define-d constants that are set
differently for Zaptel and for DAHDI (through DAHDI_COMPAT), and then 3
more common defined names,
It uses those to reduce code duplication regarding /dev/zap vs. /dev/dahdi
.
e.g.:
#ifdef HAVE_ZAPTEL
fd = open("/dev/zap/pseudo", O_RDWR);
#else
fd = open("/dev/dahdi/pseudo", O_RDWR);
#endif
is replaced with:
fd = open(DAHDI_FILE_PSEUDO, O_RDRW);
======================================================================
----------------------------------------------------------------------
(0096659) jpeeler (administrator) - 2008-12-18 14:42
http://bugs.digium.com/view.php?id=13480#c96659
----------------------------------------------------------------------
Did you consider using dahdi_chan_name rather than the DAHDI_NAME define? I
bring this up simply for no reason other than limiting the number of places
the string is set.
Issue History
Date Modified Username Field Change
======================================================================
2008-12-18 14:42 jpeeler Note Added: 0096659
======================================================================
More information about the asterisk-bugs
mailing list