Bug 369155

Description Patrick Tasse 2012-01-19 17:46:09 EST

In TmfExperiment, the methods seekLocation() and getNextEvent() use an experiment context that contains the location of each trace. The locations are copied from the trace context. When getting the next event of each trace, the trace context and its location can change. Since the locations in the experiment context are a copy of the location object reference, they can be modified when it is not desired. The solution is for the experiment context to clone the locations taken from the trace context.

The experiment context's location object is an instance of the class TmfLocationArray. This class does not implement equals() and hashCode() therefore the comparison of experiment context locations give incorrect results. The solution is to implement equals() and hashCode() in this class.

The class TmfRawEventViewer maintains a cache of parsed event text lines which contain the event's location. Since the locations in the line cache are a copy of the location object reference, they can be modified when it is not desired. The solution is for the line cache to clone the locations of each event.