[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