Code style and warnings cleanup.

diff --git a/gss/src/gr/ebs/gss/client/SearchResults.java b/gss/src/gr/ebs/gss/client/SearchResults.java
index 5ed5a7c..16cb7ec 100644 (file)
--- a/gss/src/gr/ebs/gss/client/SearchResults.java
+++ b/gss/src/gr/ebs/gss/client/SearchResults.java
@@ -107,7 +107,7 @@ public class SearchResults extends Composite implements TableListener, ClickList
        /**\r
         * The table widget with the file list.\r
         */\r
-       private Grid table;// = new Grid(GSS.VISIBLE_FILE_COUNT + 1, 8);\r
+       private Grid table;\r
 \r
        /**\r
         * The navigation bar for paginating the results.\r
@@ -117,20 +117,18 @@ public class SearchResults extends Composite implements TableListener, ClickList
        /**\r
         * The number of files in the search results\r
         */\r
-       int folderFileCount;\r
+       private int folderFileCount;\r
 \r
        /**\r
         * Total search results size\r
         */\r
-       long folderTotalSize;\r
-\r
+       private long folderTotalSize;\r
 \r
        /**\r
         * A cache of the files in the list.\r
         */\r
        private List<FileResource> files;\r
 \r
-\r
        /**\r
         * The widget's image bundle.\r
         */\r
@@ -165,6 +163,8 @@ public class SearchResults extends Composite implements TableListener, ClickList
        public SearchResults(final Images _images) {\r
                images = _images;\r
                table = new Grid(GSS.VISIBLE_FILE_COUNT + 1, 8){\r
+\r
+                       @Override\r
                        public void onBrowserEvent(Event event) {\r
                                if (files == null || files.size() == 0)\r
                                        return;\r
@@ -216,7 +216,7 @@ public class SearchResults extends Composite implements TableListener, ClickList
                table.addTableListener(this);\r
 \r
                // Create the 'navigation' bar at the upper-right.\r
-               final HorizontalPanel innerNavBar = new HorizontalPanel();\r
+               HorizontalPanel innerNavBar = new HorizontalPanel();\r
                innerNavBar.setStyleName("gss-ListNavBar");\r
                innerNavBar.setSpacing(8);\r
                innerNavBar.add(prevButton);\r
@@ -266,8 +266,6 @@ public class SearchResults extends Composite implements TableListener, ClickList
                }\r
        }\r
 \r
-\r
-\r
        /**\r
         * Retrieve the root folder for the current user.\r
         *\r
@@ -281,13 +279,7 @@ public class SearchResults extends Composite implements TableListener, ClickList
                return DONE;\r
        }\r
 \r
-       /*\r
-        * (non-Javadoc)\r
-        *\r
-        * @see com.google.gwt.user.client.ui.TableListener#onCellClicked(com.google.gwt.user.client.ui.SourcesTableEvents,\r
-        *      int, int)\r
-        */\r
-       public void onCellClicked(@SuppressWarnings("unused") SourcesTableEvents sender, int row, @SuppressWarnings("unused") int cell) {\r
+       public void onCellClicked(SourcesTableEvents sender, int row, int cell) {\r
                // Select the row that was clicked (-1 to account for header row).\r
                if (row > folderFileCount)\r
                        return;\r
@@ -429,7 +421,6 @@ public class SearchResults extends Composite implements TableListener, ClickList
                }\r
                prevButton.setVisible(false);\r
                nextButton.setVisible(false);\r
-\r
        }\r
 \r
        /**\r
@@ -451,7 +442,6 @@ public class SearchResults extends Composite implements TableListener, ClickList
                                        }\r
                                        selectedRows.add(startIndex + row);\r
                                        styleRow(row, true);\r
-\r
                                }\r
                        else {\r
                                clearSelectedRows();\r
@@ -464,7 +454,6 @@ public class SearchResults extends Composite implements TableListener, ClickList
                                GSS.get().setCurrentSelection(getSelectedFiles());\r
                        //contextMenu.setFiles(getSelectedFiles());\r
                        table.setWidget(row + 1, 0, contextMenu);\r
-\r
                }\r
        }\r
 \r
@@ -586,10 +575,8 @@ public class SearchResults extends Composite implements TableListener, ClickList
 \r
        /**\r
         * Update the file cache with data from the server.\r
-        *\r
-        * @param userId the ID of the current user\r
         */\r
-       public void updateFileCache( String query) {\r
+       public void updateFileCache(String query) {\r
                clearSelectedRows();\r
                sortingProperty = "name";\r
                clearLabels();\r
@@ -605,7 +592,8 @@ public class SearchResults extends Composite implements TableListener, ClickList
                } else{\r
                        searchResults.setHTML("Search results for " + query);\r
 \r
-                       ExecuteGet<SearchResource> eg = new ExecuteGet<SearchResource>(SearchResource.class, GSS.GSS_REST_PATH+"search/"+URL.encodeComponent(query)){\r
+                       ExecuteGet<SearchResource> eg = new ExecuteGet<SearchResource>(SearchResource.class,\r
+                                               GSS.GSS_REST_PATH+"search/"+URL.encodeComponent(query)){\r
 \r
                                @Override\r
                                public void onComplete() {\r
@@ -614,6 +602,7 @@ public class SearchResults extends Composite implements TableListener, ClickList
                                        update();\r
                                }\r
 \r
+                               @Override\r
                                public void onError(Throwable t) {\r
                                        if(t instanceof RestException)\r
                                                GSS.get().displayError("Unable to perform search:"+((RestException)t).getHttpStatusText());\r
@@ -633,13 +622,12 @@ public class SearchResults extends Composite implements TableListener, ClickList
         * @param _files\r
         * @param filePaths the files to set\r
         */\r
-       void setFiles(final List<FileResource> _files) {\r
+       private void setFiles(List<FileResource> _files) {\r
                files = _files;\r
                Collections.sort(files, new Comparator<FileResource>() {\r
 \r
                        public int compare(FileResource arg0, FileResource arg1) {\r
                                return arg0.getName().compareTo(arg1.getName());\r
-\r
                        }\r
 \r
                });\r
@@ -764,9 +752,6 @@ public class SearchResults extends Composite implements TableListener, ClickList
             $doc.body.onselectstart = null;\r
         }-*/;\r
 \r
-       /**\r
-        *\r
-        */\r
        public void selectAllRows() {\r
                clearSelectedRows();\r
                int count = folderFileCount;\r
@@ -788,7 +773,6 @@ public class SearchResults extends Composite implements TableListener, ClickList
                GSS.get().setCurrentSelection(getSelectedFiles());\r
                //contextMenu.setFiles(getSelectedFiles());\r
                table.setWidget(i - 1, 0, contextMenu);\r
-\r
        }\r
 \r
 }\r