Splitted channeldata into two ints
Previously it did encode the current and previous used channels into the same as well as mask it with 0xFF. Which lead to an overflow every 256 gridnodes requiring a channel. This will not happen at > 2^31 Also removes the need to bitshift them for every access. Fixes #1510
This commit is contained in:
parent
52d62b8202
commit
9c3ca0e3d7
|
@ -71,7 +71,8 @@ public class GridNode implements IGridNode, IPathItem
|
|||
private Object visitorIterationNumber = null;
|
||||
// connection criteria
|
||||
private int compressedData = 0;
|
||||
private int channelData = 0;
|
||||
private int usedChannels = 0;
|
||||
private int lastUsedChannels = 0;
|
||||
|
||||
public GridNode( IGridBlock what )
|
||||
{
|
||||
|
@ -90,7 +91,7 @@ public class GridNode implements IGridNode, IPathItem
|
|||
|
||||
public int usedChannels()
|
||||
{
|
||||
return this.channelData >> 8;
|
||||
return this.lastUsedChannels;
|
||||
}
|
||||
|
||||
public Class<? extends IGridHost> getMachineClass()
|
||||
|
@ -358,7 +359,7 @@ public class GridNode implements IGridNode, IPathItem
|
|||
@Override
|
||||
public boolean meetsChannelRequirements()
|
||||
{
|
||||
return ( !this.gridProxy.getFlags().contains( GridFlags.REQUIRE_CHANNEL ) || this.getUsedChannels() > 0 );
|
||||
return( !this.gridProxy.getFlags().contains( GridFlags.REQUIRE_CHANNEL ) || this.getUsedChannels() > 0 );
|
||||
}
|
||||
|
||||
@Override
|
||||
|
@ -384,7 +385,7 @@ public class GridNode implements IGridNode, IPathItem
|
|||
|
||||
public int getUsedChannels()
|
||||
{
|
||||
return this.channelData & 0xff;
|
||||
return this.usedChannels;
|
||||
}
|
||||
|
||||
public void FindConnections()
|
||||
|
@ -581,7 +582,7 @@ public class GridNode implements IGridNode, IPathItem
|
|||
public void setGridStorage( GridStorage s )
|
||||
{
|
||||
this.myStorage = s;
|
||||
this.channelData = 0;
|
||||
this.usedChannels = 0;
|
||||
}
|
||||
|
||||
@Override
|
||||
|
@ -600,7 +601,7 @@ public class GridNode implements IGridNode, IPathItem
|
|||
{
|
||||
if( zeroOut )
|
||||
{
|
||||
this.channelData &= ~0xff;
|
||||
this.usedChannels = 0;
|
||||
}
|
||||
|
||||
int idx = this.connections.indexOf( fast );
|
||||
|
@ -631,7 +632,7 @@ public class GridNode implements IGridNode, IPathItem
|
|||
@Override
|
||||
public void incrementChannelCount( int usedChannels )
|
||||
{
|
||||
this.channelData += usedChannels;
|
||||
this.usedChannels += usedChannels;
|
||||
}
|
||||
|
||||
@Override
|
||||
|
@ -650,8 +651,7 @@ public class GridNode implements IGridNode, IPathItem
|
|||
|
||||
if( this.getLastUsedChannels() != this.getUsedChannels() )
|
||||
{
|
||||
this.channelData &= 0xff;
|
||||
this.channelData |= this.channelData << 8;
|
||||
this.lastUsedChannels = this.usedChannels;
|
||||
|
||||
if( this.getInternalGrid() != null )
|
||||
{
|
||||
|
@ -662,7 +662,7 @@ public class GridNode implements IGridNode, IPathItem
|
|||
|
||||
public int getLastUsedChannels()
|
||||
{
|
||||
return ( this.channelData >> 8 ) & 0xff;
|
||||
return lastUsedChannels;
|
||||
}
|
||||
|
||||
private static class MachineSecurityBreak implements Callable<Void>
|
||||
|
@ -683,7 +683,6 @@ public class GridNode implements IGridNode, IPathItem
|
|||
}
|
||||
}
|
||||
|
||||
|
||||
private static class ConnectionComparator implements Comparator<IGridConnection>
|
||||
{
|
||||
private final IGridNode gn;
|
||||
|
|
Loading…
Reference in New Issue