[Title] Fix bug candidate in static analysis
authorBon-Yong Lee <bonyong.lee@samsung.com>
Sun, 26 Aug 2012 02:15:54 +0000 (11:15 +0900)
committerBon-Yong Lee <bonyong.lee@samsung.com>
Sun, 26 Aug 2012 02:15:54 +0000 (11:15 +0900)
[Type]      Enhancement
[Module]    Sub
[Priority]  Minor
[CQ#]       // CQ Issue Number
[Redmine#]  // Redmine Isuue Number
[Problem]   // Problem Description
[Cause]     // Cause Description
[Solution]  // Solution Description
[TestCase]  // Executed the test-target (How to)

org.tizen.common.connection/src/org/tizen/common/connection/explorer/ConnectionExplorerPanel.java [changed mode: 0644->0755]
org.tizen.common.connection/src/org/tizen/common/connection/properties/ConnectionExplorerPermissionPropertyPages.java [changed mode: 0644->0755]
org.tizen.common.connection/src/org/tizen/common/connection/ui/ConnectionExplorer.java [changed mode: 0644->0755]
org.tizen.common.externals/src/org/mihalis/opal/columns/ColumnBrowserWidget.java [changed mode: 0644->0755]
org.tizen.common.externals/src/org/mihalis/opal/multiChoice/MultiChoice.java [changed mode: 0644->0755]
org.tizen.common.externals/src/org/mihalis/opal/rangeSlider/RangeSlider.java [changed mode: 0644->0755]
org.tizen.common.externals/src/org/mihalis/opal/tipOfTheDay/TipOfTheDay.java [changed mode: 0644->0755]
org.tizen.common/src/org/tizen/common/file/StandardFileHandler.java [changed mode: 0644->0755]

old mode 100644 (file)
new mode 100755 (executable)
index bab1c36..f5d9b46
@@ -86,7 +86,6 @@ import org.tizen.common.ui.view.console.AnsicodeAdapter;
 import org.tizen.common.ui.view.console.ConsoleManager;
 import org.tizen.common.util.DialogUtil;
 import org.tizen.common.util.FilenameUtil;
-import org.tizen.common.util.OSChecker;
 import org.tizen.common.util.log.Logger;
 import org.tizen.sdblib.FileListingService.FileEntry;
 import org.tizen.sdblib.IDevice;
@@ -909,6 +908,8 @@ public class ConnectionExplorerPanel extends Panel implements IDeviceChangeListe
                             yesAll = true;
                         case IDialogConstants.YES_ID:
                             break;
+                        default:
+                            break;
                         }
                     }
                 }
@@ -1075,6 +1076,8 @@ public class ConnectionExplorerPanel extends Panel implements IDeviceChangeListe
                         return ConnectionUIMessages.Explorer_Message_Invalid_Character + FilenameUtil.getInvalidCharacters(os);
                     case FilenameUtil.HAS_NO_NAME:
                         return "";
+                    default:
+                        break;
                 }
             }
             return null;
old mode 100644 (file)
new mode 100755 (executable)
index 538a411..f286a6f
@@ -128,14 +128,10 @@ public class ConnectionExplorerPermissionPropertyPages extends PropertyPage {
                table.setHeaderVisible(true);
                table.setLinesVisible(true);
                
-               TableColumn colMsg = createTableColumn(table, "", SWT.NONE,
-               "EXECUTE");
-               colMsg = createTableColumn(table, "Read", SWT.NONE,
-               "EXECUTE");
-               colMsg = createTableColumn(table, "Write", SWT.NONE,
-               "EXECUTE");
-               colMsg = createTableColumn(table, "Execute", SWT.NONE,
-               "EXECUTE");
+               createTableColumn(table, "", SWT.NONE, "EXECUTE");
+               createTableColumn(table, "Read", SWT.NONE, "EXECUTE");
+               createTableColumn(table, "Write", SWT.NONE, "EXECUTE");
+               createTableColumn(table, "Execute", SWT.NONE, "EXECUTE");
                
                
                itemUser = new TableItem(table, SWT.NONE);
old mode 100644 (file)
new mode 100755 (executable)
index a756a74..0b5f061
@@ -388,7 +388,7 @@ public class ConnectionExplorer extends ViewPart
         toolBar = new ToolBar(shell, SWT.NULL);
         toolBar.setLayoutData(layoutData);
 
-        ToolItem item = new ToolItem(toolBar, SWT.SEPARATOR);
+        new ToolItem(toolBar, SWT.SEPARATOR);   // Separator
 
         emulManagerToolItem = new ToolItem(toolBar, SWT.PUSH);
         emulManagerToolItem.setImage(ConnectionPlugin.getImageDescriptorFromPlugin("icons/emulator_manager.gif").createImage());
@@ -409,7 +409,7 @@ public class ConnectionExplorer extends ViewPart
         });
         emulManagerToolItem.setEnabled(true);
 
