Merge pull request #467 from compnerd/warnings

ktopley-apple · web-flow · commit cc04868e8ffe · 2019-04-03T12:53:53.000-07:00
Fix warnings (treated as errors)
diff --git a/cmake/modules/DispatchCompilerWarnings.cmake b/cmake/modules/DispatchCompilerWarnings.cmake
@@ -38,7 +38,7 @@ else()
     add_compile_options(-Wsign-conversion)
     add_compile_options(-Wstatic-in-inline)
     add_compile_options(-Wsuper-class-method-mismatch)
-    add_compile_options(-Wswitch-enum)
+    add_compile_options(-Wswitch)
     add_compile_options(-Wunguarded-availability)
     add_compile_options(-Wunreachable-code)
     add_compile_options(-Wunused)
diff --git a/src/init.c b/src/init.c
@@ -542,8 +542,6 @@ dispatch_queue_attr_make_with_autorelease_frequency(dispatch_queue_attr_t dqa,
 	case DISPATCH_AUTORELEASE_FREQUENCY_WORK_ITEM:
 	case DISPATCH_AUTORELEASE_FREQUENCY_NEVER:
 		break;
-	default:
-		return (dispatch_queue_attr_t)dqa;
 	}
 	dispatch_queue_attr_info_t dqai = _dispatch_queue_attr_to_info(dqa);
 	dqai.dqai_autorelease_frequency = (uint16_t)frequency;

Original file line number	Diff line number	Diff line change
`@@ -542,8 +542,6 @@ dispatch_queue_attr_make_with_autorelease_frequency(dispatch_queue_attr_t dqa,`
`542`	`542`	`case DISPATCH_AUTORELEASE_FREQUENCY_WORK_ITEM:`
`543`	`543`	`case DISPATCH_AUTORELEASE_FREQUENCY_NEVER:`
`544`	`544`	`break;`
`545`		`- default:`
`546`		`- return (dispatch_queue_attr_t)dqa;`
`547`	`545`	`}`
`548`	`546`	`dispatch_queue_attr_info_t dqai = _dispatch_queue_attr_to_info(dqa);`
`549`	`547`	`dqai.dqai_autorelease_frequency = (uint16_t)frequency;`