Skip to content
This repository has been archived by the owner. It is now read-only.

Finished CExplosion #390

Merged
merged 1 commit into from Apr 9, 2020
Merged

Finished CExplosion #390

merged 1 commit into from Apr 9, 2020

Conversation

@Sergeanur
Copy link
Collaborator

@Sergeanur Sergeanur commented Apr 8, 2020

No description provided.

@@ -230,15 +230,15 @@ CRubbish::Update(void)
spawnDist = (CGeneral::GetRandomNumber()&0xFF)/256.0f + RUBBISH_MAX_DIST;
uint8 r = CGeneral::GetRandomNumber();
if(r&1)
spawnAngle = (CGeneral::GetRandomNumber()&0xFF)/256.0f * 6.28f;
spawnAngle = (CGeneral::GetRandomNumber()&0xFF)/256.0f * TWOPI;

This comment has been minimized.

@ghost

ghost Apr 8, 2020

maybe adding #if ALWAYS_ACCURATE_PI ?

This comment has been minimized.

@aap

aap Apr 9, 2020
Member

we should keep it as 6.28 for now and if we decide to get rid of 6.28 we'll do it everywhere in one PR and not piecemeal.

@Sergeanur Sergeanur force-pushed the Sergeanur:Explosion branch from f0afbb2 to 1f212df Apr 8, 2020
@Sergeanur Sergeanur force-pushed the Sergeanur:Explosion branch from 1f212df to cb262c7 Apr 9, 2020
@aap aap merged commit a6d449a into GTAmodding:master Apr 9, 2020
1 check was pending
1 check was pending
@appveyor
continuous-integration/appveyor/pr Waiting for AppVeyor build to complete
Details
@Sergeanur Sergeanur deleted the Sergeanur:Explosion branch May 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants