From patchwork Mon Mar 2 11:57:53 2009 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: jori.hamalainen@teliasonera.com X-Patchwork-Id: 12706 Received: from sehan5453ashub2.han.telia.se ([131.115.18.199]) by www.linuxtv.org with esmtp (Exim 4.63) (envelope-from ) id 1Le6mn-0003Sh-An for vdr@linuxtv.org; Mon, 02 Mar 2009 12:58:19 +0100 Received: from sehan001bb.han.telia.se (131.115.18.152) by sehan5453ashub2.tcad.telia.se (131.115.18.199) with Microsoft SMTP Server id 8.1.340.0; Mon, 2 Mar 2009 12:58:13 +0100 Received: from SEHAN022MB.tcad.telia.se ([131.115.18.163]) by sehan001bb.han.telia.se with Microsoft SMTPSVC(6.0.3790.1830); Mon, 2 Mar 2009 12:58:12 +0100 X-MimeOLE: Produced By Microsoft Exchange V6.5 Content-Class: urn:content-classes:message MIME-Version: 1.0 Date: Mon, 2 Mar 2009 12:57:53 +0100 Message-ID: <9D4CAB5E5B58B743BC96CB3705D3D8CB0551EDAA@SEHAN022MB.tcad.telia.se> In-Reply-To: <49ABB0B1.1000803@pp.nic.fi> X-MS-Has-Attach: yes X-MS-TNEF-Correlator: Thread-Topic: [vdr] Vote your VDR patches thread-index: AcmbHzsWscb8ZSbuS/qX6VduYIW0QQADauTw References: <23120804.946321235773543484.JavaMail.servlet@kundenserver><9D4CAB5E5B58B743BC96CB3705D3D8CB0551ED71@SEHAN022MB.tcad.telia.se> <49ABB0B1.1000803@pp.nic.fi> From: To: X-OriginalArrivalTime: 02 Mar 2009 11:58:12.0630 (UTC) FILETIME=[29A5C360:01C99B2E] X-LSpam-Score: -4.2 (----) X-LSpam-Report: No, score=-4.2 required=5.0 tests=AWL=-1.632, BAYES_00=-2.599 autolearn=ham Subject: Re: [vdr] Vote your VDR patches X-BeenThere: vdr@linuxtv.org X-Mailman-Version: 2.1.9 Precedence: list Reply-To: VDR Mailing List List-Id: VDR Mailing List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 02 Mar 2009 11:58:19 -0000 Status: O X-Status: X-Keywords: X-UID: 19685 >> I think one important patch is missing for DVB subtitles users, which >> makes old vdr-subtitles -plugin's recorded subtitles visible also for >> new subtitles by Klaus. > Is this included in liemikuutio? I don't know as I don't use liemikuutio. But I downloaded the liemikuutio- patch and saw the above patch for device.c which is the same. So the answer is yes, it is included. LanguagePreference) || + ((i == ttSubtitleFirst + 8) && !(*TrackId->language) && (LanguagePreference == INT_MAX)))) PreferredTrack = eTrackType(i); } // Make sure we're set to an available subtitle track: diff -Nru vdr-1.6.0-vanilla/device.c vdr-1.6.0-liemikuutio/device.c --- vdr-1.6.0-vanilla/device.c 2008-03-09 12:03:34.000000000 +0200 +++ vdr-1.6.0-liemikuutio/device.c 2009-02-27 23:06:59.000000000 +0200 @@ -1100,7 +1100,8 @@ int LanguagePreference = INT_MAX; // higher than the maximum possible value for (int i = ttSubtitleFirst; i <= ttSubtitleLast; i++) { const tTrackId *TrackId = GetTrack(eTrackType(i)); - if (TrackId && TrackId->id && I18nIsPreferredLanguage(Setup.SubtitleLanguages, TrackId->language, LanguagePreference)) + if (TrackId && TrackId->id && (I18nIsPreferredLanguage(Setup.SubtitleLanguages, TrackId->language,