-
Notifications
You must be signed in to change notification settings - Fork 1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Refactor for objects #1209
Refactor for objects #1209
Conversation
c5b8f62
to
69e68e1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a few additions/suggestions
NitroxClient/Communication/NetworkingLayer/LiteNetLib/LiteNetLibClient.cs
Outdated
Show resolved
Hide resolved
NitroxClient/Communication/NetworkingLayer/LiteNetLib/LiteNetLibClient.cs
Outdated
Show resolved
Hide resolved
NitroxClient/Communication/NetworkingLayer/LiteNetLib/LiteNetLibClient.cs
Outdated
Show resolved
Hide resolved
NitroxClient/Communication/Packets/Processors/ModuleAddedProcessor.cs
Outdated
Show resolved
Hide resolved
NitroxClient/Communication/Packets/Processors/ModuleAddedProcessor.cs
Outdated
Show resolved
Hide resolved
...-Subnautica/DataStructures/GameLogic/Buildings/Rotation/SubnauticaRotationMetadataFactory.cs
Outdated
Show resolved
Hide resolved
@@ -22,7 +22,8 @@ public static bool Prefix(ExosuitClawArm __instance) | |||
if (pickupable != null && pickupable.isPickupable && componentInParent.storageContainer.container.HasRoomFor(pickupable)) | |||
{ | |||
NitroxServiceLocator.LocateService<Item>().PickedUp(pickupable.gameObject, pickupable.GetTechType()); | |||
} else if(component != null) | |||
} | |||
else if (component != null) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
else if (component != null) | |
else if (component) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If this is nullable, you'll need to if (component == true)
.
@@ -12,14 +11,20 @@ public void Prepare(Entity parentEntity, DeterministicBatchGenerator determinist | |||
{ | |||
Entity crashFish = SpawnChild(parentEntity, deterministicBatchGenerator, TechType.Crash, "7d307502-46b7-4f86-afb0-65fe8867f893"); | |||
crashFish.Transform.LocalRotation = new NitroxQuaternion(-0.7071068f, 0, 0, 0.7071068f); | |||
parentEntity.ChildEntities.Add(crashFish); | |||
//parentEntity.ChildEntities.Add(crashFish); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I assume this can be removed then
NitroxClient/Communication/Packets/Processors/PDAScannerEntryProgressProcessor.cs
Outdated
Show resolved
Hide resolved
NitroxModel/MultiplayerSession/MultiplayerSessionReservationState.cs
Outdated
Show resolved
Hide resolved
NitroxClient/Communication/MultiplayerSession/MultiplayerSessionConnectionStage.cs
Outdated
Show resolved
Hide resolved
c669002
to
1fca0a6
Compare
1fca0a6
to
b4a0724
Compare
Co-authored-by: Coding-Hen <drhen123@googlemail.com>
c3e6058
to
640a128
Compare
closes #1182