[Title] fix potential bug
authorgreatim <jaewon81.lim@samsung.com>
Wed, 2 Apr 2014 02:19:23 +0000 (11:19 +0900)
committergreatim <jaewon81.lim@samsung.com>
Wed, 2 Apr 2014 02:19:23 +0000 (11:19 +0900)
[Desc.]
[Issue]

org.tizen.dynamicanalyzer.workbench/src/org/tizen/dynamicanalyzer/workbench/Application.java
org.tizen.dynamicanalyzer.workbench/src/org/tizen/dynamicanalyzer/workbench/LayoutManager.java

index 1a963ac..257a2eb 100755 (executable)
@@ -29,6 +29,7 @@ package org.tizen.dynamicanalyzer.workbench;
 import java.io.File;
 import java.io.FileOutputStream;
 import java.io.PrintStream;
+import java.net.MalformedURLException;
 import java.net.URL;
 import java.text.SimpleDateFormat;
 import java.util.ArrayList;
@@ -62,9 +63,14 @@ public class Application implements IApplication {
                if (pathValidness) {
 
                        // set workspace directory for dynamic analyzer
+                       // why?
                        Location instanceLoc = Platform.getInstanceLocation();
                        String path = PathManager.DA_INSTALL_PATH + File.separator;
-                       instanceLoc.set(new URL("file", null, path), false);
+                       try {
+                               instanceLoc.set(new URL("file", null, path), false);
+                       } catch (MalformedURLException e) {
+                               e.printStackTrace();
+                       }
 
                        // set log postfix
                        SimpleDateFormat format = new SimpleDateFormat(
index 13033b6..f2ce563 100644 (file)
@@ -24,7 +24,7 @@ public class LayoutManager {
 
        public static void createComposite(IWorkbenchWindowConfigurer configurer,
                        Shell shell) {
-               
+
                // Sets shell layout.
                final FillLayout shellLayout = new FillLayout();
                shellLayout.marginHeight = 0;
@@ -41,24 +41,23 @@ public class LayoutManager {
 
                // create page composite with base composite
                Control page = configurer.createPageComposite(baseComposite);
+               if (null != page) {
+                       // layout each composite
+                       FormData data = new FormData();
+                       data.top = new FormAttachment(0, 0);
+                       data.left = new FormAttachment(0, 0);
+                       data.right = new FormAttachment(100, 0);
+                       data.height = 38;
+                       toolbarComposite.setLayoutData(data);
 
-               // layout each composite
-               FormData data = new FormData();
-               data.top = new FormAttachment(0, 0);
-               data.left = new FormAttachment(0, 0);
-               data.right = new FormAttachment(100, 0);
-               data.height = 38;
-               toolbarComposite.setLayoutData(data);
-
-               data = new FormData();
-               data.top = new FormAttachment(toolbarComposite, 0, SWT.BOTTOM);
-               data.left = new FormAttachment(0, 0);
-               data.right = new FormAttachment(100, 0);
-               data.bottom = new FormAttachment(100, 0);
-               page.setLayoutData(data);
+                       data = new FormData();
+                       data.top = new FormAttachment(toolbarComposite, 0, SWT.BOTTOM);
+                       data.left = new FormAttachment(0, 0);
+                       data.right = new FormAttachment(100, 0);
+                       data.bottom = new FormAttachment(100, 0);
+                       page.setLayoutData(data);
 
-               configurer.getWindow().getShell().layout(true);
-               if (null != page) {
+                       configurer.getWindow().getShell().layout(true);
                        ((Composite) page).layout(true);
                }
        }