MISC: fix a bug and refactoring

- Applying "rootstrap" method related to super class
- Fixed bugs detected by SVACE and applying Protex result.

Signed-off-by: JaeMin Kim <jm.jemin.kim@samsung.com>

diff --git a/org.tizen.nativeplatform/src/org/tizen/nativeplatform/PlatformBuildTargetManager.java b/org.tizen.nativeplatform/src/org/tizen/nativeplatform/PlatformBuildTargetManager.java
index b34998b..280b1ca 100644 (file)
--- a/org.tizen.nativeplatform/src/org/tizen/nativeplatform/PlatformBuildTargetManager.java
+++ b/org.tizen.nativeplatform/src/org/tizen/nativeplatform/PlatformBuildTargetManager.java
@@ -30,6 +30,7 @@ import java.util.List;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 import org.tizen.nativecommon.build.CommonBuildTargetManager;
+import org.tizen.nativecommon.build.exception.SBIException;
 import org.tizen.sbilib.ide.PluginManager;
 import org.tizen.sbilib.ide.Rootstrap;
 
@@ -39,16 +40,16 @@ public class PlatformBuildTargetManager extends CommonBuildTargetManager {
             .getLogger(PlatformBuildTargetManager.class);
 
     /* (non-Javadoc)
-     * @see org.tizen.nativecommon.build.CommonBuildTargetManager#getDefaultToolchainId(java.lang.String, java.lang.String)
+     * @see org.tizen.nativecommon.build.CommonBuildTargetManager#getDefaultToolchainId(java.lang.String, java.lang.String[])
      */
     @Override
-    public String getDefaultToolchainId(String rootstrapId, String projectType) {
+    public String getDefaultToolchainId(String rootstrapId) {
 
-        String toolchainID = PlatformConfigurationManager.getToolChainId(rootstrapId, projectType);
+        String toolchainID = PlatformConfigurationManager.getToolChainId(rootstrapId);
         if (toolchainID != null) {
             return toolchainID;
         } else {
-            logger.warn("Cannot find toolchain from projectType : " + projectType);
+            logger.warn("Cannot find toolchain from rootstrapId : " + rootstrapId);
             return null;
         }
     }

diff --git a/org.tizen.nativeplatform/src/org/tizen/nativeplatform/PlatformConfigurationManager.java b/org.tizen.nativeplatform/src/org/tizen/nativeplatform/PlatformConfigurationManager.java
index 7a338de..0946532 100644 (file)
--- a/org.tizen.nativeplatform/src/org/tizen/nativeplatform/PlatformConfigurationManager.java
+++ b/org.tizen.nativeplatform/src/org/tizen/nativeplatform/PlatformConfigurationManager.java
@@ -239,9 +239,9 @@ public class PlatformConfigurationManager extends CommonConfigurationManager {
      * @return If null, caller notify warn message.
      * @throws SBIException
      */
-    public static String getToolChainId(String rootstrapId, String projectType) {
+    public static String getToolChainId(String rootstrapId) {
         try {
-            List<String> availableToolchain = getToolchainList(rootstrapId, projectType);
+            List<String> availableToolchain = getToolchainList(rootstrapId);
             // In case of tizen.platform type there is only one toolchain per rootstrap
             if (availableToolchain != null && availableToolchain.size() > 0) {
                 return availableToolchain.get(0);