From 388e04bb6dcdf33ee26c2efe5163642fa17189c0 Mon Sep 17 00:00:00 2001
From: Johannes Schickel
Date: Thu, 29 Mar 2012 02:12:32 +0200
Subject: GUI: Add TODO/FIXME to PredictiveDialog::handleTickle.

---
 gui/predictivedialog.cpp | 4 ++++
 1 file changed, 4 insertions(+)

(limited to 'gui')

diff --git a/gui/predictivedialog.cpp b/gui/predictivedialog.cpp
index 4ce2a51979..238a832b10 100644
--- a/gui/predictivedialog.cpp
+++ b/gui/predictivedialog.cpp
@@ -534,6 +534,10 @@ void PredictiveDialog::processBtnActive(ButtonId button) {
 }
 
 void PredictiveDialog::handleTickle() {
+	// TODO/FIXME: This code does not seem to make any sense. It is only
+	// triggered when _lastTime is zero and sets _lastTime to zero again
+	// under some condition. This should really be a nop. Probably this
+	// code intends to check "_lastTime" instead of "!_lastTime".
 	if (!_lastTime) {
 		if ((_curTime - _lastTime) > kRepeatDelay) {
 			_lastTime = 0;
-- 
cgit v1.2.3