Correct member documentation in TofEvent

[Russell Taylor]

It states that TofEvent::m_tof in Events.h is in nanoseconds. This is not correct (it will be microseconds) and can confuse the uninitiated. The comments should also make clear that this is not necessarily a time-of-flight if the units have been changed.

I don't intend to go down the rabbit hole of changing the actual member name....

[Russell Taylor]

Re #5455. Change Events.h comments wrongly referring to nanoseconds.

Also act on a TODO about changing the TofEvent member variable visibility back to protected from public. This accounts for most of the changes as people had just used the members directly instead of accessor methods. A few places were using it to modify the members directly. I was able to change the logic to get around this in all cases except for LoadEventNexus.

Changeset: 87430b6c9032a6a5ddf46cfa59183b2de0728d66

[Russell Taylor]

Re #5455. Fix errors made in refactor yesterday.

Changeset: ea564818779e85e72957582328b35eedea27873c

[Russell Taylor]

Re #5455. Change Events.h comments wrongly referring to nanoseconds.

Also act on a TODO about changing the TofEvent member variable visibility back to protected from public. This accounts for most of the changes as people had just used the members directly instead of accessor methods. A few places were using it to modify the members directly. I was able to change the logic to get around this in all cases except for LoadEventNexus.

Changeset: 87430b6c9032a6a5ddf46cfa59183b2de0728d66

[Russell Taylor]

Re #5455. Fix errors made in refactor yesterday.

Changeset: ea564818779e85e72957582328b35eedea27873c

[Russell Taylor]

Re #5455. Change Events.h comments wrongly referring to nanoseconds.

Also act on a TODO about changing the TofEvent member variable visibility back to protected from public. This accounts for most of the changes as people had just used the members directly instead of accessor methods. A few places were using it to modify the members directly. I was able to change the logic to get around this in all cases except for LoadEventNexus.

Changeset: 87430b6c9032a6a5ddf46cfa59183b2de0728d66

[Russell Taylor]

Re #5455. Fix errors made in refactor yesterday.

Changeset: ea564818779e85e72957582328b35eedea27873c

[Stuart Campbell]

This ticket has been transferred to github issue ​6301