Ticket #10917: 0008-fix-braino-in-parser-of-delivery-system-descriptor.patch

File 0008-fix-braino-in-parser-of-delivery-system-descriptor.patch, 3.3 KB (added by dekarl@…, 12 years ago)

  • mythtv/libs/libmythtv/dtvmultiplex.cpp 

    From 59d1eb3595f46aa2bd5c381d295f88770deff3ce Mon Sep 17 00:00:00 2001
From: Karl Dietz <dekarl@users.sourceforge.net>
Date: Mon, 16 Jul 2012 07:41:45 +0200
Subject: [PATCH 8/8] fix braino in parser of delivery system descriptor

Use "a" instead of "auto" as default for DTVInversion to silence error.
Actualy show what inversion parameter is considered invalid.
Make both copies of the error message identical while here.

Fixes #10917
---
 mythtv/libs/libmythtv/dtvmultiplex.cpp |   14 ++++++++------
 1 files changed, 8 insertions(+), 6 deletions(-)

diff --git a/mythtv/libs/libmythtv/dtvmultiplex.cpp b/mythtv/libs/libmythtv/dtvmultiplex.cpp
index 86e476e..a7b8915 100644
    a b bool DTVMultiplex::ParseDVB_T( 
    198198    if (!ok)
    199199    {
    200200        LOG(VB_GENERAL, LOG_WARNING, LOC +
    201             "Invalid inversion, falling back to 'auto'.");
     201            QString("Invalid inversion parameter '%1', falling back to 'auto'.")
     202                .arg(_inversion));
    202203        ok = true;
    203204    }
    204205
    … … bool DTVMultiplex::ParseDVB_S_and_C( 
    224225    if (!ok)
    225226    {
    226227        LOG(VB_GENERAL, LOG_WARNING, LOC +
    227             "Invalid inversion, falling back to 'auto'");
     228            QString("Invalid inversion parameter '%1', falling back to 'auto'.")
     229                .arg(_inversion));
    228230
    229231        ok = true;
    230232    }
    … … bool DTVMultiplex::FillFromDeliverySystemDesc(DTVTunerType type, 
    398400            const TerrestrialDeliverySystemDescriptor cd(desc);
    399401
    400402            return ParseDVB_T(
    401                 QString().number(cd.FrequencyHz()), "auto",
     403                QString().number(cd.FrequencyHz()), "a",
    402404                cd.BandwidthString(),               cd.CodeRateHPString(),
    403405                cd.CodeRateLPString(),              cd.ConstellationString(),
    404406                cd.TransmissionModeString(),        cd.GuardIntervalString(),
    … … bool DTVMultiplex::FillFromDeliverySystemDesc(DTVTunerType type, 
    418420                }
    419421
    420422                return ParseDVB_S_and_C(
    421                     QString().number(cd.FrequencyHz()),  "auto",
     423                    QString().number(cd.FrequencyHz()),  "a",
    422424                    QString().number(cd.SymbolRateHz()), cd.FECInnerString(),
    423425                    cd.ModulationString(),
    424426                    cd.PolarizationString());
    … … bool DTVMultiplex::FillFromDeliverySystemDesc(DTVTunerType type, 
    427429            if (type == DTVTunerType::kTunerTypeDVBS2)
    428430            {
    429431                return ParseDVB_S2(
    430                     QString().number(cd.FrequencyHz()),  "auto",
     432                    QString().number(cd.FrequencyHz()),  "a",
    431433                    QString().number(cd.SymbolRateHz()), cd.FECInnerString(),
    432434                    cd.ModulationString(),
    433435                    cd.PolarizationString(),
    … … bool DTVMultiplex::FillFromDeliverySystemDesc(DTVTunerType type, 
    444446            const CableDeliverySystemDescriptor cd(desc);
    445447
    446448            return ParseDVB_S_and_C(
    447                     QString().number(cd.FrequencyHz()),  "auto",
     449                    QString().number(cd.FrequencyHz()),  "a",
    448450                    QString().number(cd.SymbolRateHz()), cd.FECInnerString(),
    449451                    cd.ModulationString(),               QString());
    450452        }