Turns out the "helpful" setting of ms and res in this
macro is completely useless after the timeout antipattern
fix.
If you're a new guy looking to write code, don't write
a macro like this one.
........
Merged revisions 376087 from http://svn.asterisk.org/svn/asterisk/branches/1.8
........
Merged revisions 376088 from http://svn.asterisk.org/svn/asterisk/branches/10
........
Merged revisions 376089 from http://svn.asterisk.org/svn/asterisk/branches/11
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@376092
65c4cc65-6c06-0410-ace0-
fbb531ad65f3
#define GENERIC_FAX_EXEC_ERROR_QUIET(fax, chan, errorstr, reason) \
do { \
GENERIC_FAX_EXEC_SET_VARS(fax, chan, errorstr, reason); \
- res = ms = -1; \
} while (0)
#define GENERIC_FAX_EXEC_ERROR(fax, chan, errorstr, reason) \
{
int ms;
int timeout = RES_FAX_TIMEOUT;
- int res, chancount;
+ int chancount;
unsigned int expected_frametype = -1;
union ast_frame_subclass expected_framesubclass = { .integer = -1 };
unsigned int t38negotiated = (ast_channel_get_t38_state(chan) == T38_STATE_NEGOTIATED);