[PATCH:xcmsdb 5/6] Remove break statements immediately after return statements

Alan Coopersmith alan.coopersmith at oracle.com
Tue Nov 26 21:45:17 PST 2013


Silences clang -Wunreachable-code warnings

Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>
---
 loadData.c |    7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/loadData.c b/loadData.c
index 170d9e0..defcb40 100644
--- a/loadData.c
+++ b/loadData.c
@@ -485,7 +485,6 @@ ProcessColorimetric(FILE *stream, XDCCC_Matrix *pMatrix, int VisualFlag)
 			"Line %d: Extraneous keyword %s.\n", 
 			linenum, keyword);
 		return (0);
-/* NOTREACHED */break;	
 	      case COLORIMETRIC_END :
 		if (state != 2) {
 		    fprintf(stderr,
@@ -502,13 +501,11 @@ ProcessColorimetric(FILE *stream, XDCCC_Matrix *pMatrix, int VisualFlag)
 			"Line %d: Unexpected keyword %s\n",
 			linenum, keyword);
 		return (0);
-/* NOTREACHED */break;		
 	    }
 	} else if (ntok < 0) {
 	    /* mismatch */
 	    fprintf(stderr, "Line %d: Unrecognized keyword\n", linenum);
 	    return (0);
-/* NOTREACHED */break;		
 	}
     }
     return (0);
@@ -692,7 +689,6 @@ ProcessIProfile(FILE *stream, XDCCC_Correction *pCorrection)
 		fprintf(stderr,"Line %d: extraneous keyword %s\n", 
 			  linenum, keyword);
 		return (0);
-/* NOTREACHED */break;
 	      case IPROFILE_END :
 		if ((state != 0) || (nTbl != pCorrection->nTables)) {
 		    fprintf(stderr,
@@ -708,7 +704,6 @@ ProcessIProfile(FILE *stream, XDCCC_Correction *pCorrection)
 		fprintf(stderr,"Line %d: unexpected keyword %s\n",
 		      linenum, keyword);
 		return (0);
-/* NOTREACHED */break;
 	    }
 	} /* else its was just a blank line */
     }
@@ -1721,7 +1716,7 @@ IProfileProcessed:
 		}
 		closeS (stream, CorrectionHead);
 		return (1);
-/* NOTREACHED */    break;
+
 	      default :
 		fprintf(stderr,"Line %d: extraneous keyword %s\n", 
 			linenum, keyword);
-- 
1.7.9.2



More information about the xorg-devel mailing list