Browse Source

Merge pull request #968 from AtomicGameEngine/JME-ATOMIC-TSFIX

Update JSComponent to used SharedPtr
JoshEngebretson 9 years ago
parent
commit
818e0604de

+ 3 - 3
Source/AtomicJS/Javascript/JSComponentFile.cpp

@@ -82,9 +82,9 @@ bool JSComponentFile::PushModule()
 
 }
 
-JSComponent* JSComponentFile::CreateJSComponent()
+SharedPtr<JSComponent> JSComponentFile::CreateJSComponent()
 {
-    JSComponent* component = NULL;
+    SharedPtr<JSComponent> component;
 
     if (!scriptClass_)
     {
@@ -118,7 +118,7 @@ JSComponent* JSComponentFile::CreateJSComponent()
 
     }
 
-    if (!component)
+    if (component.Null())
     {
         ATOMIC_LOGERRORF("Failed to create script class from component file %s", GetName().CString());
         component =  new JSComponent(context_);

+ 1 - 1
Source/AtomicJS/Javascript/JSComponentFile.h

@@ -54,7 +54,7 @@ public:
 
     bool GetScriptClass() { return scriptClass_; }
 
-    JSComponent* CreateJSComponent();
+    SharedPtr<JSComponent> CreateJSComponent();
     bool PushModule();
 
 private:

+ 1 - 1
Source/AtomicJS/Javascript/JSScene.cpp

@@ -66,7 +66,7 @@ static int Node_CreateJSComponent(duk_context* ctx)
         return 1;
     }
 
-    JSComponent* jsc = file->CreateJSComponent();
+    SharedPtr<JSComponent> jsc = file->CreateJSComponent();
 
     node->AddComponent(jsc, jsc->GetID(), LOCAL);