<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
  <head>
    <meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
    <title></title>
  </head>
  <body bgcolor="#ffffff" text="#000000">
    Hi,<br>
    <br>
    well, the intended behaviour was:<br>
    setting 'if present': two events match if both have non empty
    episode names that match. If not, the events are handled as
    different, resulting in more eventually double recordings.<br>
    setting 'Yes': the match is achieved if the episode names match,
    also if both are empty. If the provider delivers no episode names
    for two different events, just one would be recorded.<br>
     <br>
    IMHO, it seems the code should actually look like this:<br>
    <br>
        if ((!compareTitle || Title1 == Title2)&amp;&amp;
    <br>
            (!compareSubtitle || (Subtitle1 == Subtitle2&amp;&amp;
    <br>
    (compareSubtitle==1 || Subtitle1!=""))))
    <br>
    <br>
    please note the "compareSubtitle==1" ('Yes') instead of
    "compareSubtitle==2 ('If present').<br>
    BTW, this was the code of my first commit on 2008-12-03, that I
    'corrected' for any reason to the current code on the same day:<br>
    <br>
<a class="moz-txt-link-freetext" href="http://projects.vdr-developer.org/git/?p=vdr-plugin-epgsearch.git;a=history;f=epgsearchtools.c;h=71a83a19c1bd4c89aa0c0c2808d216e30b03ca34;hb=ba87be9b2650796a507249c098eca815bff9b5e5">http://projects.vdr-developer.org/git/?p=vdr-plugin-epgsearch.git;a=history;f=epgsearchtools.c;h=71a83a19c1bd4c89aa0c0c2808d216e30b03ca34;hb=ba87be9b2650796a507249c098eca815bff9b5e5</a><br>
    <br>
    The documentation should also be fixed to:<br>
    <br>
    ...Caution: if your EPG data has no episode names for different
    episodes
    <br>
    don't use option 'yes'! 'if present' will then be the better choice
    even if this
    <br>
    results in double recordings.
    <br>
    <br>
    Probably it would be the best solution to drop the 'Yes' setting.<br>
    <br>
    BR,<br>
    Christian<br>
    <br>
    <br>
    Am 10.11.2010 14:43, schrieb Dominic Evans:
    <blockquote cite="mid:ibe7hr$sin$1@dough.gmane.org" type="cite">On
      10/11/10 13:25, Dominic Evans wrote:
      <br>
      <blockquote type="cite">I don't think its a bug, it seems to be
        the intended function.
        <br>
        <br>
        1) First it checks that we either said 'don't compare the title'
        or the
        <br>
        titles match (!compareTitle || Title1 == Title2)
        <br>
        <br>
        2a) Then it ANDs this with a check that either we said 'don't
        compare
        <br>
        the subtitle' (!compareSubtitle) OR the subtitles match
        (Subtitle1 ==
        <br>
        Subtitle2) AND also we've either said 'only if present'
        <br>
        (compareSubtitle==2) OR at least one of the sub-titles is
        non-empty
        <br>
        (Subtitle1!="")
        <br>
        <br>
        I just don't know why its a useful function :-)
        <br>
      </blockquote>
      <br>
      I did a bit more digging and discovered this was a new option
      since 0.9.25.beta7 and is listed in the HISTORY as:
      <br>
      <br>
      <blockquote type="cite">Avoid repeats: 'Compare subtitle' has now
        a third value 'if present'
        <br>
        besides 'no' and 'yes'. With this setting epgsearch will
        classify two
        <br>
        events only as equal if their episode names match and are not
        empty.
        <br>
        Caution: if your EPG data has no episode names for different
        episodes
        <br>
        don't use this option! 'yes' will then be the better choice even
        if this
        <br>
        results in double recordings.
        <br>
      </blockquote>
      <br>
      I think the description here is incorrect though. "epgsearch will
      classify two events only as equal if their episode names match and
      are not empty" seems to be the behaviour when this option is set
      to 'yes', not when it is set to the new 'if present' third value.
      The last sentence also seems to be incorrect. 'if-present' is the
      optimal choice if your EPG data doesn't always have episode names,
      using them for the comparison if they're present, ignoring them if
      they're missing.
      <br>
      <br>
      For anyone interested, I've attached the git-diff of 53677636
      (when this was introduced).
      <br>
      <br>
      <pre wrap="">
<fieldset class="mimeAttachmentHeader"></fieldset>
_______________________________________________
vdr mailing list
<a class="moz-txt-link-abbreviated" href="mailto:vdr@linuxtv.org">vdr@linuxtv.org</a>
<a class="moz-txt-link-freetext" href="http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr">http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr</a>
</pre>
    </blockquote>
    <br>
  </body>
</html>