StardewValley
/
SMAPI
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix some method references only partially rewritten 

Thanks to Bepis on Discord for helping find the issue!
This commit is contained in:
Jesse Plamondon-Willard 2020-08-25 22:49:00 -04:00
parent b9a9fe36bb
commit abfe40bf69
No known key found for this signature in database
GPG Key ID: CF8B1456B3E29F49
1 changed files with 8 additions and 20 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

28
src/SMAPI/Framework/ModLoading/Rewriters/MethodWithMissingOptionalParameterRewriter.cs
View File

@ -68,29 +68,17 @@ namespace StardewModdingAPI.Framework.ModLoading.Rewriters
if (method == null)
return false;
// get instructions to inject
var injectables = method.Parameters.Skip(methodRef.Parameters.Count)
.Select(p => new { Parameter = p, LoadValueInstruction = this.GetLoadValueInstruction(p.Constant) })
// get instructions to inject parameter values
var loadInstructions = method.Parameters.Skip(methodRef.Parameters.Count)
.Select(p => this.GetLoadValueInstruction(p.Constant))
.ToArray();
if (injectables.Any(p => p.LoadValueInstruction == null))
if (loadInstructions.Any(p => p == null))
return false; // SMAPI needs to load the value onto the stack before the method call, but the optional parameter type wasn't recognized
// inject new parameters
foreach (var entry in injectables)
{
// load value onto stack
cil.InsertBefore(instruction, entry.LoadValueInstruction);
// add parameter
ParameterDefinition parameter = entry.Parameter;
var newParameter = new ParameterDefinition(
name: parameter.Name,
attributes: parameter.Attributes,
parameterType: module.ImportReference(parameter.ParameterType)
);
newParameter.Constant = parameter.Constant;
methodRef.Parameters.Add(newParameter);
}
// rewrite method reference
foreach (Instruction loadInstruction in loadInstructions)
cil.InsertBefore(instruction, loadInstruction);
instruction.Operand = module.ImportReference(method);
this.Phrases.Add($"{methodRef.DeclaringType.Name}.{methodRef.Name} (added missing optional parameters)");
return this.MarkRewritten();