Anvilcraft/buildcraft
3
1
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

math.ceil() is very slow

Browse source 
see http://imgur.com/nT62QYQ
math.ceil is almost 100% of the cpu cost of pipePowerWood and transport.pipeTransportPower
(full nps from 
https://www.dropbox.com/s/01ywgc8lnyr2es2/1361111585024.nps?dl=1
several hundred k items extracted from ex-dim barrels, sent to a set of act's, crafted to blocks, and then sent back to barrels.

sendStack->queueRoutedItem() is to do with logistics HUD display, and is being worked on lag-wise also.)
This commit is contained in:
Andrew Hill 2013-02-18 12:26:25 +10:00
parent fefd3df7e2
commit 2e2e1d8ce8
1 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
common/buildcraft/transport/PipeTransportPower.java
View file

@ -123,8 +123,8 @@ public class PipeTransportPower extends PipeTransport {
nearbyTransport.receiveEnergy(ForgeDirection.VALID_DIRECTIONS[j].getOpposite(), watts);
displayPower[j] += (short) Math.ceil(watts * DISPLAY_POWER_FACTOR);
displayPower[i] += (short) Math.ceil(watts * DISPLAY_POWER_FACTOR);
displayPower[j] += (short) (watts * DISPLAY_POWER_FACTOR + .9999);
displayPower[i] += (short) (watts * DISPLAY_POWER_FACTOR + .9999);
internalPower[i] -= watts;
} else if (tiles[j] instanceof IPowerReceptor) {
@ -135,8 +135,8 @@ public class PipeTransportPower extends PipeTransport {
if (prov != null) {
prov.receiveEnergy((float) watts, ForgeDirection.VALID_DIRECTIONS[j].getOpposite());
displayPower[j] += (short) Math.ceil(watts * DISPLAY_POWER_FACTOR);
displayPower[i] += (short) Math.ceil(watts * DISPLAY_POWER_FACTOR);
displayPower[j] += (short) (watts * DISPLAY_POWER_FACTOR + .9999);
displayPower[i] += (short) (watts * DISPLAY_POWER_FACTOR + .9999);
internalPower[i] -= watts;
}