Fix [-Wimplicit-fallthrough] warnings in MBString extension

Girgias · Girgias · commit c73877dc446b · 2021-01-15T04:02:21.000Z
diff --git a/ext/mbstring/libmbfl/filters/mbfilter_sjis_mac.c b/ext/mbstring/libmbfl/filters/mbfilter_sjis_mac.c
@@ -373,6 +373,7 @@ mbfl_filt_conv_wchar_sjis_mac(int c, mbfl_convert_filter *filter)
 			break;
 		}
 		s1 = s2 = 0;
+		fallthrough;
 
 	case 0:
 
diff --git a/ext/mbstring/mbstring.c b/ext/mbstring/mbstring.c
@@ -3357,7 +3357,7 @@ static int _php_mbstr_parse_mail_headers(HashTable *ht, const char *str, size_t
 							state = 3;
 							break;
 						}
-						/* break is missing intentionally */
+						fallthrough;
 
 					case 3:
 						if (crlf_state == -1) {

Original file line number	Diff line number	Diff line change
`@@ -373,6 +373,7 @@ mbfl_filt_conv_wchar_sjis_mac(int c, mbfl_convert_filter *filter)`
`373`	`373`	`break;`
`374`	`374`	`}`
`375`	`375`	`s1 = s2 = 0;`
	`376`	`+ fallthrough;`
`376`	`377`
`377`	`378`	`case 0:`
`378`	`379`
Original file line number	Diff line number	Diff line change
`@@ -3357,7 +3357,7 @@ static int _php_mbstr_parse_mail_headers(HashTable ht, const char str, size_t`
`3357`	`3357`	`state = 3;`
`3358`	`3358`	`break;`
`3359`	`3359`	`}`
`3360`		`- /* break is missing intentionally */`
	`3360`	`+ fallthrough;`
`3361`	`3361`
`3362`	`3362`	`case 3:`
`3363`	`3363`	`if (crlf_state == -1) {`