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