[mythtv] Unexpected commDetectMethod: 255

[Daniel Kristjansson]

On Mon, 2008-05-19 at 17:58 -0700, Gregorio Gervasio, Jr. wrote:
> I believe rev 17318 changed the definition of COMM_DETECT_ALL
> from 255 to a bitwise-OR of the available methods (currently a value
> of 7).  When I run mythcommflag, I get the message in the subject
> line.  Although the recording is marked as "commflagged", no
> commercial breaks are detected.
> 
>         I can manually change the database setting to 7 and commercial
> flagging works as normal.  However, wouldn't this mean that the
> database setting should be updated each time a new method is added?
> 
>         Also, the CommercialSkipMethod setting in
> programs/mythfrontend/globalsettings.cpp still uses a hard-coded value
> of 255 for "All Available Methods".

Hmm, I missed the hard-coded values.

I'll work on a DB upgrade function that fixes this tomorrow.

Looks like the CommMethod class in channelsettings.cpp needs to be
synced as well.

-- Daniel