frontend: use const char pointers in getDelSys()

this avoids a lot of ugly casts, and the pointer targets were
const char* anyway


Origin commit data
------------------
Branch: ni/coolstream
Commit: cc24b2cf65
Author: Stefan Seyfried <seife@tuxbox-git.slipkontur.de>
Date: 2015-02-09 (Mon, 09 Feb 2015)



------------------
This commit was generated by Migit
This commit is contained in:
Stefan Seyfried
2015-02-09 20:30:06 +01:00
parent ff703b2915
commit 7b52e64295
4 changed files with 41 additions and 41 deletions

View File

@@ -124,7 +124,7 @@ void CScanTs::testFunc()
{ {
int w = x + width - xpos2; int w = x + width - xpos2;
char buffer[128]; char buffer[128];
char *f, *s, *m, *f2; const char *f, *s, *m, *f2;
if (CFrontend::isSat(delsys)) { if (CFrontend::isSat(delsys)) {
CFrontend::getDelSys(TP.feparams.delsys, TP.feparams.fec_inner, TP.feparams.modulation, f, s, m); CFrontend::getDelSys(TP.feparams.delsys, TP.feparams.fec_inner, TP.feparams.modulation, f, s, m);
@@ -412,7 +412,7 @@ neutrino_msg_t CScanTs::handleMsg(neutrino_msg_t msg, neutrino_msg_data_t data)
case NeutrinoMessages::EVT_SCAN_REPORT_FREQUENCYP: case NeutrinoMessages::EVT_SCAN_REPORT_FREQUENCYP:
{ {
FrontendParameters *feparams = (FrontendParameters*) data; FrontendParameters *feparams = (FrontendParameters*) data;
char * f, *s, *m; const char *f, *s, *m;
CFrontend::getDelSys(feparams->delsys, feparams->fec_inner, feparams->modulation, f, s, m); CFrontend::getDelSys(feparams->delsys, feparams->fec_inner, feparams->modulation, f, s, m);
uint32_t freq = feparams->frequency/1000; uint32_t freq = feparams->frequency/1000;

View File

@@ -207,7 +207,7 @@ class CFrontend
uint16_t getSignalStrength(void) const; uint16_t getSignalStrength(void) const;
fe_status_t getStatus(void) const; fe_status_t getStatus(void) const;
uint32_t getUncorrectedBlocks(void) const; uint32_t getUncorrectedBlocks(void) const;
void getDelSys(int f, int m, char * &fec, char * &sys, char * &mod); void getDelSys(int f, int m, const char * &fec, const char * &sys, const char * &mod);
void getFEInfo(void); void getFEInfo(void);
int32_t getCurrentSatellitePosition() { return currentSatellitePosition; } int32_t getCurrentSatellitePosition() { return currentSatellitePosition; }
@@ -272,7 +272,7 @@ class CFrontend
satellite_map_t & getSatellites() { return satellites; } satellite_map_t & getSatellites() { return satellites; }
void setSatellites(satellite_map_t satmap) { satellites = satmap; } void setSatellites(satellite_map_t satmap) { satellites = satmap; }
int getNumber() { return fenumber; }; int getNumber() { return fenumber; };
static void getDelSys(delivery_system_t delsys, int f, int m, char * &fec, char * &sys, char * &mod); static void getDelSys(delivery_system_t delsys, int f, int m, const char * &fec, const char * &sys, const char * &mod);
fe_work_mode_t getMode() { return femode; } fe_work_mode_t getMode() { return femode; }
void setMode(int mode) {femode = (fe_work_mode_t) mode; } void setMode(int mode) {femode = (fe_work_mode_t) mode; }
int getMaster() { return masterkey; } int getMaster() { return masterkey; }

View File

@@ -737,7 +737,7 @@ struct dvb_frontend_event CFrontend::getEvent(void)
return event; return event;
} }
void CFrontend::getDelSys(int f, int m, char *&fec, char *&sys, char *&mod) void CFrontend::getDelSys(int f, int m, const char *&fec, const char *&sys, const char *&mod)
{ {
return getDelSys(getCurrentDeliverySystem(), f, m, fec, sys, mod); return getDelSys(getCurrentDeliverySystem(), f, m, fec, sys, mod);
} }
@@ -811,25 +811,25 @@ void CFrontend::getXMLDelsysFEC(fe_code_rate_t xmlfec, delivery_system_t & delsy
} }
} }
void CFrontend::getDelSys(delivery_system_t delsys, int f, int m, char *&fec, char *&sys, char *&mod) void CFrontend::getDelSys(delivery_system_t delsys, int f, int m, const char *&fec, const char *&sys, const char *&mod)
{ {
switch (delsys) { switch (delsys) {
case DVB_S: case DVB_S:
sys = (char *)"DVB"; sys = "DVB";
mod = (char *)"QPSK"; mod = "QPSK";
break; break;
case DVB_S2: case DVB_S2:
sys = (char *)"DVB-S2"; sys = "DVB-S2";
switch (m) { switch (m) {
case QPSK: case QPSK:
mod = (char *)"QPSK"; mod = "QPSK";
break; break;
case PSK_8: case PSK_8:
mod = (char *)"8PSK"; mod = "8PSK";
break; break;
default: default:
printf("[frontend] unknown modulation %d!\n", m); printf("[frontend] unknown modulation %d!\n", m);
mod = (char *)"UNKNOWN"; mod = "UNKNOWN";
} }
break; break;
case DVB_C: case DVB_C:
@@ -837,103 +837,103 @@ void CFrontend::getDelSys(delivery_system_t delsys, int f, int m, char *&fec, ch
case DTMB: case DTMB:
switch(delsys) { switch(delsys) {
case DVB_C: case DVB_C:
sys = (char *)"DVB-C(Annex A)"; sys = "DVB-C(Annex A)";
break; break;
case DVB_T: case DVB_T:
sys = (char *)"DVB-T"; sys = "DVB-T";
break; break;
case DVB_T2: case DVB_T2:
sys = (char *)"DVB-T2"; sys = "DVB-T2";
break; break;
case DTMB: case DTMB:
sys = (char *)"DTMB"; sys = "DTMB";
break; break;
default: default:
printf("[frontend] unknown delsys %d!\n", delsys); printf("[frontend] unknown delsys %d!\n", delsys);
sys = (char *)"UNKNOWN"; sys = "UNKNOWN";
break; break;
} }
switch (m) { switch (m) {
case QAM_16: case QAM_16:
mod = (char *)"QAM_16"; mod = "QAM_16";
break; break;
case QAM_32: case QAM_32:
mod = (char *)"QAM_32"; mod = "QAM_32";
break; break;
case QAM_64: case QAM_64:
mod = (char *)"QAM_64"; mod = "QAM_64";
break; break;
case QAM_128: case QAM_128:
mod = (char *)"QAM_128"; mod = "QAM_128";
break; break;
case QAM_256: case QAM_256:
mod = (char *)"QAM_256"; mod = "QAM_256";
break; break;
#if _HAVE_DVB57 #if _HAVE_DVB57
case QAM_4_NR: case QAM_4_NR:
mod = (char *)"QAM_4_NR"; mod = "QAM_4_NR";
break; break;
#endif #endif
case QPSK: case QPSK:
if (delsys == DVB_T || delsys == DVB_T2 || delsys == DTMB) { if (delsys == DVB_T || delsys == DVB_T2 || delsys == DTMB) {
mod = (char *)"QPSK"; // AKA QAM_4 mod = "QPSK"; // AKA QAM_4
break; break;
} }
/* fallthrouh for FE_QAM... */ /* fallthrouh for FE_QAM... */
case QAM_AUTO: case QAM_AUTO:
default: default:
mod = (char *)"QAM_AUTO"; mod = "QAM_AUTO";
break; break;
} }
break; break;
default: default:
INFO("unknown delsys %d!", delsys); INFO("unknown delsys %d!", delsys);
sys = (char *)"UNKNOWN"; sys = "UNKNOWN";
mod = (char *)"UNKNOWN"; mod = "UNKNOWN";
break; break;
} }
switch (f) { switch (f) {
case FEC_1_2: case FEC_1_2:
fec = (char *)"1/2"; fec = "1/2";
break; break;
case FEC_2_3: case FEC_2_3:
fec = (char *)"2/3"; fec = "2/3";
break; break;
case FEC_3_4: case FEC_3_4:
fec = (char *)"3/4"; fec = "3/4";
break; break;
case FEC_4_5: case FEC_4_5:
fec = (char *)"4/5"; fec = "4/5";
break; break;
case FEC_5_6: case FEC_5_6:
fec = (char *)"5/6"; fec = "5/6";
break; break;
case FEC_6_7: case FEC_6_7:
fec = (char *)"6/7"; fec = "6/7";
break; break;
case FEC_7_8: case FEC_7_8:
fec = (char *)"7/8"; fec = "7/8";
break; break;
case FEC_8_9: case FEC_8_9:
fec = (char *)"8/9"; fec = "8/9";
break; break;
case FEC_3_5: case FEC_3_5:
fec = (char *)"3/5"; fec = "3/5";
break; break;
case FEC_9_10: case FEC_9_10:
fec = (char *)"9/10"; fec = "9/10";
break; break;
#if _HAVE_DVB57 #if _HAVE_DVB57
case FEC_2_5: case FEC_2_5:
fec = (char *)"2/5"; fec = "2/5";
break; break;
#endif #endif
default: default:
INFO("unknown FEC: %d!", f); INFO("unknown FEC: %d!", f);
case FEC_AUTO: case FEC_AUTO:
fec = (char *)"AUTO"; fec = "AUTO";
break; break;
} }
} }
@@ -1597,7 +1597,7 @@ int CFrontend::setParameters(transponder *TP, bool nowait)
freq = (int) feparams.frequency; freq = (int) feparams.frequency;
char * f, *s, *m; const char *f, *s, *m;
bool high_band; bool high_band;
switch (feparams.delsys) { switch (feparams.delsys) {

View File

@@ -198,7 +198,7 @@ char transponder::pol(unsigned char p)
std::string transponder::description() std::string transponder::description()
{ {
char buf[128] = {0}; char buf[128] = {0};
char *f, *s, *m, *f2; const char *f, *s, *m, *f2;
if (CFrontend::isSat(feparams.delsys)) { if (CFrontend::isSat(feparams.delsys)) {
CFrontend::getDelSys(feparams.delsys, getFEParams()->fec_inner, getFEParams()->modulation, f, s, m); CFrontend::getDelSys(feparams.delsys, getFEParams()->fec_inner, getFEParams()->modulation, f, s, m);