History

vec3FromScriptValue replacement attempt

ksuprynowicz committed 3 years ago

Commit dba09251 parent 3a2075012 files · +133 −10Browse files

Showing the changes to libraries/script-engine/src/ScriptValueUtils.cpp (1 of 12 files). Show all

Unified · Split

libraries/script-engine/src/ScriptValueUtils.cpp+12−5
@@ -33,12 +33,7 @@ · 32 unmodified lines
3333 #include "ScriptEngineCast.h"
3434 #include "ScriptValueIterator.h"
3535 #include "ScriptEngineLogging.h"
36−
37−#define CONVERSIONS_OPTIMIZED_FOR_V8
38−
39−#ifdef CONVERSIONS_OPTIMIZED_FOR_V8
4036 #include "v8/FastScriptValueUtils.h"
41−#endif
4237
4338 bool isListOfStrings(const ScriptValue& arg) {
4439 if (!arg.isArray()) {
@@ -136,7 +131,11 @@ ScriptValue qVector3DToScriptValue(ScriptEngine* engine, const QVector3D& qVecto · 91 unmodified lines
136131
137132 bool qVector3DFromScriptValue(const ScriptValue& object, QVector3D& qVector3D) {
138133 glm::vec3 vec3;
134+#ifdef CONVERSIONS_OPTIMIZED_FOR_V8
135+ if (vec3FromScriptValueFast(object, vec3)) {
136+#else
139137 if (vec3FromScriptValue(object, vec3)) {
138+#endif
140139 qVector3D.setX(vec3.x);
141140 qVector3D.setY(vec3.y);
142141 qVector3D.setZ(vec3.z);
@@ -485,7 +484,11 @@ QVector<glm::vec3> qVectorVec3FromScriptValue(const ScriptValue& array) { · 342 unmodified lines
485484
486485 for (int i = 0; i < length; i++) {
487486 glm::vec3 newVec3 = glm::vec3();
487+#ifdef CONVERSIONS_OPTIMIZED_FOR_V8
488+ vec3FromScriptValueFast(array.property(i), newVec3);
489+#else
488490 vec3FromScriptValue(array.property(i), newVec3);
491+#endif
489492 newVector << newVec3;
490493 }
491494 return newVector;
@@ -496,7 +499,11 @@ bool qVectorVec3FromScriptValue(const ScriptValue& array, QVector<glm::vec3>& ve · 4 unmodified lines
496499
497500 for (int i = 0; i < length; i++) {
498501 glm::vec3 newVec3 = glm::vec3();
502+#ifdef CONVERSIONS_OPTIMIZED_FOR_V8
503+ vec3FromScriptValueFast(array.property(i), newVec3);
504+#else
499505 vec3FromScriptValue(array.property(i), newVec3);
506+#endif
500507 vector << newVec3;
501508 }
502509 return true;