zapit.cpp

[GetAway]

Mich irritiert das dort ein break; fehlt.

Code: Alles auswählen

	case CZapitMessages::CMD_SET_SUBTITLE:
	{
		/* Zapit does NOT need this info. This is just a storage */
		CZapitMessages::commandSetSubtitle msgSetSubtitle;
		CBasicServer::receive_data(connfd, &msgSetSubtitle, sizeof(msgSetSubtitle));
		change_subtitle(msgSetSubtitle.index);

	}

[seife]

Ja, das ist falsch. Zum Glück wird der Code aber nie angesprungen:

Code: Alles auswählen

/local/seife/git/tuxbox-apps> git grep CMD_SET_SUBTITLE
dvb/zapit/include/zapit/client/msgtypes.h:                      CMD_SET_SUBTITLE                   = 81,
dvb/zapit/src/zapit.cpp:        case CZapitMessages::CMD_SET_SUBTITLE:

Deswegen ist es auch noch nie aufgefallen

[rhabarber1848]

erledigt: http://article.gmane.org/gmane.comp.vid ... x.scm/1111

[seife]

Es wäre auch diskussionswürdig solchen Code, der niemals benutzt wird, erstmals mittels "#ifdef UNUSED_CRAP" auszuklammern und wenn es keiner vermisst irgendwann komplett zu entfernen.

Sowas ähnliches gibt's im sectionsd, wo der Code zum debuggen noch eingeschaltet werden kann (aber das habe ich auch noch nie gemacht )