aboutsummaryrefslogtreecommitdiff
path: root/NorthstarDLL
diff options
context:
space:
mode:
authorJan <sentrycraft123@gmail.com>2023-07-25 16:31:02 +0200
committerGitHub <noreply@github.com>2023-07-25 16:31:02 +0200
commitddb4670354a5f29e6ebd08f625edb2576de700e6 (patch)
tree34bcfea868b620ad9ba4ea63dec19b5476123c05 /NorthstarDLL
parentcf8b267d31dc168673e68e336fd5727aad4e7333 (diff)
downloadNorthstarLauncher-ddb4670354a5f29e6ebd08f625edb2576de700e6.tar.gz
NorthstarLauncher-ddb4670354a5f29e6ebd08f625edb2576de700e6.zip
Remove Vector3 Constructor (#494)
- Packing was enabled because the compiler may add padding which we don't want. - Constructors were removed because they were either rather primitive or broken (the `pRawFloats` one). - Use of constructor was replaced with a simple cast, which works just as well.
Diffstat (limited to 'NorthstarDLL')
-rw-r--r--NorthstarDLL/core/math/vector.h7
-rw-r--r--NorthstarDLL/scripts/scriptdatatables.cpp4
2 files changed, 2 insertions, 9 deletions
diff --git a/NorthstarDLL/core/math/vector.h b/NorthstarDLL/core/math/vector.h
index 112fabdf..8684908f 100644
--- a/NorthstarDLL/core/math/vector.h
+++ b/NorthstarDLL/core/math/vector.h
@@ -13,13 +13,6 @@ union Vector3
float raw[3];
- Vector3() : x(0), y(0), z(0) {}
- Vector3(float x, float y, float z) : x(x), y(y), z(z) {}
- Vector3(float* pRawFloats) // assumes float[3] => vector
- {
- memcpy(raw, pRawFloats, sizeof(this));
- }
-
void MakeValid()
{
for (auto& fl : raw)
diff --git a/NorthstarDLL/scripts/scriptdatatables.cpp b/NorthstarDLL/scripts/scriptdatatables.cpp
index 8aa52fdb..532624f3 100644
--- a/NorthstarDLL/scripts/scriptdatatables.cpp
+++ b/NorthstarDLL/scripts/scriptdatatables.cpp
@@ -765,8 +765,8 @@ std::string DataTableToString(Datatable* datatable)
case DatatableType::VECTOR:
{
- Vector3 pVector((float*)pUntypedVal);
- sCSVString += fmt::format("<{},{},{}>", pVector.x, pVector.y, pVector.z);
+ Vector3* pVector = (Vector3*)(pUntypedVal);
+ sCSVString += fmt::format("<{},{},{}>", pVector->x, pVector->y, pVector->z);
break;
}