Cinka/OldThink
1
0
Fork 0
Code Issues Pull Requests Actions 3 Packages Projects Releases Wiki Activity

Stop casting AME energy into int because it underflows. (#4762)

Browse Source
This commit is contained in:
20kdc
2021-10-04 01:58:54 +01:00
committed by GitHub
parent 661105dadd
commit 2b67a22485
1 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
Content.Server/AME/AMENodeGroup.cs
View File

@@ -102,7 +102,7 @@ namespace Content.Server.AME
} }
} }
public int InjectFuel(int fuel, out bool overloading) public float InjectFuel(int fuel, out bool overloading)
{ {
overloading = false; overloading = false;
if(fuel > 0 && CoreCount > 0) if(fuel > 0 && CoreCount > 0)
@@ -137,7 +137,8 @@ namespace Content.Server.AME
} }
} }
// Note the float conversions. The maths will completely fail if not done using floats. // Note the float conversions. The maths will completely fail if not done using floats.
return (int) ((((float) fuel) / CoreCount) * fuel * 20000); // Oh, and don't ever stuff the result of this in an int. Seriously.
return (((float) fuel) / CoreCount) * fuel * 20000;
} }
return 0; return 0;
} }