Changeset 1432


Ignore:
Timestamp:
09/07/06 09:14:43 (14 years ago)
Author:
curtis
Message:

Turn on indentation check, fix a few style issues in VisBio.

Location:
trunk
Files:
9 edited

Legend:

Unmodified
Added
Removed
  • trunk/checkstyle.xml

    r1389 r1432  
    150150      <property name="message" value="Line has trailing spaces."/> 
    151151    </module> 
    152 <!-- 
    153152    <module name="Indentation"> 
    154153      <property name="basicOffset" value="2"/> 
    155154      <property name="caseIndent" value="2"/> 
    156155    </module> 
    157 --> 
    158156    <module name="TodoComment"> 
    159157      <property name="format" value="TODO"/> 
  • trunk/loci/visbio/util/BreakawayPanel.java

    r1390 r1432  
    233233 
    234234  /** If breakaway panel is in a separate window, ensures it is visible. */ 
    235   public void reshow() { if (edge == null) window.setVisible(true); } 
     235  public void reshow() { 
     236    if (edge == null) window.setVisible(true); 
     237  } 
    236238 
    237239  /** Enlarges the window to its preferred size if it is too small. */ 
  • trunk/loci/visbio/util/DataUtil.java

    r1390 r1432  
    9999    int keep = 0; 
    100100    if (range != null) { 
    101       for (int i=0; i<range.length; i++) if (range[i]) keep++; 
     101      for (int i=0; i<range.length; i++) { 
     102        if (range[i]) keep++; 
     103      } 
    102104    } 
    103105 
  • trunk/loci/visbio/util/InstanceServer.java

    r1264 r1432  
    136136        notifyListeners(new SpawnEvent(args)); 
    137137      } 
    138       catch (IOException exc) { if (alive) exc.printStackTrace(); } 
     138      catch (IOException exc) { 
     139        if (alive) exc.printStackTrace(); 
     140      } 
    139141    } 
    140142  } 
  • trunk/loci/visbio/util/ObjectUtil.java

    r1390 r1432  
    8989    if (a1 == null || a2 == null) return a1 == null && a2 == null; 
    9090    if (a1.length != a2.length) return false; 
    91     for (int i=0; i<a1.length; i++) if (a1[i] != a2[i]) return false; 
     91    for (int i=0; i<a1.length; i++) { 
     92      if (a1[i] != a2[i]) return false; 
     93    } 
    9294    return true; 
    9395  } 
     
    9799    if (a1 == null || a2 == null) return a1 == null && a2 == null; 
    98100    if (a1.length != a2.length) return false; 
    99     for (int i=0; i<a1.length; i++) if (a1[i] != a2[i]) return false; 
     101    for (int i=0; i<a1.length; i++) { 
     102      if (a1[i] != a2[i]) return false; 
     103    } 
    100104    return true; 
    101105  } 
     
    105109    if (a1 == null || a2 == null) return a1 == null && a2 == null; 
    106110    if (a1.length != a2.length) return false; 
    107     for (int i=0; i<a1.length; i++) if (a1[i] != a2[i]) return false; 
     111    for (int i=0; i<a1.length; i++) { 
     112      if (a1[i] != a2[i]) return false; 
     113    } 
    108114    return true; 
    109115  } 
     
    113119    if (a1 == null || a2 == null) return a1 == null && a2 == null; 
    114120    if (a1.length != a2.length) return false; 
    115     for (int i=0; i<a1.length; i++) if (a1[i] != a2[i]) return false; 
     121    for (int i=0; i<a1.length; i++) { 
     122      if (a1[i] != a2[i]) return false; 
     123    } 
    116124    return true; 
    117125  } 
     
    121129    if (a1 == null || a2 == null) return a1 == null && a2 == null; 
    122130    if (a1.length != a2.length) return false; 
    123     for (int i=0; i<a1.length; i++) if (a1[i] != a2[i]) return false; 
     131    for (int i=0; i<a1.length; i++) { 
     132      if (a1[i] != a2[i]) return false; 
     133    } 
    124134    return true; 
    125135  } 
     
    129139    if (a1 == null || a2 == null) return a1 == null && a2 == null; 
    130140    if (a1.length != a2.length) return false; 
    131     for (int i=0; i<a1.length; i++) if (a1[i] != a2[i]) return false; 
     141    for (int i=0; i<a1.length; i++) { 
     142      if (a1[i] != a2[i]) return false; 
     143    } 
    132144    return true; 
    133145  } 
     
    137149    if (a1 == null || a2 == null) return a1 == null && a2 == null; 
    138150    if (a1.length != a2.length) return false; 
    139     for (int i=0; i<a1.length; i++) if (a1[i] != a2[i]) return false; 
     151    for (int i=0; i<a1.length; i++) { 
     152      if (a1[i] != a2[i]) return false; 
     153    } 
    140154    return true; 
    141155  } 
     
    145159    if (a1 == null || a2 == null) return a1 == null && a2 == null; 
    146160    if (a1.length != a2.length) return false; 
    147     for (int i=0; i<a1.length; i++) if (a1[i] != a2[i]) return false; 
     161    for (int i=0; i<a1.length; i++) { 
     162      if (a1[i] != a2[i]) return false; 
     163    } 
    148164    return true; 
    149165  } 
  • trunk/loci/visbio/view/DisplayWindow.java

    r1264 r1432  
    369369      // lay out components 
    370370      pane.add(display.getComponent(), BorderLayout.CENTER); 
    371       controls.setContentPane(FormsUtil.makeColumn(new Object[] { 
    372         viewHandler.getPanel(), FormsUtil.makeRow(new Object[] { 
    373         captureHandler.getPanel()}), "Data", transformHandler.getPanel(), 
    374         sliders}, null, true)); 
     371      Object[] rows = { 
     372        viewHandler.getPanel(), 
     373        FormsUtil.makeRow(new Object[] {captureHandler.getPanel()}), 
     374        "Data", 
     375        transformHandler.getPanel(), 
     376        sliders 
     377      }; 
     378      controls.setContentPane(FormsUtil.makeColumn(rows, null, true)); 
    375379      pack(); 
    376380      repack(); 
  • trunk/loci/visbio/view/TransformHandler.java

    r1264 r1432  
    289289    DataTransform[] dt = bsw.getTransforms(); 
    290290    int[] ndx = bsw.getIndices(); 
    291     for (int t=0; t<dt.length; t++) if (trans == dt[t]) return ndx[t]; 
     291    for (int t=0; t<dt.length; t++) { 
     292      if (trans == dt[t]) return ndx[t]; 
     293    } 
    292294    return -1; 
    293295  } 
  • trunk/loci/visbio/view/TransformPanel.java

    r1264 r1432  
    445445    builder.add(FormsUtil.makeRow(new Object[] {"Arrow keys:", 
    446446      "Le&ft/right", leftRightBox, "Up/do&wn", upDownBox}), cc.xy(1, 1)); 
    447     builder.add(FormsUtil.makeRow(new Object[] 
    448       {animate, "&FPS", fps, animBox}), cc.xy(1, 3)); 
     447    builder.add(FormsUtil.makeRow(new Object[] {animate, 
     448      "&FPS", fps, animBox}), cc.xy(1, 3)); 
    449449    return builder.getPanel(); 
    450450  } 
  • trunk/loci/visbio/view/ViewHandler.java

    r1264 r1432  
    392392    // clear old aspect ratio from projection matrix 
    393393    double[] undoOldAspect = { 
    394       1 / xasp,        0,        0, 0, 
    395              0, 1 / yasp,        0, 0, 
    396              0,        0, 1 / zasp, 0, 
    397              0,        0,        0, 1 
     394      1 / xasp, 0, 0, 0, 
     395      0, 1 / yasp, 0, 0, 
     396      0, 0, 1 / zasp, 0, 
     397      0, 0, 0, 1 
    398398    }; 
    399399    double[] newMatrix = display.multiply_matrix(oldMatrix, undoOldAspect); 
     
    422422    double rotate = window.is3D() ? DEFAULT_ROTATION : 0; 
    423423    double[] newMatrix = display.make_matrix(rotate, 0, 0, scale, 0, 0, 0); 
    424     double[] aspect = new double[] 
    425       {xasp, 0, 0, 0, 0, yasp, 0, 0, 0, 0, zasp, 0, 0, 0, 0, 1}; 
     424    double[] aspect = new double[] {xasp, 
     425      0, 0, 0, 0, yasp, 0, 0, 0, 0, zasp, 0, 0, 0, 0, 1}; 
    426426    return display.multiply_matrix(newMatrix, aspect); 
    427427  } 
Note: See TracChangeset for help on using the changeset viewer.