[Asterisk-code-review] sig pri.h: force restart unavailable chans in wrong scope (asterisk[11])

Mark Michelson asteriskteam at digium.com
Fri Jul 17 12:44:06 CDT 2015


Mark Michelson has submitted this change and it was merged.

Change subject: sig_pri.h: force_restart_unavailable_chans in wrong scope
......................................................................


sig_pri.h: force_restart_unavailable_chans in wrong scope

In channels/sig_pri.h, struct sig_pri_span, the field
force_restart_unavailable_chans is only defined if

#if defined(HAVE_PRI_MCID) is true.

All other occurences of force_restart_unavailable_chans are outside of the

#if defined(HAVE_PRI_MCID)
endif

scope.

ASTERISK-25257 #close
Reported by: Patric Marschall

Change-Id: I071de89cc2cd0d85927a013036e235851f672549
---
M channels/sig_pri.h
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Mark Michelson: Looks good to me, approved
  Richard Mudgett: Looks good to me, but someone else must approve
  Anonymous Coward #1000019: Verified



diff --git a/channels/sig_pri.h b/channels/sig_pri.h
index 8debbc5..1a95b6e 100644
--- a/channels/sig_pri.h
+++ b/channels/sig_pri.h
@@ -481,9 +481,9 @@
 #if defined(HAVE_PRI_MCID)
 	/*! \brief TRUE if allow sending MCID request on this span. */
 	unsigned int mcid_send:1;
+#endif	/* defined(HAVE_PRI_MCID) */
 	/*! \brief TRUE if forcing RESTART when receive cause 44 on this span. */
 	unsigned int force_restart_unavailable_chans:1;
-#endif	/* defined(HAVE_PRI_MCID) */
 #if defined(HAVE_PRI_DATETIME_SEND)
 	/*! \brief Configured date/time ie send policy option. */
 	int datetime_send;

-- 
To view, visit https://gerrit.asterisk.org/923
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I071de89cc2cd0d85927a013036e235851f672549
Gerrit-PatchSet: 2
Gerrit-Project: asterisk
Gerrit-Branch: 11
Gerrit-Owner: Patric Marschall <patric.marschall at 1und1.de>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Mark Michelson <mmichelson at digium.com>
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>



More information about the asterisk-code-review mailing list