Revert "Do not warn about non-default parameters for coroutines."

Breaks parameter inference for async method overrides.

This reverts commit dd24a1bfe1a78bf1a37d35f5d8dd6480fd4a8941.

https://bugzilla.gnome.org/show_bug.cgi?id=792660

diff --git a/vala/valamethod.vala b/vala/valamethod.vala
index b0a742a3c25b24d08209f47e085a2d8e4a9a3cb1..ab85c121ed434ba3c3910bb651886d712f8d29d4 100644 (file)
--- a/vala/valamethod.vala
+++ b/vala/valamethod.vala
@@ -721,20 +721,17 @@ public class Vala.Method : Subroutine, Callable {
                        Report.error (parameters[0].source_reference, "Named parameter required before `...'");
                }
 
-               if (!coroutine) {
-                       // TODO: begin and end parameters must be checked separately for coroutines
-                       var optional_param = false;
-                       foreach (Parameter param in parameters) {
-                               param.check (context);
-                               if (coroutine && param.direction == ParameterDirection.REF) {
-                                       error = true;
-                                       Report.error (param.source_reference, "Reference parameters are not supported for async methods");
-                               }
-                               if (optional_param && param.initializer == null && !param.ellipsis) {
-                                       Report.warning (param.source_reference, "parameter without default follows parameter with default");
-                               } else if (param.initializer != null) {
-                                       optional_param = true;
-                               }
+               var optional_param = false;
+               foreach (Parameter param in parameters) {
+                       param.check (context);
+                       if (coroutine && param.direction == ParameterDirection.REF) {
+                               error = true;
+                               Report.error (param.source_reference, "Reference parameters are not supported for async methods");
+                       }
+                       if (optional_param && param.initializer == null && !param.ellipsis) {
+                               Report.warning (param.source_reference, "parameter without default follows parameter with default");
+                       } else if (param.initializer != null) {
+                               optional_param = true;
                        }
                }