Fix not renaming constructors' arguments

yck1509 · yck1509 · commit 6527f300a388 · 2016-02-12T20:32:01.000+08:00
Fix yck1509#432
diff --git a/Confuser.Renamer/RenamePhase.cs b/Confuser.Renamer/RenamePhase.cs
@@ -37,7 +37,7 @@ protected override void Execute(ConfuserContext context, ProtectionParameters pa
 
 				if (def is MethodDef) {
 					var method = (MethodDef)def;
-					if (canRename && parameters.GetParameter(context, def, "renameArgs", true)) {
+					if ((canRename || method.IsConstructor) && parameters.GetParameter(context, def, "renameArgs", true)) {
 						foreach (ParamDef param in ((MethodDef)def).ParamDefs)
 							param.Name = null;
 					}

Original file line number	Diff line number	Diff line change
`@@ -37,7 +37,7 @@ protected override void Execute(ConfuserContext context, ProtectionParameters pa`
`37`	`37`
`38`	`38`	`if (def is MethodDef) {`
`39`	`39`	`var method = (MethodDef)def;`
`40`		`- if (canRename && parameters.GetParameter(context, def, "renameArgs", true)) {`
	`40`	`+ if ((canRename \|\| method.IsConstructor) && parameters.GetParameter(context, def, "renameArgs", true)) {`
`41`	`41`	`foreach (ParamDef param in ((MethodDef)def).ParamDefs)`
`42`	`42`	`param.Name = null;`
`43`	`43`	`}`