From e479a0b7552c3f944a4c8e527ae473426c85cdff Mon Sep 17 00:00:00 2001
From: Adam Sutton <dev@adamsutton.me.uk>
Date: Wed, 20 Jun 2012 14:44:55 +0100
Subject: [PATCH] Remove redundant TODOs and move interesting ones to github
 issues.

---
 src/dvr/dvr_autorec.c | 9 ---------
 src/dvr/dvr_db.c      | 3 ---
 2 files changed, 12 deletions(-)

diff --git a/src/dvr/dvr_autorec.c b/src/dvr/dvr_autorec.c
index 237f9128..f748f78d 100644
--- a/src/dvr/dvr_autorec.c
+++ b/src/dvr/dvr_autorec.c
@@ -95,10 +95,6 @@ autorec_cmp(dvr_autorec_entry_t *dae, epg_broadcast_t *e)
   }
 
   // Note: ignore channel test if we allow quality unlocking 
-  // TODO: should we only apply this setting if this is actually 
-  ///      created as a series link?
-  // TODO: I could just REMOVE the channel, but I think we probably still
-  //       want the channel as a "preferred" option
   cfg = dvr_config_find_by_name_default(dae->dae_config_name);
   if (cfg->dvr_sl_quality_lock)
     if(dae->dae_channel != NULL &&
@@ -561,7 +557,6 @@ dvr_autorec_add(const char *config_name,
                    NULL, NULL, 0, NULL, creator, comment);
 }
 
-/* TODO: configurable brand/series selection */
 void dvr_autorec_add_series_link 
   ( const char *dvr_config_name, epg_broadcast_t *event,
     const char *creator, const char *comment )
@@ -614,19 +609,15 @@ dvr_autorec_check_event(epg_broadcast_t *e)
 
 void dvr_autorec_check_brand(epg_brand_t *b)
 {
-#ifdef TODO_BRAND_UPDATED_SUPPORT
 // Note: for the most part this will only be relevant should an episode
 //       to which a broadcast is linked suddenly get added to a new brand
 //       this is pretty damn unlikely!
-#endif
 }
 
 void dvr_autorec_check_season(epg_season_t *s)
 {
-#ifdef TODO_SEASON_SUPPORT
 // Note: I guess new episodes might have been added, but again its likely
 //       this will already have been picked up by the check_event call
-#endif
 }
 
 /**
diff --git a/src/dvr/dvr_db.c b/src/dvr/dvr_db.c
index 40378ffb..b9a7e371 100644
--- a/src/dvr/dvr_db.c
+++ b/src/dvr/dvr_db.c
@@ -369,7 +369,6 @@ dvr_entry_create_by_event(const char *config_name,
 
 static int _dvr_duplicate_event ( epg_broadcast_t *e )
 {
-  // TODO: include other searches?
   dvr_entry_t *de;
   LIST_FOREACH(de, &dvrentries, de_global_link) {
     if (de->de_bcast && (de->de_bcast->episode == e->episode)) return 1;
@@ -386,7 +385,6 @@ dvr_entry_create_by_autorec(epg_broadcast_t *e, dvr_autorec_entry_t *dae)
   char buf[200];
 
   /* Dup detection */
-  // TODO: need to allow overrides
   if (_dvr_duplicate_event(e)) return;
 
   if(dae->dae_creator) {
@@ -838,7 +836,6 @@ dvr_entry_find_by_event_fuzzy(epg_broadcast_t *e)
 dvr_entry_t *
 dvr_entry_find_by_episode(epg_broadcast_t *e)
 {
-  // TODO: should be configurable?
   if (e->episode) {
     dvr_entry_t *de;
     epg_broadcast_t *ebc;