mirrors/PocketMine-MP
1
0
Fork 0
mirror of https://github.com/pmmp/PocketMine-MP.git synced 2025-05-17 19:28:55 +00:00
Code Issues Projects Releases Wiki Activity

Entity: get rid of temporalVector mutation

Browse Source 
except for checkBlockCollision, these are all cold paths ... this gets us one step closer to immutable Vector3
This commit is contained in:
Dylan K. Taylor 2020-06-20 21:41:19 +01:00
parent 42637f97c6
commit 80e150c803
1 changed files with 3 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
src/entity/Entity.php
View File

@ -115,9 +115,6 @@ abstract class Entity{
/** @var bool */
protected $forceMovementUpdate = false;
/** @var Vector3 */
public $temporalVector;
/** @var AxisAlignedBB */
public $boundingBox;
/** @var bool */
@ -224,8 +221,6 @@ abstract class Entity{
public function __construct(Location $location, ?CompoundTag $nbt = null){
$this->timings = Timings::getEntityTimings($this);
$this->temporalVector = new Vector3(0, 0, 0);
if($this->eyeHeight === null){
$this->eyeHeight = $this->height / 2 + 0.1;
}
@ -918,7 +913,7 @@ abstract class Entity{
$x = -$xz * sin(deg2rad($this->location->yaw));
$z = $xz * cos(deg2rad($this->location->yaw));
return $this->temporalVector->setComponents($x, $y, $z)->normalize();
return (new Vector3($x, $y, $z))->normalize();
}
public function getDirectionPlane() : Vector2{
@ -1267,7 +1262,7 @@ abstract class Entity{
}
protected function checkBlockCollision() : void{
$vector = $this->temporalVector->setComponents(0, 0, 0);
$vector = new Vector3(0, 0, 0);
foreach($this->getBlocksAround() as $block){
$block->onEntityInside($this);
@ -1423,7 +1418,7 @@ abstract class Entity{
}
$pos = $ev->getTo();
$this->setMotion($this->temporalVector->setComponents(0, 0, 0));
$this->setMotion(new Vector3(0, 0, 0));
if($this->setPositionAndRotation($pos, $yaw ?? $this->location->yaw, $pitch ?? $this->location->pitch)){
$this->resetFallDistance();
$this->setForceMovementUpdate();