Skip to content

Conversation

@Aingar
Copy link
Collaborator

@Aingar Aingar commented Nov 17, 2023

  • Major performance optimization to health manipulation
  • Earhart randomization should now never trigger after a server crash
  • Flattening of Dock_Call hook method
  • Minor cleanup of unused variables in CoreModule
  • /base setshield now only accepts valid numeric input

- Major performance optimization to health manipulation
- Earhart randomization should now never trigger after a server crash
- Flattening of Dock_Call hook method
- Minor cleanup of unused variables in CoreModule
- /base setshield now only accepts valid numeric input
@Aingar Aingar changed the title POB Rework POB Update Nov 17, 2023
}

WritePrivateProfileString("Timer", "lastRandomization", itos((int)currTime).c_str(), cfg_filehyperspaceHubTimer.c_str());
WritePrivateProfileString("Timer", "lastRandomization", itos((int)(currTime - (currTime % randomizationCooldown))).c_str(), cfg_filehyperspaceHubTimer.c_str());
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

wot - isn't this some other PR?


pub::SpaceObj::SetRelativeHealth(space_obj, base->base_health / base->max_base_health);

base->baseCSolar = (CSolar*)CObject::Find(space_obj, CObject::CSOLAR_OBJECT);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You must call Release on this whenever we bin a PlayerBase object, e.g. probably in basedespawn

base_shield_reinforcement_threshold = FLT_MAX;
}
}
if (baseCSolar && base_health <= max_base_health)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok but what does this have to do with checking vulnerability windows?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants