[clblas] 28/125: Fix compilation warnings: - assignment used as condition warnings for the error checking - writeable char constant warning. - Fix sprintf of size_t values to not (potentially) print garbage. - Remove extra parentheses around condition in if(). - Replace explicit NULL dereference by a call to abort().

Ghislain Vaillant ghisvail-guest at moszumanska.debian.org
Fri May 29 06:57:19 UTC 2015


This is an automated email from the git hooks/post-receive script.

ghisvail-guest pushed a commit to branch master
in repository clblas.

commit bb0ce37c16d3004eeea202ac851e99d4687be496
Author: Arnaud Bergeron <abergeron at gmail.com>
Date:   Wed Sep 18 21:24:44 2013 -0400

    Fix compilation warnings:
     - assignment used as condition warnings for the error checking
     - writeable char constant warning.
     - Fix sprintf of size_t values to not (potentially) print garbage.
     - Remove extra parentheses around condition in if().
     - Replace explicit NULL dereference by a call to abort().
---
 src/include/kerngen.h                 |  6 ++++++
 src/library/blas/gens/ger_lds.cpp     |  4 ++--
 src/library/blas/gens/her2_lds.cpp    |  2 +-
 src/library/blas/gens/her_lds.cpp     |  2 +-
 src/library/blas/gens/kprintf.cpp     |  2 +-
 src/library/blas/gens/symm_cached.cpp |  8 ++++----
 src/library/blas/gens/syr2_lds.cpp    |  2 +-
 src/library/blas/gens/syr_lds.cpp     |  2 +-
 src/library/blas/gens/trmv_reg.cpp    |  4 ++--
 src/library/blas/gens/trsv_gemv.cpp   | 10 +++++-----
 src/library/blas/xaxpy.c              |  4 ++--
 src/library/blas/xcopy.c              |  4 ++--
 src/library/blas/xdot.c               |  8 ++++----
 src/library/blas/xgemm2.c             |  8 ++++----
 src/library/blas/xger.c               |  8 ++++----
 src/library/blas/xhemv.c              | 10 +++++-----
 src/library/blas/xher.c               |  6 +++---
 src/library/blas/xher2.c              |  8 ++++----
 src/library/blas/xher2k.c             |  8 ++++----
 src/library/blas/xherk.c              |  6 +++---
 src/library/blas/xhpmv.c              | 12 ++++++------
 src/library/blas/xrot.c               |  4 ++--
 src/library/blas/xrotg.c              |  8 ++++----
 src/library/blas/xrotm.c              |  6 +++---
 src/library/blas/xrotmg.c             | 10 +++++-----
 src/library/blas/xscal.c              |  2 +-
 src/library/blas/xshbmv.c             | 10 +++++-----
 src/library/blas/xspmv.c              | 12 ++++++------
 src/library/blas/xswap.c              |  4 ++--
 src/library/blas/xsymm.c              | 10 +++++-----
 src/library/blas/xsymv.c              | 10 +++++-----
 src/library/blas/xsyr.c               |  6 +++---
 src/library/blas/xsyr2.c              |  8 ++++----
 src/library/blas/xsyr2k.c             |  8 ++++----
 src/library/blas/xsyrk.c              |  6 +++---
 src/library/blas/xtbmv.c              |  8 ++++----
 src/library/blas/xtrmm.c              | 10 +++++-----
 src/library/blas/xtrmv.c              |  8 ++++----
 src/library/blas/xtrsm.c              | 10 +++++-----
 src/library/blas/xtrsv.c              |  6 +++---
 src/library/tools/tplgen/tplgen.cpp   |  2 +-
 src/library/tools/tune/subdim.c       |  2 +-
 42 files changed, 140 insertions(+), 134 deletions(-)

diff --git a/src/include/kerngen.h b/src/include/kerngen.h
index dd44b9e..73ee191 100644
--- a/src/include/kerngen.h
+++ b/src/include/kerngen.h
@@ -42,6 +42,12 @@
  */
 /*@{*/
 
