bug6209: removed obsolete TODO

Axel Uhl commited 2026-02-20 17:43:32 +0100 commit 7acefefb5dc30d2916e1ff40271ccd0fd0e9041e

				java/com.sap.sailing.domain/src/com/sap/sailing/domain/tracking/impl/GPSFixTrackImpl.java
			
          @@ -619,7 +619,7 @@ public abstract class GPSFixTrackImpl<ItemType, FixType extends GPSFix> extends 

               @Override

               public SpeedWithBearing getEstimatedSpeed(TimePoint at) {

                   lockForRead();

          -        FixType ceil = getInternalFixes().ceiling(createDummyGPSFix(at)); // TODO bug6209: if "at" is the time point of an outlier, ceil would be the first valid fix *after* the outlier! We shouldn't be updating the fix at "at" but some later valid fix which, if asked at its timepoint, may have computed a different estimation altogether

          +        FixType ceil = getInternalFixes().ceiling(createDummyGPSFix(at));

                   try {

                       final SpeedWithBearing result;

                       if (ceil != null && ceil.getTimePoint().equals(at) && ceil.isEstimatedSpeedCached()) {

...	...	@@ -619,7 +619,7 @@ public abstract class GPSFixTrackImpl<ItemType, FixType extends GPSFix> extends
619	619	@Override
620	620	public SpeedWithBearing getEstimatedSpeed(TimePoint at) {
621	621	lockForRead();
622		- FixType ceil = getInternalFixes().ceiling(createDummyGPSFix(at)); // TODO bug6209: if "at" is the time point of an outlier, ceil would be the first valid fix after the outlier! We shouldn't be updating the fix at "at" but some later valid fix which, if asked at its timepoint, may have computed a different estimation altogether
	622	+ FixType ceil = getInternalFixes().ceiling(createDummyGPSFix(at));
623	623	try {
624	624	final SpeedWithBearing result;
625	625	if (ceil != null && ceil.getTimePoint().equals(at) && ceil.isEstimatedSpeedCached()) {