Refactor:traffic/traff_android:remove TODO commenttraff_0_8

Signed-off-by: mvglasow <michael -at- vonglasow.com>

1 files changed, 8 insertions, 2 deletions

diff --git a/navit/android/src/org/navitproject/navit/NavitTraff.java b/navit/android/src/org/navitproject/navit/NavitTraff.java
index 621ad39a9..fd499b70b 100644
--- a/navit/android/src/org/navitproject/navit/NavitTraff.java
+++ b/navit/android/src/org/navitproject/navit/NavitTraff.java
@@ -220,8 +220,14 @@ public class NavitTraff extends BroadcastReceiver {
                 } else {
                     /* 0.7 feed */
                     String packageName = intent.getStringExtra(EXTRA_PACKAGE);
-                    /* if the feed comes from a TraFF 0.8+ source and we are subscribed, skip it */
-                    // TODO what if we don’t have a subscription yet? First subscribe, then poll (still no guarantee)
+                    /*
+                     * If the feed comes from a TraFF 0.8+ source and we are subscribed, skip it.
+                     * As a side effect of the current implementation, if a “bilingual” TraFF 0.7/0.8
+                     * source sends a broadcast feed before we have subscribed to it, we would process
+                     * the whole feed first, and then subscribe to a subset of that data.
+                     * If that turns out to be an issue, we would need to detect TraFF 0.8-capable
+                     * sources and discard broadcast feeds from them.
+                     */
                     if ((packageName != null) && subscriptions.containsValue(packageName))
                         return;
                     String feed = intent.getStringExtra(EXTRA_FEED);