Yleisesti:

- Tehty ohjelman ilmoittamat korjaukset, jotka liittyivät seuraaviin asioihin:
	- Käyttämättömät muuttujat
	- Käyttämättömät metodit
	- Ilmoitetut virheet kommenteissa
	- Tutkittu ilmoitetut loogisuusvirheet

JTest:

- Muokattu JTest-ohjelmistoa sallimaan protected-tyypin käytön muuttujien 
  näkyvyydessä. Tämä tarvitaan sisäluokkien kanssa.
- Muokattu JTest-ohjelmistoa sallimaan samannimiset muuttujat sekä luokissa
  että metodeissa.
- Muokattu JTest-ohjelmistosta ehtoa, joka varoittaa liian monesta
  Case-lauseesta Switch-lausekkeessa.

UIManager:

- korvattu kaikki muuttujat, joiden näkyvyys oli määrittelemättä tai julkinen,
  näkyvyysarvolla private tai protected, riippuen siitä, oliko muuttujan 
  tarkoitus olla näkyvissä sisäluokista vai ei.
- Poistettu lähdekoodista käyttämättömiä muuttujia ja tarkastettu metodien 
  käyttömättömiä parametrejä. (1
- Muokattu JTest-ohjelmistoa olemaan huomauttamatta liian monesta metodin 
  kutsusta.
- Korjattu turhat true-vertaukset ehtolauseista
- Poistettu käyttämättömiä metodeita
	- ANode
	- Muitakin, joita en enää muista
- Korvattu undo() ja redo()-metodit olemassa olevilla doUndo() ja doRedo()-
  metodeilla, ja poistettu ko. metodit lähdekoodista.
- Poistettu vanhoja kommentteja ja kommentoituja koodilohkoja
- Parannettu ja muokattu JavaDoc-kommentteja


TreeManager & Siihen liittyvät luokat:

###############################################################################
       Report for class Element
       ------------------------

-------------------------------------------------------------------------------
Initial capacity is not defined.  (OPT.DIC-3)
    at [Element.java, line 147]
Korjattu 9.4.
-------------------------------------------------------------------------------
cloneTree: java.lang.NullPointerException
    at Element.cloneTree ()  [Element.java, line 107]
- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
For input:
    Element THIS = new Element (null, null);
    TreeNode RETVAL = THIS.cloneTree ();
Korjattu 9.4.
-------------------------------------------------------------------------------

###############################################################################
       Report for class TreeManager
       ----------------------------

-------------------------------------------------------------------------------
"static" field 'blocks' not initialized by "static" initializer  (INIT.SF-2)
    at [TreeManager.java, line 21]

Korjattu 9.4.
NYT:
private static Hashtable blocks;
static {
  blocks = new Hashtable(BLOCK_NAMES.length);
  for (int i = 0; i < BLOCK_NAMES.length; i++)
    blocks.put(BLOCK_NAMES[i], "");
}

OLI:
private static Hashtable blocks;
...
public TreeManager() {
  ...
  if (blocks == null) {
    block = new Hashtable(BLOCK_NAMES.length);
    for (int i = 0; i < BLOCK_NAMES.length; i++)
      blocks.put(BLOCK_NAMES[i], "");
  }
}

-------------------------------------------------------------------------------
Initial capacity is not defined.  (OPT.DIC-3)
    at [TreeManager.java, line 49]
Korjattu 9.4.
-------------------------------------------------------------------------------
Initial capacity is not defined.  (OPT.DIC-3)
    at [TreeManager.java, line 74]
Korjattu 9.4.
-------------------------------------------------------------------------------
Initial capacity is not defined.  (OPT.DIC-3)
    at [TreeManager.java, line 182]
Korjattu 9.4.
NYT:
newChildren = new Vector(oldChildren.capacity()-1);
-------------------------------------------------------------------------------
Initial capacity is not defined.  (OPT.DIC-3)
    at [TreeManager.java, line 248]
Korjattu 9.4.
NYT:
Vector newChildren = new Vector(oldChildren.capacity()+startChild-endChild);
-------------------------------------------------------------------------------
Initial capacity is not defined.  (OPT.DIC-3)
    at [TreeManager.java, line 266]
Korjattu 9.4.
NYT:
Vector encChildren = new Vector(1 + (4 * (endChild - startChild)) / 3);
-------------------------------------------------------------------------------
Initial capacity is not defined.  (OPT.DIC-3)
    at [TreeManager.java, line 335]
Korjattu 9.4.
NYT:
Vector v = new Vector(1);
-------------------------------------------------------------------------------
Initial capacity is not defined.  (OPT.DIC-3)
    at [TreeManager.java, line 351]
Korjattu 9.4.
NYT:
Vector v = new Vector(1);
-------------------------------------------------------------------------------
Initial capacity is not defined.  (OPT.DIC-3)
    at [TreeManager.java, line 375]
Korjattu 9.4.
NYT:
parent.setChildren(new Vector(0));
-------------------------------------------------------------------------------
Initial capacity is not defined.  (OPT.DIC-3)
    at [TreeManager.java, line 391]
Korjattu 9.4.
-------------------------------------------------------------------------------
Initial capacity is not defined.  (OPT.DIC-3)
    at [TreeManager.java, line 393]
Korjattu 9.4.
-------------------------------------------------------------------------------
Initial capacity is not defined.  (OPT.DIC-3)
    at [TreeManager.java, line 616]
Korjattu 9.4.
-------------------------------------------------------------------------------
VIRHE:
add- ja canAdd-metodit
NYT:
if (parent.getChildren() == null)
  parent.setChildren(new Vector(0));
int s = parent.getChildren().size();
return canReplace(parent, s, 0, s, 0, source);

OLI:
Vector v = parent.getChildren();
if (v == null)
  parent.setChildren(new Vector(0));
int s = v.size();
return canReplace(parent, s, 0, s, 0, source);
-------------------------------------------------------------------------------
Unused local variable: i  (UC.AUV-2)
    at [TreeManager.java, line 708]
Poistettu 9.4.
-------------------------------------------------------------------------------
Initial capacity is not defined.  (OPT.DIC-3)
    at [TreeManager.java, line 1138]
Korjattu 9.4.
-------------------------------------------------------------------------------
Initial capacity is not defined.  (OPT.DIC-3)
    at [TreeManager.java, line 1282]
Korjattu 9.4.
-------------------------------------------------------------------------------
Initial capacity is not defined.  (OPT.DIC-3)
    at [TreeManager.java, line 1365]
Korjattu 9.4.
-------------------------------------------------------------------------------
acceptChildren: java.lang.NullPointerException
    at TreeManager.acceptChildren (Element)  [TreeManager.java, line 1425]
- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
For input:
    Element t0 = new Element (null, null);
    TreeManager THIS = new TreeManager ();
    boolean RETVAL = THIS.acceptChildren (t0);
NYT:
String[] proto;
if (parent.getChildren() == null)
  proto = new String[0];
else
  proto = childrenToStringArray(parent.getChildren());
OLI:
String[] proto = childrenToStringArray(parent.getChildren());
-------------------------------------------------------------------------------

###############################################################################
       Report for class TreeSelection
       ------------------------------

-------------------------------------------------------------------------------
Unused local variable: parent  (UC.AUV-2)
    at [TreeSelection.java, line 16]
Poistettu 9.4.
-------------------------------------------------------------------------------


Body:

Yleiset korjaukset