Skip to content

Commit 37e9aa0

Browse files
authored
Merge pull request #210 from krunt/origin/_v8
unify v8 and node rtScript code
2 parents d04a13e + cbee0ab commit 37e9aa0

34 files changed

+790
-3125
lines changed

examples/pxScene2d/src/init.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@ limitations under the License.
1717
*/
1818

1919
var isDuk = (typeof Duktape != "undefined")?true:false;
20+
var isV8 = (typeof _isV8 != "undefined")?true:false;
2021

2122
if (isDuk) {
2223
global.console = require('console');

examples/pxScene2d/src/pxScene2d.cpp

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -3804,16 +3804,12 @@ void pxScriptView::runScript()
38043804

38053805
if (mCtx)
38063806
{
3807-
#ifdef PXSCENE_V8_TEST
38083807
mPrintFunc = new rtFunctionCallback(printFunc, this);
3809-
#endif
38103808
mGetScene = new rtFunctionCallback(getScene, this);
38113809
mMakeReady = new rtFunctionCallback(makeReady, this);
38123810
mGetContextID = new rtFunctionCallback(getContextID, this);
38133811

3814-
#ifdef PXSCENE_V8_TEST
38153812
mCtx->add("print", mPrintFunc.getPtr());
3816-
#endif
38173813
mCtx->add("getScene", mGetScene.getPtr());
38183814
mCtx->add("makeReady", mMakeReady.getPtr());
38193815
mCtx->add("getContextID", mGetContextID.getPtr());
@@ -3822,9 +3818,6 @@ void pxScriptView::runScript()
38223818
mReady = new rtPromise();
38233819
#endif
38243820

3825-
//#ifdef PXSCENE_V8_TEST
3826-
//mCtx->runFile(mUrl);
3827-
//#else
38283821
mCtx->runFile("init.js");
38293822

38303823
char buffer[MAX_URL_SIZE + 50];
@@ -3857,7 +3850,6 @@ void pxScriptView::runScript()
38573850
#endif //ENABLE_RT_NODE
38583851
}
38593852

3860-
#ifdef PXSCENE_V8_TEST
38613853
rtError pxScriptView::printFunc(int numArgs, const rtValue* args, rtValue* result, void* ctx)
38623854
{
38633855
rtLogInfo(__FUNCTION__);
@@ -3875,7 +3867,6 @@ rtError pxScriptView::printFunc(int numArgs, const rtValue* args, rtValue* resul
38753867

38763868
return RT_OK;
38773869
}
3878-
#endif
38793870

38803871
rtError pxScriptView::getScene(int numArgs, const rtValue* args, rtValue* result, void* ctx)
38813872
{

examples/pxScene2d/src/pxScene2d.h

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -44,8 +44,6 @@
4444
#include "rtPromise.h"
4545
#include "rtThreadQueue.h"
4646

47-
#define PXSCENE_V8_TEST // TODO: remove
48-
4947
#define ANIMATION_ROTATE_XYZ
5048

5149
#include "pxResource.h"
@@ -1156,9 +1154,7 @@ class pxScriptView: public pxIView
11561154

11571155
protected:
11581156

1159-
#ifdef PXSCENE_V8_TEST
11601157
static rtError printFunc(int /*numArgs*/, const rtValue* /*args*/, rtValue* result, void* ctx);
1161-
#endif
11621158

11631159
static rtError getScene(int /*numArgs*/, const rtValue* /*args*/, rtValue* result, void* ctx);
11641160
static rtError makeReady(int /*numArgs*/, const rtValue* /*args*/, rtValue* result, void* ctx);
@@ -1276,9 +1272,7 @@ class pxScriptView: public pxIView
12761272
rtObjectRef mReady;
12771273
rtObjectRef mScene;
12781274
rtRef<pxIView> mView;
1279-
#ifdef PXSCENE_V8_TEST
12801275
rtRef<rtFunctionCallback> mPrintFunc;
1281-
#endif
12821276
rtRef<rtFunctionCallback> mGetScene;
12831277
rtRef<rtFunctionCallback> mMakeReady;
12841278
rtRef<rtFunctionCallback> mGetContextID;

examples/pxScene2d/src/rcvrcore/AppSceneContext.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@ limitations under the License.
1919
//"use strict";
2020

2121
var isDuk=(typeof Duktape != "undefined")?true:false;
22+
var isV8=(typeof _isV8 != "undefined")?true:false;
2223

2324
var url = require('url');
2425
var path = require('path');

examples/pxScene2d/src/rcvrcore/utils/AsyncFileAcquisition.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@ limitations under the License.
1919
"use strict";
2020

2121
var isDuk=(typeof Duktape != "undefined")?true:false;
22+
var isV8=(typeof _isV8 != "undefined")?true:false;
2223
var Logger = require('rcvrcore/Logger').Logger;
2324
var SceneModuleLoader = require('rcvrcore/SceneModuleLoader');
2425
var log = new Logger('AsyncFileAcquisition');

examples/pxScene2d/src/rcvrcore/utils/FileArchive.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@ limitations under the License.
1919
"use strict";
2020

2121
var isDuk = (typeof timers != "undefined")?true:false;
22+
var isV8 = (typeof _isV8 != "undefined")?true:false;
2223

2324
var fs = require('fs');
2425
var url = require('url');

examples/pxScene2d/src/shell.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@ limitations under the License.
1717
*/
1818

1919
var isDuk=(typeof Duktape != "undefined")?true:false;
20+
var isV8 = (typeof _isV8 != "undefined")?true:false;
2021

2122
px.import({ scene: 'px:scene.1.js',
2223
keys: 'px:tools.keys.js'

examples/pxScene2d/src/test_module_binding.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,4 +17,4 @@ uv_timer_start(tm1, 5000, 0, function () { print("OK1"); uv_timer_stop(tm2); });
1717

1818
uv_run_in_context("print(\"in eval context\");");
1919

20-
print("isV8 " + isV8);
20+
print("isV8 " + _isV8);

src/CMakeLists.txt

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -214,16 +214,15 @@ endif()
214214
if (SUPPORT_NODE)
215215
message("Adding Node scripting support")
216216
add_definitions(-DRTSCRIPT_SUPPORT_NODE)
217-
set(PXCORE_FILES ${PXCORE_FILES} rtScriptNode/rtScriptNode.cpp rtScriptNode/jsCallback.cpp rtScriptNode/rtFunctionWrapper.cpp
218-
rtScriptNode/rtObjectWrapper.cpp rtScriptNode/rtWrapperUtils.cpp)
217+
set(PXCORE_FILES ${PXCORE_FILES} rtScriptV8Node/rtScriptNode.cpp rtScriptV8Node/jsCallback.cpp rtScriptV8Node/rtFunctionWrapper.cpp
218+
rtScriptV8Node/rtObjectWrapper.cpp rtScriptV8Node/rtWrapperUtils.cpp)
219219
endif()
220220

221221
if (SUPPORT_V8)
222222
message("Adding V8 scripting support")
223223
add_definitions(-DRTSCRIPT_SUPPORT_V8)
224-
set(PXCORE_FILES ${PXCORE_FILES} rtScriptV8/rtScriptV8.cpp rtScriptV8/jsCallbackV8.cpp
225-
rtScriptV8/rtFunctionWrapperV8.cpp rtScriptV8/rtObjectWrapperV8.cpp rtScriptV8/rtWrapperUtilsV8.cpp
226-
rtScriptV8/rtJsModulesV8.cpp)
224+
set(PXCORE_FILES ${PXCORE_FILES} rtScriptV8Node/rtScriptV8.cpp rtScriptV8Node/jsCallback.cpp rtScriptV8Node/rtFunctionWrapper.cpp
225+
rtScriptV8Node/rtObjectWrapper.cpp rtScriptV8Node/rtWrapperUtils.cpp)
227226
endif()
228227

229228
if (PXCORE_MATRIX_HELPERS)

src/rtScript.cpp

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -26,12 +26,8 @@
2626

2727
#include "assert.h"
2828

29-
#ifdef RTSCRIPT_SUPPORT_NODE
30-
#include "rtScriptNode/rtScriptNode.h"
31-
#endif
32-
33-
#ifdef RTSCRIPT_SUPPORT_V8
34-
#include "rtScriptV8/rtScriptV8.h"
29+
#if defined RTSCRIPT_SUPPORT_NODE || defined RTSCRIPT_SUPPORT_V8
30+
#include "rtScriptV8Node/rtScriptV8Node.h"
3531
#endif
3632

3733
#ifdef RTSCRIPT_SUPPORT_DUKTAPE

0 commit comments

Comments
 (0)