Util: Implement SequentialUtil

[Naii-the-Baf]

Most likely utility functions for koopa freerunning and timers.

---

This change is 

[MonsterDruide1]

Reviewed 3 of 3 files at r1, all commit messages.
Reviewable status: all files reviewed, 3 unresolved discussions (waiting on @Naii-the-Baf)

---

src/Util/SequentialUtil.h line 4 at r1 (raw file):

#include <basis/seadTypes.h>
#include <math/seadVector.h>

Useless include

---

src/Util/SequentialUtil.cpp line 7 at r1 (raw file):

void rs::convertCsecTo(s32* minutes, s32* seconds, s32* csec, s32 time) {
    *minutes = time / 6000;
    *seconds = (time - (time / 6000) * 6000) / 100;

Suggestion:

*seconds = (time % 6000) / 100;

---

src/Util/SequentialUtil.cpp line 20 at r1 (raw file):

s32 RaceTimeFunction::clampRaceRecordCsec(s32 csec) {
    return sead::Mathi::clamp(csec, 0, 359999);

Avoid magic numbers that are already defined once

Suggestion:

return sead::Mathi::clamp(csec, 0, getRaceTimeMaxCsec());

[MonsterDruide1]

Reviewable status: all files reviewed, 2 unresolved discussions (waiting on @Naii-the-Baf)

---

src/Util/SequentialUtil.cpp line 7 at r1 (raw file):

void rs::convertCsecTo(s32* minutes, s32* seconds, s32* csec, s32 time) {
    *minutes = time / 6000;
    *seconds = (time - (time / 6000) * 6000) / 100;

Nevermind, saw the decomp.me on that now - rather keep the match.

[Naii-the-Baf]

Reviewable status: 1 of 3 files reviewed, 2 unresolved discussions (waiting on @MonsterDruide1)

---

src/Util/SequentialUtil.h line 4 at r1 (raw file):

Previously, MonsterDruide1 wrote…

Useless include

Done.

---

src/Util/SequentialUtil.cpp line 20 at r1 (raw file):

Previously, MonsterDruide1 wrote…

Avoid magic numbers that are already defined once

Done.

[MonsterDruide1]

Reviewed 2 of 2 files at r2, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @Naii-the-Baf)

---

-- commits line 0 at r2:
Requires rebase.

[Naii-the-Baf]

Reviewable status: 2 of 3 files reviewed, 1 unresolved discussion (waiting on @MonsterDruide1)

---

-- commits line at r2:

Previously, MonsterDruide1 wrote…

Requires rebase.

Done.

[MonsterDruide1]

Reviewed 1 of 1 files at r3, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @Naii-the-Baf)