Revision: 9035
Author: [email protected]
Date: Tue Oct 12 14:43:01 2010
Log: synchronize ResourceOracleImpl.refresh()

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

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

Modified:
/trunk/dev/core/src/com/google/gwt/dev/resource/impl/ResourceOracleImpl.java

=======================================
--- /trunk/dev/core/src/com/google/gwt/dev/resource/impl/ResourceOracleImpl.java Tue Sep 21 04:21:01 2010 +++ /trunk/dev/core/src/com/google/gwt/dev/resource/impl/ResourceOracleImpl.java Tue Oct 12 14:43:01 2010
@@ -173,12 +173,15 @@

   /**
    * Rescans the associated paths to recompute the available resources.
-   *
+   *
+ * TODO(conroy,scottb): This synchronization could be improved upon to allow
+   * disjoint sets of oracles to be refreshed simultaneously.
+   *
    * @param logger status and error details are written here
    * @param first At least one ResourceOracleImpl must be passed to refresh
    * @param rest Callers may optionally pass several oracles
    */
-  public static void refresh(
+  public static synchronized void refresh(
TreeLogger logger, ResourceOracleImpl first, ResourceOracleImpl... rest) {
     int len = 1 + rest.length;
     ResourceOracleImpl[] oracles = new ResourceOracleImpl[1 + rest.length];

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

Reply via email to