You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucy.apache.org by nw...@apache.org on 2013/06/03 17:27:49 UTC

[lucy-commits] [2/3] git commit: refs/heads/master - Error out on variadic methods

Error out on variadic methods

Variadic methods don't work at the moment. So bail out in CFC instead of
generating non-working method wrappers.


Project: http://git-wip-us.apache.org/repos/asf/lucy/repo
Commit: http://git-wip-us.apache.org/repos/asf/lucy/commit/fa366442
Tree: http://git-wip-us.apache.org/repos/asf/lucy/tree/fa366442
Diff: http://git-wip-us.apache.org/repos/asf/lucy/diff/fa366442

Branch: refs/heads/master
Commit: fa3664425f4af79f5552883dabe5d40119bf00d2
Parents: 3212e48
Author: Nick Wellnhofer <we...@aevum.de>
Authored: Sat Jun 1 22:46:57 2013 +0200
Committer: Nick Wellnhofer <we...@aevum.de>
Committed: Mon Jun 3 17:23:52 2013 +0200

----------------------------------------------------------------------
 clownfish/compiler/src/CFCBindMethod.c |    3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/lucy/blob/fa366442/clownfish/compiler/src/CFCBindMethod.c
----------------------------------------------------------------------
diff --git a/clownfish/compiler/src/CFCBindMethod.c b/clownfish/compiler/src/CFCBindMethod.c
index 996192a..c2214a1 100644
--- a/clownfish/compiler/src/CFCBindMethod.c
+++ b/clownfish/compiler/src/CFCBindMethod.c
@@ -97,6 +97,9 @@ S_virtual_method_def(CFCMethod *method, CFCClass *klass) {
 
     // Prepare parameter lists, minus invoker.  The invoker gets forced to
     // "self" later.
+    if (CFCParamList_variadic(param_list)) {
+        CFCUtil_die("Variadic methods not supported");
+    }
     const char *arg_names_minus_invoker = CFCParamList_name_list(param_list);
     const char *params_minus_invoker    = CFCParamList_to_c(param_list);
     while (*arg_names_minus_invoker && *arg_names_minus_invoker != ',') {