Update of /var/cvs/src/org/mmbase/bridge/util
In directory james.mmbase.org:/tmp/cvs-serv1977
Modified Files:
CloudThreadLocal.java
Log Message:
I don't see the point of wrapping every method in a private do-version
See also: http://cvs.mmbase.org/viewcvs/src/org/mmbase/bridge/util
Index: CloudThreadLocal.java
===================================================================
RCS file: /var/cvs/src/org/mmbase/bridge/util/CloudThreadLocal.java,v
retrieving revision 1.2
retrieving revision 1.3
diff -u -b -r1.2 -r1.3
--- CloudThreadLocal.java 17 Jul 2008 17:17:08 -0000 1.2
+++ CloudThreadLocal.java 9 Aug 2008 09:48:43 -0000 1.3
@@ -16,7 +16,7 @@
/**
* ThreadLocal to store an MMBase Bridge Cloud
* @since MMBase-1.9
- * @version $Id: CloudThreadLocal.java,v 1.2 2008/07/17 17:17:08 michiel Exp $
+ * @version $Id: CloudThreadLocal.java,v 1.3 2008/08/09 09:48:43 michiel Exp $
*/
public class CloudThreadLocal {
@@ -38,7 +38,9 @@
*/
public static void bind(Cloud cloud) {
cleanupAnyOrphanedCloud();
- doBind( cloud );
+ if ( cloud != null ) {
+ context.set( cloud );
+ }
}
/**
@@ -47,28 +49,19 @@
* @return The cloud which was unbound.
*/
public static Cloud unbind() {
- return doUnbind();
+ Cloud cloud = context.get();
+ if ( cloud != null ) {
+ context.set( null );
+ }
+ return cloud;
}
private static void cleanupAnyOrphanedCloud() {
- Cloud orphan = doUnbind();
+ Cloud orphan = unbind();
if ( orphan != null ) {
log.warn( "Already cloud bound on call to bind(); make sure you
clean up your cloud!" );
}
}
- private static void doBind(Cloud cloud) {
- if ( cloud != null ) {
- context.set( cloud );
- }
- }
-
- private static Cloud doUnbind() {
- Cloud cloud = context.get();
- if ( cloud != null ) {
- context.set( null );
- }
- return cloud;
- }
}
_______________________________________________
Cvs mailing list
[email protected]
http://lists.mmbase.org/mailman/listinfo/cvs
