dropped option 'yes' in 'compare subtitle' when checking for a repeat and replaced it with 'if present'

9 files changed, 17 insertions, 30 deletions

diff --git a/HISTORY b/HISTORY
index 9c08678..cc2d87f 100644
--- a/HISTORY
+++ b/HISTORY
@@ -64,11 +64,9 @@ new:
   miss one only because of a buggy EPG.
 - searchtimers: if a timers filename results in an empty string or contains 
   "!^invalid^!" it will be skipped for programming now.
-- Avoid repeats: 'Compare subtitle' has now a third value 'if present' besides 'no'
-  and 'yes'. With this setting epgsearch will classify two events only as equal if 
-  their episode names match and are not empty. Caution: if your EPG data has no episode 
-  names for different episodes don't use this option! 'yes' will then be the better
-  choice even if this results in double recordings.
+- Avoid repeats: The option 'Yes' for 'Compare subtitle' is now replaced with 'if present'.
+  With this setting epgsearch will classify two events only as equal if 
+  their episode names match and are not empty. 
 - epgsearch now uses the shutdown handler (introduced in vdr 1.5.1) to prevent a 
   search timer update to be interrupted.
 - the SVDRP command UPDS for triggering search timer updates has now a new option 
diff --git a/HISTORY.DE b/HISTORY.DE
index 45dd4f6..4260408 100644
--- a/HISTORY.DE
+++ b/HISTORY.DE
@@ -65,12 +65,9 @@ neu:
   wird verhindert, dass man eine Folge nur wegen eines fehlerhaften EPGs verpasst.
 - Suchtimer: Falls der Datei-Eintrag eines Timers leer ist oder "!^invalid^!" enthält,
   wird er für die Programmierung übersprungen.
-- Wiederholung vermeiden: 'Vergleiche Untertitel' hat nun eine dritte Variante 'falls 
-  vorhanden' neben 'nein' und 'ja'. Damit stuft epgsearch zwei Sendungen nur dann als 
-  identisch ein, wenn die Episodennamen gleich sind und nicht leer. Vorsicht: Wenn die 
-  EPG-Daten keine Untertitel für unterschiedliche Folgen enthalten, sollte diese Option
-  nicht verwendet werden. 'ja' ist in diesem Fall besser, selbst wenn dadurch doppelt 
-  aufgezeichnet werden kann.
+- Wiederholung vermeiden: Die Option 'ja' bei 'Vergleiche Untertitel' wurde durch 'falls 
+  vorhanden' ersetzt. Damit stuft epgsearch zwei Sendungen nur dann als 
+  identisch ein, wenn die Episodennamen gleich sind und nicht leer.
 - epgsearch verwendet nun den shutdown handler (eingeführt in vdr 1.5.1) um die Unter-
   brechung eines Suchtimerupdates zu verhindern.
 - das SVDRP-Kommando UPDS zum Auslösen von Suchtimer-Updates hat nun eine neue Option
diff --git a/doc-src/de/epgsearch.1.txt b/doc-src/de/epgsearch.1.txt
index 874b37e..4d465ee 100644
--- a/doc-src/de/epgsearch.1.txt
+++ b/doc-src/de/epgsearch.1.txt
@@ -345,12 +345,8 @@ Titel verglichen werden soll.
 =item - B<Vergleiche Untertitel>
 
 Einstellung, ob beim Test, ob eine Sendung identisch ist, auch der
-Untertitel verglichen werden soll. Neben 'nein' und 'ja' existiert noch
-'falls vorhanden'. Damit stuft epgsearch zwei Sendungen nur dann als 
-identisch ein, wenn die Episodennamen gleich sind und nicht leer. Vorsicht: Wenn die 
-EPG-Daten keine Untertitel für unterschiedliche Folgen enthalten, sollte diese Option
-nicht verwendet werden. 'ja' ist in diesem Fall besser, selbst wenn dadurch doppelt 
-aufgezeichnet werden kann.
+Untertitel verglichen werden soll. Bei 'falls vorhanden' stuft epgsearch zwei Sendungen nur dann als 
+identisch ein, wenn die Episodennamen gleich sind und nicht leer. 
 
 =item - B<Vergleiche Beschreibung>
 
diff --git a/doc-src/en/epgsearch.1.txt b/doc-src/en/epgsearch.1.txt
index 6198885..f18c163 100644
--- a/doc-src/en/epgsearch.1.txt
+++ b/doc-src/en/epgsearch.1.txt
@@ -340,11 +340,8 @@ compared.
 =item - B<Compare subtitle:>
 
 When comparing to events then specify here if the subtitle should be
-compared. Besides 'no' and 'yes' there is also the option 'if present'.
-With this setting epgsearch will classify two events only as equal if 
-their episode names match and are not empty. Caution: if your EPG data has no episode 
-names for different episodes don't use this option! 'yes' will then be the better
-choice even if this results in double recordings.
+compared. With 'if present' epgsearch will classify two events only as equal if 
+their episode names match and are not empty. 
 
 =item - B<Compare description:>
 
