[gwt-contrib] [google-web-toolkit] r10243 committed - Fix warnings in samples/expenses...

Tue, 31 May 2011 10:32:19 -0700 

Revision: 10243
Author:   [email protected]
Date:     Fri May 27 11:03:45 2011
Log:      Fix warnings in samples/expenses

Review at http://gwt-code-reviews.appspot.com/1447810

Review by: [email protected]
http://code.google.com/p/google-web-toolkit/source/detail?r=10243

Modified:
/trunk/samples/expenses/src/main/java/com/google/gwt/mobile/client/TouchHandler.java /trunk/samples/expenses/src/main/java/com/google/gwt/sample/expenses/client/ExpenseReportDetails.java /trunk/samples/expenses/src/main/java/com/google/gwt/sample/expenses/client/ExpenseTree.java /trunk/samples/expenses/src/main/java/com/google/gwt/sample/expenses/client/MobileExpenseList.java /trunk/samples/expenses/src/main/java/com/google/gwt/sample/expenses/client/PhaseAnimation.java /trunk/samples/expenses/src/main/java/com/google/gwt/sample/expenses/client/SlidingPanel.java /trunk/samples/expenses/src/main/java/com/google/gwt/sample/expenses/server/domain/EntityCounter.java /trunk/samples/expenses/src/main/java/com/google/gwt/sample/gaerequest/client/GaeAuthRequestTransport.java 

=======================================
--- /trunk/samples/expenses/src/main/java/com/google/gwt/mobile/client/TouchHandler.java Tue Nov 30 12:17:15 2010 +++ /trunk/samples/expenses/src/main/java/com/google/gwt/mobile/client/TouchHandler.java Fri May 27 11:03:45 2011 
@@ -415,12 +415,10 @@
       scrollOffTimer.schedule(100);
     }

-    boolean firstDrag = false;
     if (!dragging) {
       if (totalMoveY > MIN_TRACKING_FOR_DRAG
           || totalMoveX > MIN_TRACKING_FOR_DRAG) {
         dragging = true;
-        firstDrag = true;
         dragDelegate.onDragStart(e);
       }
     }
=======================================
--- /trunk/samples/expenses/src/main/java/com/google/gwt/sample/expenses/client/ExpenseReportDetails.java Tue Apr 5 10:47:39 2011 +++ /trunk/samples/expenses/src/main/java/com/google/gwt/sample/expenses/client/ExpenseReportDetails.java Fri May 27 11:03:45 2011 
@@ -42,7 +42,6 @@
 import com.google.gwt.i18n.client.NumberFormat;
 import com.google.gwt.resources.client.ImageResource;
 import com.google.gwt.safehtml.client.SafeHtmlTemplates;
-import com.google.gwt.safehtml.client.SafeHtmlTemplates.Template;
 import com.google.gwt.safehtml.shared.SafeHtml;
 import com.google.gwt.safehtml.shared.SafeHtmlBuilder;
 import com.google.gwt.safehtml.shared.SafeHtmlUtils;
=======================================
--- /trunk/samples/expenses/src/main/java/com/google/gwt/sample/expenses/client/ExpenseTree.java Tue Apr 5 10:47:39 2011 +++ /trunk/samples/expenses/src/main/java/com/google/gwt/sample/expenses/client/ExpenseTree.java Fri May 27 11:03:45 2011 
@@ -22,7 +22,6 @@
 import com.google.gwt.core.client.GWT;
 import com.google.gwt.dom.client.Style.Overflow;
 import com.google.gwt.safehtml.client.SafeHtmlTemplates;
-import com.google.gwt.safehtml.client.SafeHtmlTemplates.Template;
 import com.google.gwt.safehtml.shared.SafeHtml;
 import com.google.gwt.safehtml.shared.SafeHtmlBuilder;
 import com.google.gwt.sample.expenses.client.style.Styles;
=======================================
--- /trunk/samples/expenses/src/main/java/com/google/gwt/sample/expenses/client/MobileExpenseList.java Tue Apr 5 10:47:39 2011 +++ /trunk/samples/expenses/src/main/java/com/google/gwt/sample/expenses/client/MobileExpenseList.java Fri May 27 11:03:45 2011 
@@ -18,7 +18,6 @@
 import com.google.gwt.cell.client.AbstractCell;
 import com.google.gwt.core.client.GWT;
 import com.google.gwt.safehtml.client.SafeHtmlTemplates;
