Better arity checks for overloads

Source/bindings/templates/interface.cpp

 {% extends 'interface_base.cpp' %}
 
 
 {##############################################################################}
 {% macro attribute_configuration(attribute) %}
 {% set getter_callback =
        '%sV8Internal::%sAttributeGetterCallback' %
             (cpp_class, attribute.name)
        if not attribute.constructor_type else
        ('%sV8Internal::%sConstructorGetterCallback' %
@@ skipping 591 matching lines @@
         {# Then resolve by testing argument #}
         {% for test, constructor in tests_constructors %}
         {# 10. If i = d, then: #}
         if ({{test}}) {
             {{cpp_class}}V8Internal::constructor{{constructor.overload_index}}(info);
             return;
         }
         {% endfor %}
         break;
     {% endfor %}
-    }
-    {# No match, throw error #}
-    {% if interface_length %}
-    if (UNLIKELY(info.Length() < {{interface_length}})) {
-        exceptionState.throwTypeError(ExceptionMessages::notEnoughArguments({{interface_length}}, info.Length()));
+    default:
+        {# Invalid arity, throw error #}
+        {# Report full list of valid arities if gaps and above minimum #}
+        {% if constructor_overloads.valid_arities %}
+        if (info.Length() >= {{constructor_overloads.minarg}}) {
+            throwArityTypeError(exceptionState, "{{constructor_overloads.valid_arities}}", info.Length());
+            return;
+        }
+        {% endif %}
+        {# Otherwise just report "not enough arguments" #}
+        exceptionState.throwTypeError(ExceptionMessages::notEnoughArguments({{constructor_overloads.minarg}}, info.Length()));
         exceptionState.throwIfNeeded();
         return;
     }
-    {% endif %}
+    {# No match, throw error #}
     exceptionState.throwTypeError("No matching constructor signature.");
     exceptionState.throwIfNeeded();
 }
 
 {% endif %}
 {% endblock %}
 
 
 {##############################################################################}
 {% block event_constructor %}
@@ skipping 699 matching lines @@
 {% endif %}
 }
 
 template<>
 v8::Handle<v8::Value> toV8NoInline({{cpp_class}}* impl, v8::Handle<v8::Object> creationContext, v8::Isolate* isolate)
 {
     return toV8(impl, creationContext, isolate);
 }
 
 {% endblock %}