Ticket #5292: libs_libmythtv-function-parameters-hidden-by-declarations.patch

File libs_libmythtv-function-parameters-hidden-by-declarations.patch, 7.3 KB (added by Erik Hovland <erik@…>, 18 years ago)

  • libs/libmythtv/osd.cpp 

    A function parameter can be hidden within the function if the function

From: Erik Hovland <erik@hovland.org>

declares a variable with the same name. This fixes any in the libmythtv
directory.
---

 libs/libmythtv/osd.cpp         |   10 ++++-----
 libs/libmythtv/siscan.cpp      |    6 +++--
 libs/libmythtv/videoout_xv.cpp |    8 +++----
 libs/libmythtv/viewschdiff.cpp |   46 ++++++++++++++++++++--------------------
 4 files changed, 35 insertions(+), 35 deletions(-)

diff --git a/libs/libmythtv/osd.cpp b/libs/libmythtv/osd.cpp
index 82c485a..e8715b5 100644
    a b void OSD::NewDialogBox(const QString &name, const QString &message,  
    21172117    OSDTypeText *text = NULL;
    21182118    do
    21192119    {
    2120         QString name = QString("option%1").arg(availoptions + 1);
    2121         text = (OSDTypeText *)container->GetType(name);
     2120        QString optionName = QString("option%1").arg(availoptions + 1);
     2121        text = (OSDTypeText *)container->GetType(optionName);
    21222122        if (text)
    21232123            availoptions++;
    21242124    }
    … … void OSD::NewDialogBox(const QString &name, const QString &message,  
    21382138
    21392139    for (int i = 1; i <= numoptions && i <= availoptions; i++)
    21402140    {
    2141         QString name = QString("option%1").arg(offset + i);
    2142         text = (OSDTypeText *)container->GetType(name);
     2141        QString optionName = QString("option%1").arg(offset + i);
     2142        text = (OSDTypeText *)container->GetType(optionName);
    21432143        if (!text)
    21442144        {
    2145             VERBOSE(VB_IMPORTANT, "Couldn't find: " << name);
     2145            VERBOSE(VB_IMPORTANT, "Couldn't find: " << optionName);
    21462146            return;
    21472147        }
    21482148

  • libs/libmythtv/siscan.cpp 

    diff --git a/libs/libmythtv/siscan.cpp b/libs/libmythtv/siscan.cpp
index eaf82e1..58411a2 100644
    a b bool SIScan::ScanTransport(int mplexid)  
    976976    while (query.next())
    977977    {
    978978        int sourceid = query.value(0).toInt();
    979         int mplexid  = query.value(1).toInt();
     979        int queriedMplexid  = query.value(1).toInt();
    980980        QString std  = query.value(2).toString();
    981981        int tsid     = query.value(3).toInt();
    982982
    983983        QString fn = (tsid) ? QString("Transport ID %1").arg(tsid) :
    984             QString("Multiplex #%1").arg(mplexid);
     984            QString("Multiplex #%1").arg(queriedMplexid);
    985985       
    986986        VERBOSE(VB_SIPARSER, LOC + "Adding " + fn);
    987987
    988         TransportScanItem item(sourceid, std, fn, mplexid, signalTimeout);
     988        TransportScanItem item(sourceid, std, fn, queriedMplexid, signalTimeout);
    989989        scanTransports += item;
    990990    }
    991991

  • libs/libmythtv/videoout_xv.cpp 

    diff --git a/libs/libmythtv/videoout_xv.cpp b/libs/libmythtv/videoout_xv.cpp
index 7a16e58..f86b4cc 100644
    a b void VideoOutputXv::InitDisplayMeasurements(uint width, uint height)  
    531531        float displayAspect = gContext->GetFloatSettingOnHost(
    532532            "XineramaMonitorAspectRatio",
    533533            gContext->GetHostName(), pixel_aspect);
    534         int height = display_dim.height();
    535         if (height <= 0)
    536             display_dim.setHeight(height = 300);
    537         display_dim.setWidth((int) round(height * displayAspect));
     534        int displayHeight = display_dim.height();
     535        if (displayHeight <= 0)
     536            display_dim.setHeight(displayHeight = 300);
     537        display_dim.setWidth((int) round(displayHeight * displayAspect));
    538538    }
    539539
    540540    VERBOSE(VB_PLAYBACK, LOC +

  • libs/libmythtv/viewschdiff.cpp 

    diff --git a/libs/libmythtv/viewschdiff.cpp b/libs/libmythtv/viewschdiff.cpp
index c321299..3dc45c3 100644
    a b void ViewScheduleDiff::updateList(QPainter *p)  
    512512                    break;
    513513
    514514                struct ProgramStruct s = recList[skip+i];
    515                 struct ProgramInfo *p = s.after;
    516                 if (!p) p = s.before;
     515                struct ProgramInfo *progInfo = s.after;
     516                if (!progInfo) progInfo = s.before;
    517517
    518518                QString temp;
    519519
    520                 temp = (p->recstartts).toString(dateformat);
    521                 temp += " " + (p->recstartts).toString(timeformat);
     520                temp = (progInfo->recstartts).toString(dateformat);
     521                temp += " " + (progInfo->recstartts).toString(timeformat);
    522522                ltype->SetItemText(i, 1, temp);
    523523
    524                 ltype->SetItemText(i, 2, p->ChannelText(channelFormat));
     524                ltype->SetItemText(i, 2, progInfo->ChannelText(channelFormat));
    525525
    526                 temp = p->title;
    527                 if ((p->subtitle).stripWhiteSpace().length() > 0)
    528                     temp += " - \"" + p->subtitle + "\"";
     526                temp = progInfo->title;
     527                if ((progInfo->subtitle).stripWhiteSpace().length() > 0)
     528                    temp += " - \"" + progInfo->subtitle + "\"";
    529529                ltype->SetItemText(i, 3, temp);
    530530
    531531                if (s.before) temp = s.before->RecStatusChar();
    … … void ViewScheduleDiff::updateList(QPainter *p)  
    541541
    542542                if (!s.after)
    543543                    ltype->EnableForcedFont(i, "disabledrecording");
    544                 else if (p->recstatus == rsRecording)
     544                else if (progInfo->recstatus == rsRecording)
    545545                    ltype->EnableForcedFont(i, "recording");
    546                 else if (p->recstatus == rsConflict ||
    547                          p->recstatus == rsOffLine ||
    548                          p->recstatus == rsAborted)
     546                else if (progInfo->recstatus == rsConflict ||
     547                         progInfo->recstatus == rsOffLine ||
     548                         progInfo->recstatus == rsAborted)
    549549                    ltype->EnableForcedFont(i, "conflictingrecording");
    550                 else if (p->recstatus == rsWillRecord)
     550                else if (progInfo->recstatus == rsWillRecord)
    551551                    ltype->EnableForcedFont(i, "record");
    552                 else if (p->recstatus == rsRepeat ||
    553                          p->recstatus == rsOtherShowing ||
    554                          (p->recstatus != rsDontRecord &&
    555                           p->recstatus <= rsEarlierShowing))
     552                else if (progInfo->recstatus == rsRepeat ||
     553                         progInfo->recstatus == rsOtherShowing ||
     554                         (progInfo->recstatus != rsDontRecord &&
     555                          progInfo->recstatus <= rsEarlierShowing))
    556556                    ltype->EnableForcedFont(i, "disabledrecording");
    557557            }
    558558        }
    … … void ViewScheduleDiff::updateInfo(QPainter *p)  
    619619    LayerSet *container = theme->GetSet("program_info");
    620620    if (container)
    621621    {
    622         ProgramInfo *p = CurrentProgram();
     622        ProgramInfo *progInfo = CurrentProgram();
    623623
    624         if (p)
     624        if (progInfo)
    625625        {
    626             p->ToMap(infoMap);
     626            progInfo->ToMap(infoMap);
    627627            container->ClearAllText();
    628628            container->SetText(infoMap);
    629629        }
    … … void ViewScheduleDiff::updateRecStatus(QPainter *p)  
    655655    LayerSet *container = theme->GetSet("status_info");
    656656    if (container)
    657657    {
    658         ProgramInfo *p = CurrentProgram();
    659         if (p)
     658        ProgramInfo *progInfo = CurrentProgram();
     659        if (progInfo)
    660660        {
    661             p->ToMap(infoMap);
     661            progInfo->ToMap(infoMap);
    662662            container->ClearAllText();
    663663            container->SetText(infoMap);
    664664        }