[BUG] controld.cpp doppelte Oder-Abfrage

Wünsche, Anträge, Fehlermeldungen
GetAway
Contributor
Beiträge: 1509
Registriert: Donnerstag 27. Dezember 2007, 12:59

[BUG] controld.cpp doppelte Oder-Abfrage

Beitrag von GetAway »

Mir ist in controld.cpp Zeile 467 eine doppelte Abfrage aufgefallen.

Code: Alles auswählen

#ifdef HAVE_DBOX_HARDWARE
	if(format == CControld::FORMAT_RGB || format == CControld::FORMAT_YUV_VBS || format == CControld::FORMAT_YUV_VBS)
		setRGBCsync(settings.csync);
#endif
Es kann sich aber auch um ein Typo handeln, dann müsste es so aussehen.

Code: Alles auswählen

#ifdef HAVE_DBOX_HARDWARE
	if(format == CControld::FORMAT_RGB || format == CControld::FORMAT_YUV_VBS || format == CControld::FORMAT_YUV_CVBS)
		setRGBCsync(settings.csync);
#endif
Da sollte mal einer mit draufschauen, der sich besser mit controld auskennt.
Gaucho316
Contributor
Beiträge: 1688
Registriert: Donnerstag 17. Februar 2005, 20:24

Re: [BUG] controld.cpp doppelte Oder-Abfrage

Beitrag von Gaucho316 »

Das ist ein Typo. In video_setup.cpp wird in Zeile 147 auch auf RGB, VBS und CVBS getestet.
Gaucho316
Contributor
Beiträge: 1688
Registriert: Donnerstag 17. Februar 2005, 20:24

Re: [BUG] controld.cpp doppelte Oder-Abfrage

Beitrag von Gaucho316 »

Damit das nicht in Vergessenheit gerät, hier mal die nötige Änderung als Patch.

controld: fix video output setting
Link entfernt, da Patch im CVS
rhabarber1848
CDK-Experte
Beiträge: 4335
Registriert: Donnerstag 3. April 2008, 14:05

Re: [BUG] controld.cpp doppelte Oder-Abfrage

Beitrag von rhabarber1848 »

committed:
Gaucho316 hat geschrieben:controld_fix-video-output-setting_2012-08-03_1816.diff
http://article.gmane.org/gmane.comp.vid ... x.scm/3540