[asterisk-commits] rmudgett: branch 1.8 r404212 - in /branches/1.8/addons: ./ ooh323c/src/

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Wed Dec 18 13:47:39 CST 2013


Author: rmudgett
Date: Wed Dec 18 13:47:34 2013
New Revision: 404212

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=404212
Log:
ooh323c: Fix gcc 4.6.3 compiler warnings.

Modified:
    branches/1.8/addons/ooh323c/src/memheap.c
    branches/1.8/addons/ooh323c/src/ooCapability.c
    branches/1.8/addons/ooh323c/src/ooSocket.c
    branches/1.8/addons/ooh323c/src/ooTimer.c
    branches/1.8/addons/ooh323c/src/ooq931.c
    branches/1.8/addons/ooh323c/src/perutil.c
    branches/1.8/addons/ooh323cDriver.c

Modified: branches/1.8/addons/ooh323c/src/memheap.c
URL: http://svnview.digium.com/svn/asterisk/branches/1.8/addons/ooh323c/src/memheap.c?view=diff&rev=404212&r1=404211&r2=404212
==============================================================================
--- branches/1.8/addons/ooh323c/src/memheap.c (original)
+++ branches/1.8/addons/ooh323c/src/memheap.c Wed Dec 18 13:47:34 2013
@@ -726,7 +726,7 @@
    OSMemLink** ppMemLink;
    OSMemBlk* pMemBlk;
    OSMemElemDescr* pElem;
-   OSMemLink* pMemLink, *pPrevMemLink = 0;
+   OSMemLink* pMemLink;
    void *newMem_p;
    unsigned nbytes, nunits;
 
@@ -758,7 +758,6 @@
          *(int*)(((char*)pMemLink) + sizeof (OSMemLink)) = nbytes_;
          return pMemLink->pMemBlk;
       }
-      pPrevMemLink = pMemLink;
    }
 
    /* Round number of bytes to nearest 8-byte boundary */