-import com.google.gwt.safehtml.client.SafeHtmlTemplates.Template;
 import com.google.gwt.safehtml.shared.SafeHtml;
 import com.google.gwt.safehtml.shared.SafeHtmlBuilder;
 import com.google.gwt.sample.expenses.shared.ExpenseProxy;
=======================================
--- /trunk/samples/expenses/src/main/java/com/google/gwt/sample/expenses/client/PhaseAnimation.java Wed Aug 18 18:29:03 2010 +++ /trunk/samples/expenses/src/main/java/com/google/gwt/sample/expenses/client/PhaseAnimation.java Fri May 27 11:03:45 2011 
@@ -59,12 +59,12 @@

     @Override
     protected T getDisplayedItem(int index) {
-      return cellList.getDisplayedItem(index);
+      return cellList.getVisibleItem(index);
     }

     @Override
     protected List<T> getDisplayedItems() {
-      return cellList.getDisplayedItems();
+      return cellList.getVisibleItems();
     }

     @Override
@@ -97,12 +97,12 @@

     @Override
     protected T getDisplayedItem(int index) {
-      return cellTable.getDisplayedItem(index);
+      return cellTable.getVisibleItem(index);
     }

     @Override
     protected List<T> getDisplayedItems() {
-      return cellTable.getDisplayedItems();
+      return cellTable.getVisibleItems();
     }

     @Override
=======================================
--- /trunk/samples/expenses/src/main/java/com/google/gwt/sample/expenses/client/SlidingPanel.java Tue Apr 5 10:47:39 2011 +++ /trunk/samples/expenses/src/main/java/com/google/gwt/sample/expenses/client/SlidingPanel.java Fri May 27 11:03:45 2011 
@@ -90,6 +90,7 @@
    * Set the widget to show, adding it to the end of our sliding set if we
    * haven't seen it before. Nulls are ignored.
    */
+  @SuppressWarnings("deprecation")
   // Conflict btw deprecated Composite#setWidget and HasOneWidget#setWidget
   @Override
   public void setWidget(Widget widget) {
=======================================
--- /trunk/samples/expenses/src/main/java/com/google/gwt/sample/expenses/server/domain/EntityCounter.java Wed Aug 18 18:29:03 2010 +++ /trunk/samples/expenses/src/main/java/com/google/gwt/sample/expenses/server/domain/EntityCounter.java Fri May 27 11:03:45 2011 
@@ -47,7 +47,7 @@

   private static final Long ZERO = Long.valueOf(0L);

-  private static final boolean DENSE_IDS = false;
+//  private static final boolean DENSE_IDS = false;

   public static final EntityManager entityManager() {
     return EMF.get().createEntityManager();
@@ -167,10 +167,10 @@
             + mmin + " and id < " + mmax;
         Number count = (Number) em.createQuery(query).getSingleResult();
         long value = count.longValue();
-        if (value == 0 && DENSE_IDS) {
-          log.info("Got 0 results between " + mmin + " and " + mmax);
-          break;
-        }
+//        if (value == 0 && DENSE_IDS) {
+//          log.info("Got 0 results between " + mmin + " and " + mmax);
+//          break;
+//        }

         mmin = mmax;
         min = Long.valueOf(mmin);
=======================================
--- /trunk/samples/expenses/src/main/java/com/google/gwt/sample/gaerequest/client/GaeAuthRequestTransport.java Tue Apr 5 10:47:39 2011 +++ /trunk/samples/expenses/src/main/java/com/google/gwt/sample/gaerequest/client/GaeAuthRequestTransport.java Fri May 27 11:03:45 2011 
@@ -20,7 +20,6 @@
 import com.google.gwt.http.client.RequestCallback;
 import com.google.gwt.http.client.Response;
import com.google.web.bindery.requestfactory.gwt.client.DefaultRequestTransport; -import com.google.web.bindery.requestfactory.shared.RequestTransport.TransportReceiver; 
 import com.google.web.bindery.requestfactory.shared.ServerFailure;

 /**

--
http://groups.google.com/group/Google-Web-Toolkit-Contributors

Reply via email to