[mythtv] [Patch] Commercial free flag for channels

Matt Zimmerman mdz at debian.org
Mon Jan 19 23:59:53 EST 2004


On Mon, Jan 19, 2004 at 09:59:49PM -0500, J. Donavan Stanley wrote:

> +        const QString updates[] = {
> +	    QString("ALTER TABLE channel ADD COLUMN commfree TINYINT NOT NULL default '0';"),
> +	    QString("ALTER TABLE record  ADD COLUMN commflag TINYINT NOT NULL default '0';"),

This looks mostly sane, but what's commflag for?  It doesn't seem to be used
by your other changes.

> +    progMap["commfreee"] = chancommfree;

Also, isn't this one too many 'e's?

> +    int chancommfree;

Probably makes more sense as a bool, unless this is intended for something
more generic, in which case it should probably have a different name.

-- 
 - mdz


More information about the mythtv-dev mailing list