Modified: branches/1.8/addons/ooh323c/src/ooCapability.c
URL: http://svnview.digium.com/svn/asterisk/branches/1.8/addons/ooh323c/src/ooCapability.c?view=diff&rev=404212&r1=404211&r2=404212
==============================================================================
--- branches/1.8/addons/ooh323c/src/ooCapability.c (original)
+++ branches/1.8/addons/ooh323c/src/ooCapability.c Wed Dec 18 13:47:34 2013
@@ -1104,14 +1104,12 @@
    (ooH323EpCapability *epCap, OOCTXT* pctxt, int dir)
 {
    H245AudioCapability *pAudio=NULL;
-   OOCapParams *params;
    if(!epCap || !epCap->params)
    {
      OOTRACEERR1("Error:Invalid capability parameters to "
                  "ooCapabilityCreateSimpleCapability.\n");
      return NULL;
    }
-   params =(OOCapParams*)epCap->params;
    pAudio = (H245AudioCapability*)memAlloc(pctxt, 
                                                 sizeof(H245AudioCapability));
    if(!pAudio)
@@ -1228,7 +1226,6 @@
    (ooH323EpCapability *epCap, OOCTXT* pctxt, int dir)
 {
    H245DataApplicationCapability *pT38=NULL;
-   OOCapParams *params;
    H245DataMode_application *pT38app;
    if(!epCap || !epCap->params)
    {
@@ -1236,7 +1233,6 @@
                  "ooCapabilityCreateSimpleCapability.\n");
      return NULL;
    }
-   params =(OOCapParams*)epCap->params;
    pT38 = (H245DataApplicationCapability*)memAlloc(pctxt, 
                                                 sizeof(H245DataApplicationCapability));
    if(!pT38)

Modified: branches/1.8/addons/ooh323c/src/ooSocket.c
URL: http://svnview.digium.com/svn/asterisk/branches/1.8/addons/ooh323c/src/ooSocket.c?view=diff&rev=404212&r1=404211&r2=404212
==============================================================================
--- branches/1.8/addons/ooh323c/src/ooSocket.c (original)
+++ branches/1.8/addons/ooh323c/src/ooSocket.c Wed Dec 18 13:47:34 2013
@@ -234,7 +234,7 @@
 int ooSocketCreateUDP (OOSOCKET* psocket) 
 {
    int on;
-   struct linger linger;
+   /*struct linger linger;*/
 
    OOSOCKET sock = socket (AF_INET,
                              SOCK_DGRAM,
@@ -252,9 +252,9 @@
       OOTRACEERR1("Error:Failed to set socket option SO_REUSEADDR\n");
       return ASN_E_INVSOCKET;
    }
-   linger.l_onoff = 1;
+   /*linger.l_onoff = 1;
    linger.l_linger = 0;
-   /*if (setsockopt (sock, SOL_SOCKET, SO_LINGER, 
+   if (setsockopt (sock, SOL_SOCKET, SO_LINGER,
                  (const char* ) &linger, sizeof (linger)) == -1)
       return ASN_E_INVSOCKET;
    */

Modified: branches/1.8/addons/ooh323c/src/ooTimer.c
URL: http://svnview.digium.com/svn/asterisk/branches/1.8/addons/ooh323c/src/ooTimer.c?view=diff&rev=404212&r1=404211&r2=404212
==============================================================================
--- branches/1.8/addons/ooh323c/src/ooTimer.c (original)
+++ branches/1.8/addons/ooh323c/src/ooTimer.c Wed Dec 18 13:47:34 2013
@@ -101,7 +101,6 @@
 void ooTimerFireExpired (OOCTXT* pctxt, DList *pList)
 {
    OOTimer* pTimer;
-   int stat;
 
    while (pList->count > 0) {
       pTimer = (OOTimer*) pList->head->data;
@@ -113,7 +112,7 @@
           */
          if (pTimer->reRegister) ooTimerReset (pctxt, pList, pTimer);
 
-         stat = (*pTimer->timeoutCB)(pTimer->cbData);
+         (*pTimer->timeoutCB)(pTimer->cbData);
 
          if (!pTimer->reRegister) {
             ooTimerDelete (pctxt, pList, pTimer);

Modified: branches/1.8/addons/ooh323c/src/ooq931.c
URL: http://svnview.digium.com/svn/asterisk/branches/1.8/addons/ooh323c/src/ooq931.c?view=diff&rev=404212&r1=404211&r2=404212
==============================================================================
--- branches/1.8/addons/ooh323c/src/ooq931.c (original)
+++ branches/1.8/addons/ooh323c/src/ooq931.c Wed Dec 18 13:47:34 2013
@@ -2183,7 +2183,7 @@
    ooAliases *pNewAlias=NULL, *alias=NULL;
    struct timespec ts;
    struct timeval tv;
-   int i=0, irand=0, ret = OO_OK;
+   int i=0, irand=0;
    /* Note: We keep same callToken, for new call which is going
       to replace an existing call, thus treating it as a single call.*/
 
@@ -2235,7 +2235,7 @@
    {
      /* No need to check registration status here as it is already checked for
         MakeCall command */
-      ret = ooGkClientSendAdmissionRequest(gH323ep.gkClient, fwdedCall, FALSE);
+      ooGkClientSendAdmissionRequest(gH323ep.gkClient, fwdedCall, FALSE);
       fwdedCall->callState = OO_CALL_WAITING_ADMISSION;
       ast_mutex_lock(&fwdedCall->Lock);
 	  tv = ast_tvnow();
@@ -2249,7 +2249,7 @@
    }
    if (fwdedCall->callState < OO_CALL_CLEAR) {
       ast_mutex_lock(&fwdedCall->Lock);
-      ret = ooH323CallAdmitted (fwdedCall);
+      ooH323CallAdmitted (fwdedCall);
       ast_mutex_unlock(&fwdedCall->Lock);
    }
 
@@ -2276,7 +2276,6 @@
 
 int ooH323MakeCall(char *dest, char *callToken, ooCallOptions *opts)
 {
-   OOCTXT *pctxt;
    OOH323CallData *call;
    int ret=OO_OK, i=0, irand=0;
    char tmp[30]="\0";
@@ -2303,7 +2302,6 @@
       return OO_FAILED;
    }
 
-   pctxt = call->pctxt;
    if(opts)
    {
       if(opts->fastStart)

Modified: branches/1.8/addons/ooh323c/src/perutil.c
URL: http://svnview.digium.com/svn/asterisk/branches/1.8/addons/ooh323c/src/perutil.c?view=diff&rev=404212&r1=404211&r2=404212
==============================================================================
--- branches/1.8/addons/ooh323c/src/perutil.c (original)
+++ branches/1.8/addons/ooh323c/src/perutil.c Wed Dec 18 13:47:34 2013
@@ -168,7 +168,7 @@
 int checkSizeConstraint(OOCTXT* pctxt, int size)
 {
    Asn1SizeCnst* pSize;
-   ASN1UINT lower, upper;
+   ASN1UINT upper;
    ASN1BOOL extbit;
    int      stat;
 
@@ -186,7 +186,6 @@
 
    pSize = getSizeConstraint (pctxt, extbit);
 
-   lower = (pSize) ? pSize->lower : 0;
    upper = (pSize) ? pSize->upper : ASN1UINT_MAX;
 
    if (upper < (ASN1UINT)size) {

Modified: branches/1.8/addons/ooh323cDriver.c
URL: http://svnview.digium.com/svn/asterisk/branches/1.8/addons/ooh323cDriver.c?view=diff&rev=404212&r1=404211&r2=404212
==============================================================================
--- branches/1.8/addons/ooh323cDriver.c (original)
+++ branches/1.8/addons/ooh323cDriver.c Wed Dec 18 13:47:34 2013
@@ -100,8 +100,10 @@
 	pfds[0].fd = mycthread->thePipe[0];
 	pfds[0].events = POLLIN;
 	ooSocketPoll(pfds, 1, SEC_TO_HOLD_THREAD * 1000);
-	if (ooPDRead(pfds, 1, mycthread->thePipe[0]))
+	if (ooPDRead(pfds, 1, mycthread->thePipe[0])) {
 		res = read(mycthread->thePipe[0], &c, 1);
+		(void) res;/* Shut up compiler: Set but not used and unused return value of read. */
+	}
 
  	ast_mutex_lock(&callThreadsLock);
 	ast_mutex_lock(&mycthread->lock);
@@ -182,6 +184,7 @@
 	cur->inUse = TRUE;
 	cur->call = call;
 	res = write(cur->thePipe[1], &c, 1);
+	(void) res;/* Shut up compiler: Set but not used and unused return value of write. */
 	ast_mutex_unlock(&cur->lock);
 
  }




More information about the asterisk-commits mailing list