id	summary	reporter	owner	description	type	status	priority	milestone	component	version	severity	resolution	keywords	cc	mlocked
1821	Reorganisation of ClassicCommDetector's logo detection and scene change detection	lucas@…	cpinkham	"this patch seperates the logo detection from !ClassicCommDetector, and puts it in its own class. I've done the same for the scene change detection, I'll open a seperate ticket with a seperate patch later on for that.

The new class, !ClassicLogoDetector, is currently still pretty intertwined with the !ClassicCommDetector. !ClassicCommDetector is putting pretty strong limitations on the implementation of a logodetector class, as in the current setup it gets one shot to search for a logo, and later on has to tell us if that logo is currently present in a  frame..   A logodetector has no freedom such that it might realize at the end of a recording that another logo is present, and that it would like to do another pass at the entire recording to look for this new logo again..    anyway, stuff like that are likely future improvements.. first things first.

This patch mantains functional equivalence to svn head. I've compared behaviour of snv head, and this patch on two recordings, and they had not only identical results, but identical verbose debug output."	patch	closed	minor	unknown	mythtv	head	low	fixed	mythcommflag		0