Changeset 3510


Ignore:
Timestamp:
12/28/07 13:56:10 (12 years ago)
Author:
curtis
Message:

Remove some synchronization that is probably (?) not necessary.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/loci/plugins/browser/CacheIndicator.java

    r3506 r3510  
    6060    cacheLength = length; 
    6161    this.axis = axis; 
    62     synchronized (cache) { 
    63       numAxes = cache.getCurrentPos().length; 
    64     } 
     62 
     63    // NB: think more on whether this should be synchronized (cache) 
     64    numAxes = cache.getCurrentPos().length; 
     65 
    6566    doUpdate = true; 
    6667    repaint(); 
     
    8485      int[] currentPos = null; 
    8586      int[][] loadList = null; 
    86       synchronized (cache) { 
    87         currentPos = cache.getCurrentPos(); 
    88         loadList = cache.getStrategy().getLoadList(currentPos); 
    89       } 
     87 
     88      // NB: think more on whether this should be synchronized (cache) 
     89      currentPos = cache.getCurrentPos(); 
     90      loadList = cache.getStrategy().getLoadList(currentPos); 
    9091 
    9192      int[] pos = new int[currentPos.length]; 
     
    111112 
    112113        boolean inCache = false; 
    113         synchronized (cache) { 
    114           inCache = cache.isInCache(pos); 
    115         } 
     114 
     115        // NB: think more on whether this should be synchronized (cache) 
     116        inCache = cache.isInCache(pos); 
    116117 
    117118        if (inCache) g.setColor(Color.BLUE); 
Note: See TracChangeset for help on using the changeset viewer.