-        item = new ToolItem(toolBar, SWT.SEPARATOR);
+        new ToolItem(toolBar, SWT.SEPARATOR);   // Separator
 
         pushToolItem = new ToolItem(toolBar, SWT.PUSH);
         pushToolItem.setImage(ConnectionPlugin.getImageDescriptorFromPlugin("icons/push.gif").createImage());
@@ -424,7 +424,7 @@ public class ConnectionExplorer extends ViewPart
         });
         pushToolItem.setEnabled(false);
 
-        item = new ToolItem(toolBar, SWT.SEPARATOR);
+        new ToolItem(toolBar, SWT.SEPARATOR);   // Separator
 
         pullToolItem = new ToolItem(toolBar, SWT.PUSH);
         pullToolItem.setImage(ConnectionPlugin.getImageDescriptorFromPlugin("icons/pull.gif").createImage());
@@ -438,7 +438,7 @@ public class ConnectionExplorer extends ViewPart
             }
         });
         pullToolItem.setEnabled(false);
-        item = new ToolItem(toolBar, SWT.SEPARATOR);
+        new ToolItem(toolBar, SWT.SEPARATOR);   // Separator
     }
 
     @Override
@@ -450,7 +450,7 @@ public class ConnectionExplorer extends ViewPart
     {
         String exePath = null;
         int osType = ConnectionPlugin.getDefault().os;
-        int linuxType = ConnectionPlugin.getDefault().LINUX;
+        int linuxType = ConnectionPlugin.LINUX;
 
         if (osType == linuxType)
         {
old mode 100644 (file)
new mode 100755 (executable)
index b358121..991f6a2
@@ -183,6 +183,8 @@ public class ColumnBrowserWidget extends ScrolledComposite {
                                        event.gc.drawImage(ColumnBrowserWidget.this.columnArrow, x, event.y + offset);\r
                                        break;\r
                                }\r
+                               default:\r
+                                   break;\r
                                }\r
                        }\r
                };\r
old mode 100644 (file)
new mode 100755 (executable)
index bb0c2ac..43271e5
@@ -1025,6 +1025,8 @@ public class MultiChoice<T> extends Composite {
                        notifyListeners(SWT.FocusOut, e);\r
                        break;\r
                }\r
+               default:\r
+                   break;\r
                }\r
        }\r
 \r
@@ -1054,6 +1056,8 @@ public class MultiChoice<T> extends Composite {
                                dropDown(false);\r
                        }\r
                        break;\r
+               default:\r
+                   break;\r
                }\r
 \r
        }\r
@@ -1073,6 +1077,8 @@ public class MultiChoice<T> extends Composite {
                        dropDown(!isDropped());\r
                        break;\r
                }\r
+               default:\r
+                   break;\r
                }\r
        }\r
 \r
old mode 100644 (file)
new mode 100755 (executable)
index 8f58b7f..4ec53c7
@@ -126,6 +126,8 @@ public class TipOfTheDay {
                                        event.detail = SWT.TRAVERSE_NONE;\r
                                        event.doit = false;\r
                                        break;\r
+                               default:\r
+                                   break;\r
                                }\r
                        }\r
                });\r
old mode 100644 (file)
new mode 100755 (executable)
index 5820078..a0b852e
@@ -35,10 +35,8 @@ import java.io.IOException;
 import java.io.InputStream;
 import java.util.ArrayList;
 import java.util.Collection;
-import java.util.Stack;
 
 import org.tizen.common.util.IOUtil;
-import org.tizen.common.util.log.Logger;
 
 /**
  * <p>