Mercurial > projects > ldc
diff dmd/mtype.c @ 630:bc83491463f0
Nicer error message on some invalid default args.
author | Christian Kamm <kamm incasoftware de> |
---|---|
date | Wed, 01 Oct 2008 21:14:15 +0200 |
parents | 4435f57956e7 |
children | 307ae566885f |
line wrap: on
line diff
--- a/dmd/mtype.c Wed Oct 01 20:32:12 2008 +0200 +++ b/dmd/mtype.c Wed Oct 01 21:14:15 2008 +0200 @@ -2934,6 +2934,14 @@ arg->defaultArg = arg->defaultArg->semantic(sc); arg->defaultArg = resolveProperties(sc, arg->defaultArg); arg->defaultArg = arg->defaultArg->implicitCastTo(sc, arg->type); + + // make sure default arguments only use variables with lower protection + // this check only catches the common case that the default arg Exp is a VarExp + if(arg->defaultArg->op == TOKvar) + { VarExp *ve = (VarExp *)arg->defaultArg; + if(ve->var->protection < sc->protection) + error(loc, "default argument %s has stronger protection than function %s", ve->var->toChars(), toChars()); + } } /* If arg turns out to be a tuple, the number of parameters may