Review (NS-110): Refix of bug 1712

Author

Kåre

Moderator

Kåre

State

Closed

Objectives

Reopened and refixed, due to a field being initialised in two
different places, and only as a synchronized list in one of them.
Made the fields final, to ensure it not happening again.
Synchronization on final static fields is really non-unittestable,
and non-releasetestable, so we need to make sure this is reviewed
properly :-)

Sorry about the fairly large changes, only very minor stuff has 
happened, the rest is automatic reformatting of code in the file.

The only relevant lines are:
65-66, 72-73, 385, and removed old lines 402-406

Summary

No comments

Total Time Used (Coding,Documentation,Review):

Time Used:
KFC: 0.1 md
SVC: 0.1 md

General comments:

Description

Classification

Status

IssuesFromNs110 (last edited 2010-08-16 10:24:49 by localhost)