this protects against the NPE but it is wrong its going to mark all 'from' lines as high. that is, it will be saying that the revision that every line of a file was added is the revision that it was deleted. the problem is not in AgeColourer, its in ...

Classification: Factually incorrect
Ranking: Major
Requires Re-Review: Yes

Created by Nick Pellow from Geoff Crain's Crucible defect on AgeColourer.java lines 160-163 https://extranet.atlassian.com/crucible/cru/CR-FE-4687#c41164 :

this protects against the NPE but it is wrong

its going to mark all 'from' lines as high. that is, it will be saying that the revision that every line of a file was added is the revision that it was deleted.

the problem is not in AgeColourer, its in diffRenderer.tag - we are passing a null colourer into the renderer