Skip to content

Added a default case to ConstructorInjectionPoint #87

Merged
merged 1 commit into from Mar 22, 2013

4 participants

@bysse
bysse commented Mar 21, 2013

I've added a default case to ConstructorInjectionPoint so that it doesn't fail silently if someone should happen to use more than ten arguments to a class.

@creynders

Anybody who uses more than 10 constructor arguments deserves a silent fail ;)

@vlaaad
vlaaad commented on 8dacf72 Mar 21, 2013

why you can't add more cases?

@tschneidereit
Owner

Yup, this makes a lot of sense. One might deserve a fail for using more than 10 args, but a silent fail is just mean. :)

why you can't add more cases?

Why would we? Really, you shouldn't use that many args for your ctors. Ever.

@tschneidereit tschneidereit merged commit 4205e4e into tschneidereit:master Mar 22, 2013
@bysse
bysse commented Mar 22, 2013

Great, thanks!

@vlaaad
vlaaad commented Mar 22, 2013

I don't use that many args, but 10 args limitation looks strange.

@tschneidereit
Owner
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.