+#ifdef _MSC_VER
+#define SPREFIX "I"
+#else
+#define SPREFIX "z"
+#endif
+
 #define SUBDIM_UNUSED (size_t)-1
 
 enum {
diff --git a/src/library/blas/gens/ger_lds.cpp b/src/library/blas/gens/ger_lds.cpp
index b74945e..ebcd577 100644
--- a/src/library/blas/gens/ger_lds.cpp
+++ b/src/library/blas/gens/ger_lds.cpp
@@ -282,8 +282,8 @@ generator(
 
 	BH = subdims->y;
 	BW = subdims->x;
-	sprintf( bhStr, "%d", BH );
-	sprintf( bwStr, "%d", BW );
+	sprintf( bhStr, "%" SPREFIX "u", BH );
+	sprintf( bwStr, "%" SPREFIX "u", BW );
 
 	#ifdef DEBUG_GER
     printf("BH = %s\n", bhStr);
diff --git a/src/library/blas/gens/her2_lds.cpp b/src/library/blas/gens/her2_lds.cpp
index a409c1a..e724f11 100644
--- a/src/library/blas/gens/her2_lds.cpp
+++ b/src/library/blas/gens/her2_lds.cpp
@@ -301,7 +301,7 @@ generator(
 	}
 	kprintf kobj( Prefix[extraFlags->dtype], vecLenA, doVLOAD, doVLOAD);
 
-	sprintf( targetRows, "%d", TARGETROWS );
+	sprintf( targetRows, "%" SPREFIX "u", TARGETROWS );
 	sprintf( blockSize, "%d", BLOCKSIZE );
 
 	#ifdef DEBUG_HER2
diff --git a/src/library/blas/gens/her_lds.cpp b/src/library/blas/gens/her_lds.cpp
index e174de2..6b489cf 100644
--- a/src/library/blas/gens/her_lds.cpp
+++ b/src/library/blas/gens/her_lds.cpp
@@ -300,7 +300,7 @@ generator(
 	}
 	kprintf kobj( Prefix[extraFlags->dtype], vecLenA, doVLOAD, doVLOAD);
 
-	sprintf( targetRows, "%d", TARGETROWS );
+	sprintf( targetRows, "%" SPREFIX "u", TARGETROWS );
 	sprintf( blockSize, "%d", BLOCKSIZE );
 
 	#ifdef DEBUG_HER
diff --git a/src/library/blas/gens/kprintf.cpp b/src/library/blas/gens/kprintf.cpp
index 54772fa..d5cbecb 100644
--- a/src/library/blas/gens/kprintf.cpp
+++ b/src/library/blas/gens/kprintf.cpp
@@ -346,7 +346,7 @@ char* kprintf::mystrtok( char* in, const char* tok)
             bool tokenFound = false;
             for( size_t i=0 ; i <= (strlen(tok) - 1); i++)
             {
-                if ((*strtokPtr == tok[i]))
+                if (*strtokPtr == tok[i])
                 {
                     if ( tok[i] == '(')
                     {
diff --git a/src/library/blas/gens/symm_cached.cpp b/src/library/blas/gens/symm_cached.cpp
index cc8c035..4001182 100644
--- a/src/library/blas/gens/symm_cached.cpp
+++ b/src/library/blas/gens/symm_cached.cpp
@@ -193,10 +193,10 @@ generator(
 		printf("WARNING: SYMM- generator: subdim and blocksize in-compatible.\n");
 	}
 
-	sprintf(width, "%d", Y);
-	sprintf(itemy, "%lu", ITEMY);
-	sprintf(itemx, "%lu", ITEMX);
-	sprintf(itemy_by_width, "%lu", (size_t) ITEMY/kextra->vecLenA);
+	sprintf(width, "%" SPREFIX "u", Y);
+	sprintf(itemy, "%" SPREFIX "u", ITEMY);
+	sprintf(itemx, "%" SPREFIX "u", ITEMX);
+	sprintf(itemy_by_width, "%" SPREFIX "u", (size_t) ITEMY/kextra->vecLenA);
 
 	kobj.put("%WIDTH", width);
 	kobj.put("%ITEMX", itemx);
diff --git a/src/library/blas/gens/syr2_lds.cpp b/src/library/blas/gens/syr2_lds.cpp
index 9fccb05..4abb4ad 100644
--- a/src/library/blas/gens/syr2_lds.cpp
+++ b/src/library/blas/gens/syr2_lds.cpp
@@ -308,7 +308,7 @@ generator(
 	}
 	kprintf kobj( Prefix[extraFlags->dtype], vecLenA, doVLOAD, doVLOAD);
 
-	sprintf( targetRows, "%d", TARGETROWS );
+	sprintf( targetRows, "%" SPREFIX "u", TARGETROWS );
 	sprintf( blockSize, "%d", BLOCKSIZE );
 
 	#ifdef DEBUG_SYR2
diff --git a/src/library/blas/gens/syr_lds.cpp b/src/library/blas/gens/syr_lds.cpp
index 0a12ef4..2379a6b 100644
--- a/src/library/blas/gens/syr_lds.cpp
+++ b/src/library/blas/gens/syr_lds.cpp
@@ -308,7 +308,7 @@ generator(
 	}
 	kprintf kobj( Prefix[extraFlags->dtype], vecLenA, doVLOAD, doVLOAD);
 
-	sprintf( targetRows, "%d", TARGETROWS );
+	sprintf( targetRows, "%" SPREFIX "u", TARGETROWS );
 	sprintf( blockSize, "%d", BLOCKSIZE );
 
 	#ifdef DEBUG_SYR
diff --git a/src/library/blas/gens/trmv_reg.cpp b/src/library/blas/gens/trmv_reg.cpp
index 28ee1f2..25e750e 100644
--- a/src/library/blas/gens/trmv_reg.cpp
+++ b/src/library/blas/gens/trmv_reg.cpp
@@ -381,8 +381,8 @@ generator(
 	}
 	kprintf kobj( Prefix[extraFlags->dtype], vecLenA, doVLOAD);
 
-    sprintf( targetRows, "%d", TARGETROWS );
-	sprintf( blockSize, "%d", BLOCKSIZE );
+    sprintf( targetRows, "%" SPREFIX "u", TARGETROWS );
+	sprintf( blockSize, "%" SPREFIX "u", BLOCKSIZE );
 
 	#ifdef DEBUG_TRMV
     printf("TARGET ROWS = %s\n", targetRows);
diff --git a/src/library/blas/gens/trsv_gemv.cpp b/src/library/blas/gens/trsv_gemv.cpp
index 49d5371..5047bf1 100644
--- a/src/library/blas/gens/trsv_gemv.cpp
+++ b/src/library/blas/gens/trsv_gemv.cpp
@@ -415,9 +415,9 @@ generator(
 		{
 			return 0;
 		}
-        sprintf( TARGETHEIGHT_S, "%d", TARGETHEIGHT );
+        sprintf( TARGETHEIGHT_S, "%" SPREFIX "u", TARGETHEIGHT );
 	    sprintf( BLOCKSIZE_S, "%d", BLOCKSIZE );
-        sprintf( TRIANGLE_HEIGHT_S, "%d", subdims->y );
+        sprintf( TRIANGLE_HEIGHT_S, "%" SPREFIX "u", subdims->y );
 
 		kobj.put("%TARGET_HEIGHT", TARGETHEIGHT_S);
 		kobj.put("%BLOCKSIZE", BLOCKSIZE_S);
@@ -433,9 +433,9 @@ generator(
 		{
 			return 0;
 		}
-        sprintf( TARGETROWS_S, "%d", TARGETROWS );
-	    sprintf( TARGETWIDTH_S, "%d", TARGETWIDTH );
-        sprintf( NLOOPS_S, "%d", NLOOPS );
+        sprintf( TARGETROWS_S, "%" SPREFIX "u", TARGETROWS );
+	    sprintf( TARGETWIDTH_S, "%" SPREFIX "u", TARGETWIDTH );
+        sprintf( NLOOPS_S, "%" SPREFIX "u", NLOOPS );
 		kobj.put("%TARGET_ROWS", TARGETROWS_S);
 		kobj.put("%TARGET_WIDTH", TARGETWIDTH_S);
 		kobj.put("%NLOOPS", NLOOPS_S);
diff --git a/src/library/blas/xaxpy.c b/src/library/blas/xaxpy.c
index 7499c41..d57b4c2 100644
--- a/src/library/blas/xaxpy.c
+++ b/src/library/blas/xaxpy.c
@@ -60,11 +60,11 @@ doAxpy(
 
 		// Check wheather enough memory was allocated
 
-		if (retCode = checkVectorSizes(kargs->dtype, N, X, offx, incx, X_VEC_ERRSET )) {
+		if ((retCode = checkVectorSizes(kargs->dtype, N, X, offx, incx, X_VEC_ERRSET))) {
 			printf("Invalid Size for X\n");
             return retCode;
 		}
-		if (retCode = checkVectorSizes(kargs->dtype, N, Y, offy, incy, Y_VEC_ERRSET )) {
+		if ((retCode = checkVectorSizes(kargs->dtype, N, Y, offy, incy, Y_VEC_ERRSET))) {
 			printf("Invalid Size for Y\n");
             return retCode;
 		}
diff --git a/src/library/blas/xcopy.c b/src/library/blas/xcopy.c
index e0ea2a0..8e37597 100644
--- a/src/library/blas/xcopy.c
+++ b/src/library/blas/xcopy.c
@@ -60,11 +60,11 @@ doCopy(
 
 		// Check wheather enough memory was allocated
 
-		if (retCode = checkVectorSizes(kargs->dtype, N, X, offx, incx, X_VEC_ERRSET )) {
+		if ((retCode = checkVectorSizes(kargs->dtype, N, X, offx, incx, X_VEC_ERRSET))) {
 			printf("Invalid Size for X\n");
             return retCode;
 		}
-		if (retCode = checkVectorSizes(kargs->dtype, N, Y, offy, incy, Y_VEC_ERRSET )) {
+		if ((retCode = checkVectorSizes(kargs->dtype, N, Y, offy, incy, Y_VEC_ERRSET))) {
 			printf("Invalid Size for Y\n");
             return retCode;
 		}
diff --git a/src/library/blas/xdot.c b/src/library/blas/xdot.c
index f29cdb6..67bf4cd 100644
--- a/src/library/blas/xdot.c
+++ b/src/library/blas/xdot.c
@@ -67,20 +67,20 @@ doDot(
 
 		// Check wheather enough memory was allocated
 
-		if (retCode = checkVectorSizes(kargs->dtype, N, X, offx, incx, X_VEC_ERRSET )) {
+		if ((retCode = checkVectorSizes(kargs->dtype, N, X, offx, incx, X_VEC_ERRSET))) {
 			printf("Invalid Size for X\n");
             return retCode;
 		}
-		if (retCode = checkVectorSizes(kargs->dtype, N, Y, offy, incy, Y_VEC_ERRSET )) {
+		if ((retCode = checkVectorSizes(kargs->dtype, N, Y, offy, incy, Y_VEC_ERRSET))) {
 			printf("Invalid Size for Y\n");
             return retCode;
 		}
 		// Minimum size of scratchBuff is N
-		if (retCode = checkVectorSizes(kargs->dtype, N, scratchBuff, 0, 1, X_VEC_ERRSET )) {
+		if ((retCode = checkVectorSizes(kargs->dtype, N, scratchBuff, 0, 1, X_VEC_ERRSET))) {
 			printf("Insufficient ScratchBuff\n");
             return retCode;
 		}
-		if (retCode = checkVectorSizes(kargs->dtype, 1, dotProduct, offDP, 1, Y_VEC_ERRSET )) {
+		if ((retCode = checkVectorSizes(kargs->dtype, 1, dotProduct, offDP, 1, Y_VEC_ERRSET))) {
 			printf("Invalid Size for dotProduct\n");
             return retCode;
 		}
diff --git a/src/library/blas/xgemm2.c b/src/library/blas/xgemm2.c
index 0a5ae43..2bba00a 100644
--- a/src/library/blas/xgemm2.c
+++ b/src/library/blas/xgemm2.c
@@ -209,18 +209,18 @@ doGemm(
 
     /* Validate arguments */
 
-    if (retCode = checkMemObjects(A, B, C, true, A_MAT_ERRSET, B_MAT_ERRSET, C_MAT_ERRSET)) {
+    if ((retCode = checkMemObjects(A, B, C, true, A_MAT_ERRSET, B_MAT_ERRSET, C_MAT_ERRSET))) {
         return retCode;
     }
     if (K != 0) {
-        if (retCode = checkMatrixSizes(kargs->dtype, order, transA, M, K, A, offA, lda, A_MAT_ERRSET )) {
+        if ((retCode = checkMatrixSizes(kargs->dtype, order, transA, M, K, A, offA, lda, A_MAT_ERRSET))) {
             return retCode;
         }
-        if (retCode = checkMatrixSizes(kargs->dtype, order, transB, K, N, B, offB, ldb, B_MAT_ERRSET )) {
+        if ((retCode = checkMatrixSizes(kargs->dtype, order, transB, K, N, B, offB, ldb, B_MAT_ERRSET))) {
             return retCode;
         }
     }
-    if (retCode = checkMatrixSizes(kargs->dtype, order, clblasNoTrans, M, N, C, offC, ldc, C_MAT_ERRSET )) {
+    if ((retCode = checkMatrixSizes(kargs->dtype, order, clblasNoTrans, M, N, C, offC, ldc, C_MAT_ERRSET))) {
             return retCode;
     }
 
diff --git a/src/library/blas/xger.c b/src/library/blas/xger.c
index 92d4b31..c9e9e1c 100644
--- a/src/library/blas/xger.c
+++ b/src/library/blas/xger.c
@@ -58,23 +58,23 @@ doGer(
 
 		/* Validate arguments */
 
-		if (retCode = checkMemObjects(A, X, Y, true, A_MAT_ERRSET, X_VEC_ERRSET, Y_VEC_ERRSET )) {
+		if ((retCode = checkMemObjects(A, X, Y, true, A_MAT_ERRSET, X_VEC_ERRSET, Y_VEC_ERRSET))) {
 			printf("Invalid mem object..\n");
             return retCode;
 		}
 
 		// Check wheather enough memory was allocated
 
-		if (retCode = checkMatrixSizes(kargs->dtype, order, clblasNoTrans, M, N, A, offa, lda, A_MAT_ERRSET )) {
+		if ((retCode = checkMatrixSizes(kargs->dtype, order, clblasNoTrans, M, N, A, offa, lda, A_MAT_ERRSET))) {
 
 			printf("Invalid Size for A %d\n",retCode );
             return retCode;
 		}
-		if (retCode = checkVectorSizes(kargs->dtype, M, X, offx, incx, X_VEC_ERRSET )) {
+		if ((retCode = checkVectorSizes(kargs->dtype, M, X, offx, incx, X_VEC_ERRSET))) {
 			printf("Invalid Size for X\n");
             return retCode;
 		}
-		if (retCode = checkVectorSizes(kargs->dtype, N, Y, offy, incy, Y_VEC_ERRSET )) {
+		if ((retCode = checkVectorSizes(kargs->dtype, N, Y, offy, incy, Y_VEC_ERRSET))) {
 			printf("Invalid Size for Y\n");
             return retCode;
 		}
diff --git a/src/library/blas/xhemv.c b/src/library/blas/xhemv.c
index 0db6a8f..21011dd 100644
--- a/src/library/blas/xhemv.c
+++ b/src/library/blas/xhemv.c
@@ -54,17 +54,17 @@ doHemv(
 
     /* Validate arguments */
 
-    if (retCode = checkMemObjects(A, x, y, true, A_MAT_ERRSET, X_VEC_ERRSET, Y_VEC_ERRSET )) {
+    if ((retCode = checkMemObjects(A, x, y, true, A_MAT_ERRSET, X_VEC_ERRSET, Y_VEC_ERRSET))) {
         return retCode;
     }
-    if (retCode = checkMatrixSizes(kargs->dtype, order, clblasNoTrans, N, N, A,
-                         offA, lda, A_MAT_ERRSET )) {
+    if ((retCode = checkMatrixSizes(kargs->dtype, order, clblasNoTrans, N, N,
+                                    A, offA, lda, A_MAT_ERRSET))) {
         return retCode;
     }
-    if (retCode = checkVectorSizes(kargs->dtype, N, x, offx, incx, X_VEC_ERRSET )) {
+    if ((retCode = checkVectorSizes(kargs->dtype, N, x, offx, incx, X_VEC_ERRSET))) {
         return retCode;
     }
-    if (retCode = checkVectorSizes(kargs->dtype, N, y, offy, incy, Y_VEC_ERRSET )) {
+    if ((retCode = checkVectorSizes(kargs->dtype, N, y, offy, incy, Y_VEC_ERRSET))) {
         return retCode;
     }
 	if ((commandQueues == NULL) || (numCommandQueues == 0))
diff --git a/src/library/blas/xher.c b/src/library/blas/xher.c
index af36962..7131057 100644
--- a/src/library/blas/xher.c
+++ b/src/library/blas/xher.c
@@ -56,16 +56,16 @@ doher(
 
     /* Validate arguments */
 
-    if (retCode = checkMemObjects(A, X, 0, false, A_MAT_ERRSET, X_VEC_ERRSET, END_ERRSET )) {
+    if ((retCode = checkMemObjects(A, X, 0, false, A_MAT_ERRSET, X_VEC_ERRSET, END_ERRSET))) {
    		printf("Invalid mem object..\n");
         return retCode;
     }
 
-    if (retCode = checkMatrixSizes(kargs->dtype, order, clblasNoTrans, N, N, A, offa, lda, A_MAT_ERRSET )) {
+    if ((retCode = checkMatrixSizes(kargs->dtype, order, clblasNoTrans, N, N, A, offa, lda, A_MAT_ERRSET))) {
         printf("Invalid Size for A\n");
         return retCode;
     }
-    if (retCode = checkVectorSizes(kargs->dtype, N, X, offx, incx, X_VEC_ERRSET)) {
+    if ((retCode = checkVectorSizes(kargs->dtype, N, X, offx, incx, X_VEC_ERRSET))) {
         printf("Invalid Size for X\n");
         return retCode;
     }
diff --git a/src/library/blas/xher2.c b/src/library/blas/xher2.c
index cb67659..21a8ddc 100644
--- a/src/library/blas/xher2.c
+++ b/src/library/blas/xher2.c
@@ -59,21 +59,21 @@ doHer2(
 
     /* Validate arguments */
 
-    if (retCode = checkMemObjects(A, X, Y, true, A_MAT_ERRSET, X_VEC_ERRSET, Y_VEC_ERRSET )) {
+    if ((retCode = checkMemObjects(A, X, Y, true, A_MAT_ERRSET, X_VEC_ERRSET, Y_VEC_ERRSET))) {
         printf("Invalid mem object..\n");
         return retCode;
     }
 
-    if (retCode = checkMatrixSizes(kargs->dtype, order, clblasNoTrans, N, N, A, offa, lda, A_MAT_ERRSET )) {
+    if ((retCode = checkMatrixSizes(kargs->dtype, order, clblasNoTrans, N, N, A, offa, lda, A_MAT_ERRSET))) {
         printf("Invalid Size for A\n");
         return retCode;
     }
-    if (retCode = checkVectorSizes(kargs->dtype, N, X, offx, incx, X_VEC_ERRSET )) {
+    if ((retCode = checkVectorSizes(kargs->dtype, N, X, offx, incx, X_VEC_ERRSET))) {
         printf("Invalid Size for X\n");
         return retCode;
     }
 
-	if (retCode = checkVectorSizes(kargs->dtype, N, Y, offy, incy, Y_VEC_ERRSET )) {
+	if ((retCode = checkVectorSizes(kargs->dtype, N, Y, offy, incy, Y_VEC_ERRSET))) {
         printf("Invalid Size for Y\n");
         return retCode;
     }
diff --git a/src/library/blas/xher2k.c b/src/library/blas/xher2k.c
index 302a648..4c3d2f2 100644
--- a/src/library/blas/xher2k.c
+++ b/src/library/blas/xher2k.c
@@ -71,7 +71,7 @@ doHer2k(
     }
 
     // Validate arguments
-    if (retCode = checkMemObjects(A, B, C, true, A_MAT_ERRSET, B_MAT_ERRSET, C_MAT_ERRSET )) {
+    if ((retCode = checkMemObjects(A, B, C, true, A_MAT_ERRSET, B_MAT_ERRSET, C_MAT_ERRSET))) {
         return retCode;
     }
 
@@ -79,15 +79,15 @@ doHer2k(
         return clblasInvalidValue;
     }
 
-    if (retCode = checkMatrixSizes(kargs->dtype, order, transA, N, K, A, offa, lda, A_MAT_ERRSET )) {
+    if ((retCode = checkMatrixSizes(kargs->dtype, order, transA, N, K, A, offa, lda, A_MAT_ERRSET))) {
         return retCode;
     }
 
-    if (retCode = checkMatrixSizes(kargs->dtype, order, transA, N, K, B, offb, ldb, B_MAT_ERRSET )) {
+    if ((retCode = checkMatrixSizes(kargs->dtype, order, transA, N, K, B, offb, ldb, B_MAT_ERRSET))) {
         return retCode;
     }
 
-    if (retCode = checkMatrixSizes(kargs->dtype, order, clblasNoTrans, N, N, C, offc, ldc, C_MAT_ERRSET )) {
+    if ((retCode = checkMatrixSizes(kargs->dtype, order, clblasNoTrans, N, N, C, offc, ldc, C_MAT_ERRSET))) {
         return retCode;
     }
 
diff --git a/src/library/blas/xherk.c b/src/library/blas/xherk.c
index 18d1fb4..b4f409d 100644
--- a/src/library/blas/xherk.c
+++ b/src/library/blas/xherk.c
@@ -64,7 +64,7 @@ doHerk(
     }
 
     // Validate arguments
-    if (retCode = checkMemObjects(A, C, NULL, false, A_MAT_ERRSET, C_MAT_ERRSET, END_ERRSET )) {
+    if ((retCode = checkMemObjects(A, C, NULL, false, A_MAT_ERRSET, C_MAT_ERRSET, END_ERRSET))) {
         return retCode;
     }
 
@@ -72,11 +72,11 @@ doHerk(
         return clblasInvalidValue;
     }
 
-    if (retCode = checkMatrixSizes(kargs->dtype, order, transA, N, K, A, offA, lda, A_MAT_ERRSET )) {
+    if ((retCode = checkMatrixSizes(kargs->dtype, order, transA, N, K, A, offA, lda, A_MAT_ERRSET))) {
         return retCode;
     }
 
-    if (retCode = checkMatrixSizes(kargs->dtype, order, false, N, N, C, offC, ldc, C_MAT_ERRSET )) {
+    if ((retCode = checkMatrixSizes(kargs->dtype, order, false, N, N, C, offC, ldc, C_MAT_ERRSET))) {
         return retCode;
     }
 
diff --git a/src/library/blas/xhpmv.c b/src/library/blas/xhpmv.c
index 991819c..1f0fe67 100644
--- a/src/library/blas/xhpmv.c
+++ b/src/library/blas/xhpmv.c
@@ -53,17 +53,17 @@ doHpmv(
 
     /* Validate arguments */
 
-    if (retCode = checkMemObjects(AP, X, Y, true, A_MAT_ERRSET, X_VEC_ERRSET, Y_VEC_ERRSET)) {
+    if ((retCode = checkMemObjects(AP, X, Y, true, A_MAT_ERRSET, X_VEC_ERRSET, Y_VEC_ERRSET))) {
         return retCode;
     }
-    if (retCode = checkMatrixSizes(kargs->dtype, order, clblasNoTrans, N, N, AP,
-                         offa, 0, A_MAT_ERRSET )) {
+    if ((retCode = checkMatrixSizes(kargs->dtype, order, clblasNoTrans, N, N,
+                                    AP, offa, 0, A_MAT_ERRSET))) {
         return retCode;
     }
-    if (retCode = checkVectorSizes(kargs->dtype, N, X, offx, incx, X_VEC_ERRSET )) {
+    if ((retCode = checkVectorSizes(kargs->dtype, N, X, offx, incx, X_VEC_ERRSET))) {
         return retCode;
     }
-    if (retCode = checkVectorSizes(kargs->dtype, N, Y, offy, incy, Y_VEC_ERRSET )) {
+    if ((retCode = checkVectorSizes(kargs->dtype, N, Y, offy, incy, Y_VEC_ERRSET))) {
         return retCode;
     }
 	if ((commandQueues == NULL) || (numCommandQueues == 0))
@@ -182,4 +182,4 @@ clblasZhpmv(
     return doHpmv(&kargs, order, uplo, N, AP, offa, X, offx, incx,
                   Y, offy, incy, numCommandQueues, commandQueues,
                   numEventsInWaitList, eventWaitList, events);
-}
\ No newline at end of file
+}
diff --git a/src/library/blas/xrot.c b/src/library/blas/xrot.c
index 7fd981b..d07ec87 100644
--- a/src/library/blas/xrot.c
+++ b/src/library/blas/xrot.c
@@ -58,11 +58,11 @@ doRot(
 
 		// Check wheather enough memory was allocated
 
-		if (retCode = checkVectorSizes(kargs->dtype, N, X, offx, incx, X_VEC_ERRSET )) {
+		if ((retCode = checkVectorSizes(kargs->dtype, N, X, offx, incx, X_VEC_ERRSET))) {
 			printf("Invalid Size for X\n");
             return retCode;
 		}
-		if (retCode = checkVectorSizes(kargs->dtype, N, Y, offy, incy, Y_VEC_ERRSET )) {
+		if ((retCode = checkVectorSizes(kargs->dtype, N, Y, offy, incy, Y_VEC_ERRSET))) {
 			printf("Invalid Size for Y\n");
             return retCode;
 		}
diff --git a/src/library/blas/xrotg.c b/src/library/blas/xrotg.c
index b7e5a0f..e497148 100644
--- a/src/library/blas/xrotg.c
+++ b/src/library/blas/xrotg.c
@@ -69,21 +69,21 @@ doRotg(
 
 		// Check wheather enough memory was allocated
 
-		if (retCode = checkVectorSizes(kargs->dtype, 1, A, offA, 1, X_VEC_ERRSET )) {
+		if ((retCode = checkVectorSizes(kargs->dtype, 1, A, offA, 1, X_VEC_ERRSET))) {
 			printf("Invalid Size for A\n");
             return retCode;
 		}
-		if (retCode = checkVectorSizes(kargs->dtype, 1, B, offB, 1, Y_VEC_ERRSET )) {
+		if ((retCode = checkVectorSizes(kargs->dtype, 1, B, offB, 1, Y_VEC_ERRSET))) {
 			printf("Invalid Size for B\n");
             return retCode;
 		}
 
-		if (retCode = checkVectorSizes(cType, 1, C, offC, 1, X_VEC_ERRSET )) {
+		if ((retCode = checkVectorSizes(cType, 1, C, offC, 1, X_VEC_ERRSET))) {
 			printf("Invalid Size for C\n");
             return retCode;
 		}
 
-		if (retCode = checkVectorSizes(kargs->dtype, 1, S, offS, 1, Y_VEC_ERRSET )) {
+		if ((retCode = checkVectorSizes(kargs->dtype, 1, S, offS, 1, Y_VEC_ERRSET))) {
 			printf("Invalid Size for S\n");
             return retCode;
 		}
diff --git a/src/library/blas/xrotm.c b/src/library/blas/xrotm.c
index fcdfcb0..4130cf5 100644
--- a/src/library/blas/xrotm.c
+++ b/src/library/blas/xrotm.c
@@ -60,15 +60,15 @@ doRotm(
 
 		// Check wheather enough memory was allocated
 
-		if (retCode = checkVectorSizes(kargs->dtype, N, X, offx, incx, X_VEC_ERRSET )) {
+		if ((retCode = checkVectorSizes(kargs->dtype, N, X, offx, incx, X_VEC_ERRSET))) {
 			printf("Invalid Size for X\n");
             return retCode;
 		}
-		if (retCode = checkVectorSizes(kargs->dtype, N, Y, offy, incy, Y_VEC_ERRSET )) {
+		if ((retCode = checkVectorSizes(kargs->dtype, N, Y, offy, incy, Y_VEC_ERRSET))) {
 			printf("Invalid Size for Y\n");
             return retCode;
 		}
-		if (retCode = checkVectorSizes(kargs->dtype, 5, param, offParam, 1, Y_VEC_ERRSET )) {
+		if ((retCode = checkVectorSizes(kargs->dtype, 5, param, offParam, 1, Y_VEC_ERRSET))) {
 			printf("Invalid Size for PARAM\n"); // PARAM is of minimum length 5
             return retCode;
 		}
diff --git a/src/library/blas/xrotmg.c b/src/library/blas/xrotmg.c
index 6598229..e6e48b6 100644
--- a/src/library/blas/xrotmg.c
+++ b/src/library/blas/xrotmg.c
@@ -65,23 +65,23 @@ doRotmg(
 
 		// Check wheather enough memory was allocated
 
-		if (retCode = checkVectorSizes(kargs->dtype, 1, D1, offD1, 1, X_VEC_ERRSET )) {
+		if ((retCode = checkVectorSizes(kargs->dtype, 1, D1, offD1, 1, X_VEC_ERRSET))) {
 			printf("Invalid Size for D1\n");
             return retCode;
 		}
-		if (retCode = checkVectorSizes(kargs->dtype, 1, D2, offD2, 1, Y_VEC_ERRSET )) {
+		if ((retCode = checkVectorSizes(kargs->dtype, 1, D2, offD2, 1, Y_VEC_ERRSET))) {
 			printf("Invalid Size for D2\n");
             return retCode;
 		}
-		if (retCode = checkVectorSizes(kargs->dtype, 1, X1, offX1, 1, X_VEC_ERRSET )) {
+		if ((retCode = checkVectorSizes(kargs->dtype, 1, X1, offX1, 1, X_VEC_ERRSET))) {
 			printf("Invalid Size for X1\n");
             return retCode;
 		}
-		if (retCode = checkVectorSizes(kargs->dtype, 1, Y1, offY1, 1, Y_VEC_ERRSET )) {
+		if ((retCode = checkVectorSizes(kargs->dtype, 1, Y1, offY1, 1, Y_VEC_ERRSET))) {
 			printf("Invalid Size for Y1\n");
             return retCode;
 		}
-		if (retCode = checkVectorSizes(kargs->dtype, 1, param, offParam, 1, Y_VEC_ERRSET )) {
+		if ((retCode = checkVectorSizes(kargs->dtype, 1, param, offParam, 1, Y_VEC_ERRSET))) {
 			printf("Invalid Size for PARAM\n");
             return retCode;
 		}
diff --git a/src/library/blas/xscal.c b/src/library/blas/xscal.c
index 6722383..b262031 100644
--- a/src/library/blas/xscal.c
+++ b/src/library/blas/xscal.c
@@ -57,7 +57,7 @@ doScal(
 
 		// Check wheather enough memory was allocated
 
-		if (retCode = checkVectorSizes(kargs->dtype, N, X, offx, incx, X_VEC_ERRSET )) {
+		if ((retCode = checkVectorSizes(kargs->dtype, N, X, offx, incx, X_VEC_ERRSET))) {
 			printf("Invalid Size for X\n");
             return retCode;
 		}
diff --git a/src/library/blas/xshbmv.c b/src/library/blas/xshbmv.c
index e0a5087..94f733d 100644
--- a/src/library/blas/xshbmv.c
+++ b/src/library/blas/xshbmv.c
@@ -68,19 +68,19 @@ doSHbmv(
     }
     /* Validate arguments */
 
-    if (retCode = checkMemObjects(A, x, y, true, A_MAT_ERRSET, X_VEC_ERRSET, Y_VEC_ERRSET ))
+    if ((retCode = checkMemObjects(A, x, y, true, A_MAT_ERRSET, X_VEC_ERRSET, Y_VEC_ERRSET)))
     {
         return retCode;
     }
 
-    if (retCode = checkBandedMatrixSizes(kargs->dtype, order, clblasNoTrans,
-                                            N, N, K, 0, A, offa, lda, A_MAT_ERRSET )) {
+    if ((retCode = checkBandedMatrixSizes(kargs->dtype, order, clblasNoTrans,
+                                          N, N, K, 0, A, offa, lda, A_MAT_ERRSET))) {
         return retCode;
     }
-    if (retCode = checkVectorSizes(kargs->dtype, N, x, offx, incx, X_VEC_ERRSET )) {
+    if ((retCode = checkVectorSizes(kargs->dtype, N, x, offx, incx, X_VEC_ERRSET))) {
         return retCode;
     }
-    if (retCode = checkVectorSizes(kargs->dtype, N, y, offy, incy, Y_VEC_ERRSET )) {
+    if ((retCode = checkVectorSizes(kargs->dtype, N, y, offy, incy, Y_VEC_ERRSET))) {
         return retCode;
     }
 
diff --git a/src/library/blas/xspmv.c b/src/library/blas/xspmv.c
index d522ba8..b40e026 100644
--- a/src/library/blas/xspmv.c
+++ b/src/library/blas/xspmv.c
@@ -53,17 +53,17 @@ doSpmv(
 
     /* Validate arguments */
 
-    if (retCode = checkMemObjects(AP, X, Y, true, A_MAT_ERRSET, X_VEC_ERRSET, Y_VEC_ERRSET )) {
+    if ((retCode = checkMemObjects(AP, X, Y, true, A_MAT_ERRSET, X_VEC_ERRSET, Y_VEC_ERRSET))) {
         return retCode;
     }
-    if (retCode = checkMatrixSizes(kargs->dtype, order, clblasNoTrans, N, N, AP,
-                         offa, 0, A_MAT_ERRSET )) {
+    if ((retCode = checkMatrixSizes(kargs->dtype, order, clblasNoTrans, N, N,
+                                    AP, offa, 0, A_MAT_ERRSET))) {
         return retCode;
     }
-    if (retCode = checkVectorSizes(kargs->dtype, N, X, offx, incx, X_VEC_ERRSET )) {
+    if ((retCode = checkVectorSizes(kargs->dtype, N, X, offx, incx, X_VEC_ERRSET))) {
         return retCode;
     }
-    if (retCode = checkVectorSizes(kargs->dtype, N, Y, offy, incy, Y_VEC_ERRSET )) {
+    if ((retCode = checkVectorSizes(kargs->dtype, N, Y, offy, incy, Y_VEC_ERRSET))) {
         return retCode;
     }
 	if ((commandQueues == NULL) || (numCommandQueues == 0))
@@ -184,4 +184,4 @@ clblasDspmv(
     return doSpmv(&kargs, order, uplo, N, AP, offa, X, offx, incx,
                   Y, offy, incy, numCommandQueues, commandQueues,
                   numEventsInWaitList, eventWaitList, events);
-}
\ No newline at end of file
+}
diff --git a/src/library/blas/xswap.c b/src/library/blas/xswap.c
index 3806618..1d83a5b 100644
--- a/src/library/blas/xswap.c
+++ b/src/library/blas/xswap.c
@@ -60,11 +60,11 @@ doSwap(
 
 		// Check wheather enough memory was allocated
 
-		if (retCode = checkVectorSizes(kargs->dtype, N, X, offx, incx, X_VEC_ERRSET )) {
+		if ((retCode = checkVectorSizes(kargs->dtype, N, X, offx, incx, X_VEC_ERRSET))) {
 			printf("Invalid Size for X\n");
             return retCode;
 		}
-		if (retCode = checkVectorSizes(kargs->dtype, N, Y, offy, incy, Y_VEC_ERRSET )) {
+		if ((retCode = checkVectorSizes(kargs->dtype, N, Y, offy, incy, Y_VEC_ERRSET))) {
 			printf("Invalid Size for Y\n");
             return retCode;
 		}
diff --git a/src/library/blas/xsymm.c b/src/library/blas/xsymm.c
index e61a33f..5c87fc6 100644
--- a/src/library/blas/xsymm.c
+++ b/src/library/blas/xsymm.c
@@ -50,31 +50,31 @@ doSymm(	CLBlasKargs *kargs, clblasOrder order, clblasUplo uplo, clblasSide side,
 
     /* Validate arguments */
 
-    if (retCode = checkMemObjects(A, B, C, true, A_MAT_ERRSET, B_MAT_ERRSET, C_MAT_ERRSET )) {
+    if ((retCode = checkMemObjects(A, B, C, true, A_MAT_ERRSET, B_MAT_ERRSET, C_MAT_ERRSET))) {
 		printf("SYMM:- Invalid mem object..\n");
         return retCode;
     }
 
 
-    if (retCode = checkMatrixSizes(kargs->dtype, order, clblasNoTrans, M, N, B, offb, ldb, B_MAT_ERRSET )) {
+    if ((retCode = checkMatrixSizes(kargs->dtype, order, clblasNoTrans, M, N, B, offb, ldb, B_MAT_ERRSET))) {
 		printf("Invalid Size for B\n");
         return retCode;
     }
 
-    if (retCode = checkMatrixSizes(kargs->dtype, order, clblasNoTrans, M, N, C, offc, ldc, C_MAT_ERRSET )) {
+    if ((retCode = checkMatrixSizes(kargs->dtype, order, clblasNoTrans, M, N, C, offc, ldc, C_MAT_ERRSET))) {
 		printf("Invalid Size for C\n");
         return retCode;
     }
 	if (side == clblasLeft)
 	{
 		// MxM x MxN
-    	if (retCode = checkMatrixSizes(kargs->dtype, order, clblasNoTrans, M, M, A, offa, lda, A_MAT_ERRSET )) {
+    	if ((retCode = checkMatrixSizes(kargs->dtype, order, clblasNoTrans, M, M, A, offa, lda, A_MAT_ERRSET))) {
 			printf("Invalid Size for A\n");
             return retCode;
     	}
 	} else {
 		// MxN x NxN
-    	if (retCode = checkMatrixSizes(kargs->dtype, order, clblasNoTrans, N, N, A, offa, lda, A_MAT_ERRSET )) {
+    	if ((retCode = checkMatrixSizes(kargs->dtype, order, clblasNoTrans, N, N, A, offa, lda, A_MAT_ERRSET))) {
 			printf("Invalid Size for A\n");
             return retCode;
     	}
diff --git a/src/library/blas/xsymv.c b/src/library/blas/xsymv.c
index 55b23e8..790e872 100644
--- a/src/library/blas/xsymv.c
+++ b/src/library/blas/xsymv.c
@@ -60,17 +60,17 @@ doSymv(
 
     /* Validate arguments */
 
-    if (retCode = checkMemObjects(A, x, y, true, A_MAT_ERRSET, X_VEC_ERRSET, Y_VEC_ERRSET)) {
+    if ((retCode = checkMemObjects(A, x, y, true, A_MAT_ERRSET, X_VEC_ERRSET, Y_VEC_ERRSET))) {
         return retCode;
     }
-    if (retCode = checkMatrixSizes(kargs->dtype, order, clblasNoTrans, N, N, A,
-                         offA, lda, A_MAT_ERRSET )) {
+    if ((retCode = checkMatrixSizes(kargs->dtype, order, clblasNoTrans, N, N,
+                                    A, offA, lda, A_MAT_ERRSET ))) {
         return retCode;
     }
-    if (retCode = checkVectorSizes(kargs->dtype, N, x, offx, incx, X_VEC_ERRSET )) {
+    if ((retCode = checkVectorSizes(kargs->dtype, N, x, offx, incx, X_VEC_ERRSET ))) {
         return retCode;
     }
-    if (retCode = checkVectorSizes(kargs->dtype, N, y, offy, incy, Y_VEC_ERRSET )) {
+    if ((retCode = checkVectorSizes(kargs->dtype, N, y, offy, incy, Y_VEC_ERRSET ))) {
         return retCode;
     }
 
diff --git a/src/library/blas/xsyr.c b/src/library/blas/xsyr.c
index d2d1ae7..9358920 100644
--- a/src/library/blas/xsyr.c
+++ b/src/library/blas/xsyr.c
@@ -55,7 +55,7 @@ doSyr(
 
     /* Validate arguments */
 
-    if (retCode = checkMemObjects(A, X, 0, false, A_MAT_ERRSET, X_VEC_ERRSET, END_ERRSET )) {
+    if ((retCode = checkMemObjects(A, X, 0, false, A_MAT_ERRSET, X_VEC_ERRSET, END_ERRSET))) {
    		printf("Invalid mem object..\n");
         return retCode;
     }
@@ -65,11 +65,11 @@ doSyr(
      * checkMatrixSizes() does not account of "offa" argument.
      * Need to be added.
      */
-    if (retCode = checkMatrixSizes(kargs->dtype, order, clblasNoTrans, N, N, A, offa, lda, A_MAT_ERRSET )) {
+    if ((retCode = checkMatrixSizes(kargs->dtype, order, clblasNoTrans, N, N, A, offa, lda, A_MAT_ERRSET))) {
         printf("Invalid Size for A\n");
         return retCode;
     }
-    if (retCode = checkVectorSizes(kargs->dtype, N, X, offx, incx, X_VEC_ERRSET )) {
+    if ((retCode = checkVectorSizes(kargs->dtype, N, X, offx, incx, X_VEC_ERRSET))) {
         printf("Invalid Size for X\n");
         return retCode;
     }
diff --git a/src/library/blas/xsyr2.c b/src/library/blas/xsyr2.c
index 2f0a185..fddcfbd 100644
--- a/src/library/blas/xsyr2.c
+++ b/src/library/blas/xsyr2.c
@@ -58,21 +58,21 @@ doSyr2(
 
     /* Validate arguments */
 
-    if (retCode = checkMemObjects(A, X, Y, true, A_MAT_ERRSET, X_VEC_ERRSET, Y_VEC_ERRSET )) {
+    if ((retCode = checkMemObjects(A, X, Y, true, A_MAT_ERRSET, X_VEC_ERRSET, Y_VEC_ERRSET))) {
         printf("Invalid mem object..\n");
         return retCode;
     }
 
-    if (retCode = checkMatrixSizes(kargs->dtype, order, clblasNoTrans, N, N, A, offa, lda, A_MAT_ERRSET )) {
+    if ((retCode = checkMatrixSizes(kargs->dtype, order, clblasNoTrans, N, N, A, offa, lda, A_MAT_ERRSET ))) {
         printf("Invalid Size for A\n");
         return retCode;
     }
-    if (retCode = checkVectorSizes(kargs->dtype, N, X, offx, incx, X_VEC_ERRSET )) {
+    if ((retCode = checkVectorSizes(kargs->dtype, N, X, offx, incx, X_VEC_ERRSET))) {
         printf("Invalid Size for X\n");
         return retCode;
     }
 
-	if (retCode = checkVectorSizes(kargs->dtype, N, Y, offy, incy, Y_VEC_ERRSET )) {
+	if ((retCode = checkVectorSizes(kargs->dtype, N, Y, offy, incy, Y_VEC_ERRSET))) {
         printf("Invalid Size for Y\n");
         return retCode;
     }
diff --git a/src/library/blas/xsyr2k.c b/src/library/blas/xsyr2k.c
index e99a617..25ed438 100644
--- a/src/library/blas/xsyr2k.c
+++ b/src/library/blas/xsyr2k.c
@@ -58,7 +58,7 @@ doSyr2k(
     }
 
     // Validate arguments
-    if (retCode = checkMemObjects(A, B, C, true, A_MAT_ERRSET, B_MAT_ERRSET, C_MAT_ERRSET )) {
+    if ((retCode = checkMemObjects(A, B, C, true, A_MAT_ERRSET, B_MAT_ERRSET, C_MAT_ERRSET))) {
         return retCode;
     }
 
@@ -66,13 +66,13 @@ doSyr2k(
         return clblasInvalidValue;
     }
 
-    if (retCode = checkMatrixSizes(kargs->dtype, order, transAB, N, K, A, offA, lda, A_MAT_ERRSET )) {
+    if ((retCode = checkMatrixSizes(kargs->dtype, order, transAB, N, K, A, offA, lda, A_MAT_ERRSET))) {
         return retCode;
     }
-    if (retCode = checkMatrixSizes(kargs->dtype, order, transAB, N, K, B, offB, ldb, B_MAT_ERRSET )) {
+    if ((retCode = checkMatrixSizes(kargs->dtype, order, transAB, N, K, B, offB, ldb, B_MAT_ERRSET))) {
         return retCode;
     }
-    if (retCode = checkMatrixSizes(kargs->dtype, order, false, N, N, C, offC, ldc, C_MAT_ERRSET )) {
+    if ((retCode = checkMatrixSizes(kargs->dtype, order, false, N, N, C, offC, ldc, C_MAT_ERRSET))) {
         return retCode;
     }
 
diff --git a/src/library/blas/xsyrk.c b/src/library/blas/xsyrk.c
index 4157d5e..2582830 100644
--- a/src/library/blas/xsyrk.c
+++ b/src/library/blas/xsyrk.c
@@ -55,7 +55,7 @@ doSyrk(
     }
 
     // Validate arguments
-    if (retCode = checkMemObjects(A, C, NULL, false, A_MAT_ERRSET, C_MAT_ERRSET, END_ERRSET)) {
+    if ((retCode = checkMemObjects(A, C, NULL, false, A_MAT_ERRSET, C_MAT_ERRSET, END_ERRSET))) {
         return retCode;
     }
 
@@ -63,10 +63,10 @@ doSyrk(
         return clblasInvalidValue;
     }
 
-    if (retCode = checkMatrixSizes(kargs->dtype, order, transA, N, K, A, offA, lda, A_MAT_ERRSET )) {
+    if ((retCode = checkMatrixSizes(kargs->dtype, order, transA, N, K, A, offA, lda, A_MAT_ERRSET))) {
         return retCode;
     }
-    if (retCode = checkMatrixSizes(kargs->dtype, order, false, N, N, C, offC, ldc, C_MAT_ERRSET )) {
+    if ((retCode = checkMatrixSizes(kargs->dtype, order, false, N, N, C, offC, ldc, C_MAT_ERRSET))) {
         return retCode;
     }
 
diff --git a/src/library/blas/xtbmv.c b/src/library/blas/xtbmv.c
index 8f59bc9..b3b0d3b 100644
--- a/src/library/blas/xtbmv.c
+++ b/src/library/blas/xtbmv.c
@@ -59,20 +59,20 @@ doTbmv(
 
     /* Validate arguments */
 
-    if (retCode = checkMemObjects(A, x, y, true, A_MAT_ERRSET, X_VEC_ERRSET, Y_VEC_ERRSET )) {
+    if ((retCode = checkMemObjects(A, x, y, true, A_MAT_ERRSET, X_VEC_ERRSET, Y_VEC_ERRSET))) {
 	printf("Invalid mem object..\n");
         return retCode;
     }
 
-    if (retCode = checkBandedMatrixSizes(kargs->dtype, order, trans, N, N, K, 0, A, offa, lda, A_MAT_ERRSET)) {
+    if ((retCode = checkBandedMatrixSizes(kargs->dtype, order, trans, N, N, K, 0, A, offa, lda, A_MAT_ERRSET))) {
 		printf("Invalid Size for A\n");
         return retCode;
     }
-    if (retCode = checkVectorSizes(kargs->dtype, N, x, offx, incx, X_VEC_ERRSET)) {
+    if ((retCode = checkVectorSizes(kargs->dtype, N, x, offx, incx, X_VEC_ERRSET))) {
 		printf("Invalid Size for X\n");
         return retCode;
     }
-    if (retCode = checkVectorSizes(kargs->dtype, N, y, 0, incx, Y_VEC_ERRSET)) {
+    if ((retCode = checkVectorSizes(kargs->dtype, N, y, 0, incx, Y_VEC_ERRSET))) {
 		printf("Invalid Size for scratch vector\n");
         return retCode;
     }
diff --git a/src/library/blas/xtrmm.c b/src/library/blas/xtrmm.c
index b7611da..8aff207 100644
--- a/src/library/blas/xtrmm.c
+++ b/src/library/blas/xtrmm.c
@@ -55,16 +55,16 @@ doTrmm(
 
     /* Validate arguments */
 
-    if (retCode = checkMemObjects(A, B, NULL, false, A_MAT_ERRSET, B_MAT_ERRSET, END_ERRSET )) {
+    if ((retCode = checkMemObjects(A, B, NULL, false, A_MAT_ERRSET, B_MAT_ERRSET, END_ERRSET))) {
         return retCode;
     }
     msize = (side == clblasLeft) ? M : N;
-    if (retCode = checkMatrixSizes(kargs->dtype, order, transA, msize, msize, A,
-                         offA, lda, A_MAT_ERRSET )) {
+    if ((retCode = checkMatrixSizes(kargs->dtype, order, transA, msize, msize,
+                                    A, offA, lda, A_MAT_ERRSET ))) {
         return retCode;
     }
-    if (retCode = checkMatrixSizes(kargs->dtype, order, clblasNoTrans, M, N, B,
-                         offB, ldb, B_MAT_ERRSET )) {
+    if ((retCode = checkMatrixSizes(kargs->dtype, order, clblasNoTrans, M, N,
+                                    B, offB, ldb, B_MAT_ERRSET ))) {
         return retCode;
     }
 
diff --git a/src/library/blas/xtrmv.c b/src/library/blas/xtrmv.c
index 2f4e216..145c799 100644
--- a/src/library/blas/xtrmv.c
+++ b/src/library/blas/xtrmv.c
@@ -57,20 +57,20 @@ doTrmv(
 
     /* Validate arguments */
 
-    if (retCode = checkMemObjects(A, x, y, true, A_MAT_ERRSET, X_VEC_ERRSET, Y_VEC_ERRSET)) {
+    if ((retCode = checkMemObjects(A, x, y, true, A_MAT_ERRSET, X_VEC_ERRSET, Y_VEC_ERRSET))) {
 	printf("Invalid mem object..\n");
         return retCode;
     }
 
-    if (retCode = checkMatrixSizes(kargs->dtype, order, trans, N, N, A, offa, lda, A_MAT_ERRSET )) {
+    if ((retCode = checkMatrixSizes(kargs->dtype, order, trans, N, N, A, offa, lda, A_MAT_ERRSET))) {
 		printf("Invalid Size for A\n");
         return retCode;
     }
-    if (retCode = checkVectorSizes(kargs->dtype, N, x, offx, incx, X_VEC_ERRSET )) {
+    if ((retCode = checkVectorSizes(kargs->dtype, N, x, offx, incx, X_VEC_ERRSET))) {
 		printf("Invalid Size for X\n");
         return retCode;
     }
-    if (retCode = checkVectorSizes(kargs->dtype, N, y, 0, incx, Y_VEC_ERRSET )) {
+    if ((retCode = checkVectorSizes(kargs->dtype, N, y, 0, incx, Y_VEC_ERRSET))) {
 		printf("Invalid Size for scratch vector\n");
         return retCode;
     }
diff --git a/src/library/blas/xtrsm.c b/src/library/blas/xtrsm.c
index 9fb5b4a..d2fd7f0 100644
--- a/src/library/blas/xtrsm.c
+++ b/src/library/blas/xtrsm.c
@@ -55,17 +55,17 @@ doTrsm(
 
     /* Validate arguments */
 
-    if (retCode = checkMemObjects(A, B, NULL, false, A_MAT_ERRSET, B_MAT_ERRSET, END_ERRSET )) {
+    if ((retCode = checkMemObjects(A, B, NULL, false, A_MAT_ERRSET, B_MAT_ERRSET, END_ERRSET ))) {
         return retCode;
     }
     msize = (side == clblasLeft) ? M : N;
 
-    if (retCode = checkMatrixSizes(kargs->dtype, order, transA, msize, msize, A,
-                         offA, lda, A_MAT_ERRSET )) {
+    if ((retCode = checkMatrixSizes(kargs->dtype, order, transA, msize, msize,
+                                    A, offA, lda, A_MAT_ERRSET ))) {
         return retCode;
     }
-    if (retCode = checkMatrixSizes(kargs->dtype, order, clblasNoTrans, M, N, B,
-                         offB, ldb, B_MAT_ERRSET )) {
+    if ((retCode = checkMatrixSizes(kargs->dtype, order, clblasNoTrans, M, N,
+                                    B, offB, ldb, B_MAT_ERRSET ))) {
         return retCode;
     }
 
diff --git a/src/library/blas/xtrsv.c b/src/library/blas/xtrsv.c
index c334228..1e48349 100644
--- a/src/library/blas/xtrsv.c
+++ b/src/library/blas/xtrsv.c
@@ -351,7 +351,7 @@ doTrsv(
 
     /* Validate arguments */
 
-    if (retCode = checkMemObjects(A, x, (cl_mem) NULL, false, A_MAT_ERRSET, X_VEC_ERRSET, END_ERRSET)) {
+    if ((retCode = checkMemObjects(A, x, (cl_mem) NULL, false, A_MAT_ERRSET, X_VEC_ERRSET, END_ERRSET))) {
 		#ifdef DEBUG_TRSV
 		printf("Invalid mem object..\n");
 		#endif
@@ -363,13 +363,13 @@ doTrsv(
  	 * checkMatrixSizes() does not account for "offa" argument.
  	 * Need to pass "offa" when "checkMatrixSizes()" is changed.
 	 */
-    if (retCode = checkMatrixSizes(kargs->dtype, order, trans, N, N, A, offa, lda, A_MAT_ERRSET)) {
+    if ((retCode = checkMatrixSizes(kargs->dtype, order, trans, N, N, A, offa, lda, A_MAT_ERRSET))) {
 		#ifdef DEBUG_TRSV
 		printf("Invalid Size for A\n");
 		#endif
         return retCode;
     }
-    if (retCode = checkVectorSizes(kargs->dtype, N, x, offx, incx, X_VEC_ERRSET )) {
+    if ((retCode = checkVectorSizes(kargs->dtype, N, x, offx, incx, X_VEC_ERRSET))) {
 		#ifdef DEBUG_TRSV
 		printf("Invalid Size for X\n");
 		#endif
diff --git a/src/library/tools/tplgen/tplgen.cpp b/src/library/tools/tplgen/tplgen.cpp
index 25150aa..e81ecd2 100644
--- a/src/library/tools/tplgen/tplgen.cpp
+++ b/src/library/tools/tplgen/tplgen.cpp
@@ -73,7 +73,7 @@ int main( int argc, char *argv[] )
     size_t found;
     string str;
     int startOptions = 1;
-    char *outputPrefix = "";
+    const char *outputPrefix = "";
 
     std::cout << "TPLGEN Running.....\n";
     if (argc < 2)
diff --git a/src/library/tools/tune/subdim.c b/src/library/tools/tune/subdim.c
index 37ead33..6eed76f 100644
--- a/src/library/tools/tune/subdim.c
+++ b/src/library/tools/tune/subdim.c
@@ -364,7 +364,7 @@ nextSubdimElem(SubDimInfo* sd)
 
     // !!! DEBUG
     if (sd->count > 500) {
-        *(int*)0 = 0;
+        abort();
     }
 
     sd->count ++;

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/debian-science/packages/clblas.git



More information about the debian-science-commits mailing list