Fix [-Wimplicit-fallthrough] warnings in SPL extension

Author: Girgias

ext/spl/spl_array.c

@@ -369,12 +369,14 @@ static zval *spl_array_get_dimension_ptr(int check_inherited, spl_array_object *
 					switch (type) {
 						case BP_VAR_R:
 							zend_error(E_WARNING, "Undefined array key \"%s\"", ZSTR_VAL(key.key));
+							fallthrough;
 						case BP_VAR_UNSET:
 						case BP_VAR_IS:
 							retval = &EG(uninitialized_zval);
 							break;
 						case BP_VAR_RW:
 							zend_error(E_WARNING,"Undefined array key \"%s\"", ZSTR_VAL(key.key));
+							fallthrough;
 						case BP_VAR_W: {
 							ZVAL_NULL(retval);
 						}
@@ -385,12 +387,14 @@ static zval *spl_array_get_dimension_ptr(int check_inherited, spl_array_object *
 			switch (type) {
 				case BP_VAR_R:
 					zend_error(E_WARNING, "Undefined array key \"%s\"", ZSTR_VAL(key.key));
+					fallthrough;
 				case BP_VAR_UNSET:
 				case BP_VAR_IS:
 					retval = &EG(uninitialized_zval);
 					break;
 				case BP_VAR_RW:
 					zend_error(E_WARNING,"Undefined array key \"%s\"", ZSTR_VAL(key.key));
+					fallthrough;
 				case BP_VAR_W: {
 				    zval value;
 					ZVAL_NULL(&value);
@@ -404,12 +408,14 @@ static zval *spl_array_get_dimension_ptr(int check_inherited, spl_array_object *
 			switch (type) {
 				case BP_VAR_R:
 					zend_error(E_WARNING, "Undefined array key " ZEND_LONG_FMT, key.h);
+					fallthrough;
 				case BP_VAR_UNSET:
 				case BP_VAR_IS:
 					retval = &EG(uninitialized_zval);
 					break;
 				case BP_VAR_RW:
 					zend_error(E_WARNING, "Undefined array key " ZEND_LONG_FMT, key.h);
+					fallthrough;
 				case BP_VAR_W: {
 				    zval value;
 					ZVAL_UNDEF(&value);

ext/spl/spl_directory.c

@@ -2345,21 +2345,21 @@ PHP_METHOD(SplFileObject, fgetcsv)
 			} else {
 				escape = (unsigned char) esc[0];
 			}
-			/* no break */
+			fallthrough;
 		case 2:
 			if (e_len != 1) {
 				zend_argument_value_error(2, "must be a single character");
 				RETURN_THROWS();
 			}
 			enclosure = enclo[0];
-			/* no break */
+			fallthrough;
 		case 1:
 			if (d_len != 1) {
 				zend_argument_value_error(1, "must be a single character");
 				RETURN_THROWS();
 			}
 			delimiter = delim[0];
-			/* no break */
+			fallthrough;
 		case 0:
 			break;
 		}
@@ -2395,21 +2395,21 @@ PHP_METHOD(SplFileObject, fputcsv)
 					zend_argument_value_error(4, "must be empty or a single character");
 					RETURN_THROWS();
 			}
-			/* no break */
+			fallthrough;
 		case 3:
 			if (e_len != 1) {
 				zend_argument_value_error(3, "must be a single character");
 				RETURN_THROWS();
 			}
 			enclosure = enclo[0];
-			/* no break */
+			fallthrough;
 		case 2:
 			if (d_len != 1) {
 				zend_argument_value_error(2, "must be a single character");
 				RETURN_THROWS();
 			}
 			delimiter = delim[0];
-			/* no break */
+			fallthrough;
 		case 1:
 		case 0:
 			break;
@@ -2447,21 +2447,21 @@ PHP_METHOD(SplFileObject, setCsvControl)
 					zend_argument_value_error(3, "must be empty or a single character");
 					RETURN_THROWS();
 			}
-			/* no break */
+			fallthrough;
 		case 2:
 			if (e_len != 1) {
 				zend_argument_value_error(2, "must be a single character");
 				RETURN_THROWS();
 			}
 			enclosure = enclo[0];
-			/* no break */
+			fallthrough;
 		case 1:
 			if (d_len != 1) {
 				zend_argument_value_error(1, "must be a single character");
 				RETURN_THROWS();
 			}
 			delimiter = delim[0];
-			/* no break */
+			fallthrough;
 		case 0:
 			break;
 		}

ext/spl/spl_iterators.c

@@ -243,13 +243,15 @@ static void spl_recursive_it_move_forward_ex(spl_recursive_it_object *object, zv
 						zend_clear_exception();
 					}
 				}
-				/* fall through */
+				fallthrough;
 			case RS_START:
 				if (iterator->funcs->valid(iterator) == FAILURE) {
 					break;
 				}
 				object->iterators[object->level].state = RS_TEST;
 				/* break; */
+				/* TODO: Check this is correct */
+				fallthrough;
 			case RS_TEST:
 				ce = object->iterators[object->level].ce;
 				zobject = &object->iterators[object->level].zobject;