diff --git a/epgsearchext.c b/epgsearchext.c
index fd269e8..3f160a7 100644
--- a/epgsearchext.c
+++ b/epgsearchext.c
@@ -558,7 +558,7 @@ bool cSearchExt::Parse(const char *s)
                   break;
                case 31: compareTitle = atoi(value);
                   break;
-               case 32: compareSubtitle = atoi(value);
+	       case 32: compareSubtitle = atoi(value)>0?1:0;
                   break;
                case 33: compareSummary = atoi(value);
                   break;
diff --git a/epgsearchtools.c b/epgsearchtools.c
index a5ac3b2..d36cd15 100644
--- a/epgsearchtools.c
+++ b/epgsearchtools.c
@@ -822,7 +822,7 @@ bool EventsMatch(const cEvent* event1, const cEvent* event2, bool compareTitle,
 
    bool match = false;
    if ((!compareTitle || Title1 == Title2) &&
-       (!compareSubtitle || (Subtitle1 == Subtitle2 && (compareSubtitle==2 || Subtitle1!=""))))
+       (!compareSubtitle || (Subtitle1 == Subtitle2 && Subtitle1!="")))
    {	
       const char* Descr1    = event1->Description();
       const char* Descr2    = event2->Description();
diff --git a/menu_searchedit.c b/menu_searchedit.c
index 0159843..ecc707f 100644
--- a/menu_searchedit.c
+++ b/menu_searchedit.c
@@ -91,8 +91,7 @@ cMenuEditSearchExt::cMenuEditSearchExt(cSearchExt *SearchExt, bool New, bool Tem
    SearchActiveModes[2] = strdup(tr("user-defined"));
 
    CompareSubtitleModes[0] = strdup(trVDR("no"));
-   CompareSubtitleModes[1] = strdup(trVDR("yes"));
-   CompareSubtitleModes[2] = strdup(tr("if present"));
+   CompareSubtitleModes[1] = strdup(tr("if present"));
 
 #if APIVERSNUM > 10710 
    // collect content string IDs
@@ -370,7 +369,7 @@ void cMenuEditSearchExt::Set()
             if (data.allowedRepeats > 0)
 	      Add(new cMenuEditIntItem(IndentMenuItem(tr("Only repeats within ... days"),2), &data.repeatsWithinDays, 0, 999));
             Add(new cMenuEditBoolItem(IndentMenuItem(tr("Compare title"),2), &data.compareTitle, trVDR("no"), trVDR("yes")));
-		Add(new cMenuEditStraItem(IndentMenuItem(tr("Compare subtitle"),2), &data.compareSubtitle, 3, CompareSubtitleModes));
+		Add(new cMenuEditStraItem(IndentMenuItem(tr("Compare subtitle"),2), &data.compareSubtitle, 2, CompareSubtitleModes));
 		Add(new cMenuEditBoolItem(IndentMenuItem(tr("Compare summary"),2), &data.compareSummary, trVDR("no"), trVDR("yes")));
 	    if (data.compareSummary)
 	      Add(new cMenuEditIntItem(IndentMenuItem(tr("Min. match in %"),3), &data.compareSummaryMatchInPercent, 1, 100));
@@ -441,7 +440,7 @@ cMenuEditSearchExt::~cMenuEditSearchExt()
       free(DelModes[i]);
    for(i=0; i<=2; i++)
       free(SearchActiveModes[i]);
-   for(i=0; i<=2; i++)
+   for(i=0; i<=1; i++)
       free(CompareSubtitleModes[i]);
 }
 
diff --git a/menu_searchedit.h b/menu_searchedit.h
index 0a2e5ed..cb11672 100644
--- a/menu_searchedit.h
+++ b/menu_searchedit.h
@@ -52,7 +52,7 @@ protected:
   char *BlacklistModes[3];
   char *DelModes[3];
   char *SearchActiveModes[3];
-  char *CompareSubtitleModes[3];
+  char *CompareSubtitleModes[2];
   std::vector<int> contentStringIDs;
   int useContentDescriptors;
   int *contentStringsFlags;
diff --git a/recdone.c b/recdone.c
index eac59b8..7556c5b 100644
--- a/recdone.c
+++ b/recdone.c
@@ -295,7 +295,7 @@ int cRecsDone::GetCountRecordings(const cEvent* event, bool compareTitle, int co
       }
 
       if ((!compareTitle || rTitle == eTitle) &&
-          (!compareSubtitle || (rSubtitle == eSubtitle && (compareSubtitle==2 || rSubtitle !=""))) &&
+          (!compareSubtitle || (rSubtitle == eSubtitle && rSubtitle !="")) &&
           (!compareSummary || DescriptionMatches(eRawDescr.c_str(), rRawDescr.c_str(), matchLimit)))
       {
          if (catvaluesAvoidRepeat != 0) // check categories