[mythtv] [mythtv-commits] mythtv commit: r14224 by nigel

Nigel Pearson nigel at ind.tansu.com.au
Mon Aug 20 03:37:15 UTC 2007


> Shouldn't these GetSetting()s have a second arg string for
> the default when there is no row in the 'settings' table?

And, looking at it more soberly, Bruce is right.
(e.g. GetDatabaseParams() does have default args)


The best fix would be to call that and store
the results early on, so that database settings
parsing is only done once.

--
Nigel Pearson, nigel at ind.tansu.com.au| All science is;
Telstra Net. Eng., Sydney, Australia | either Physics,
Office: 9202 3900    Fax:  9261 3912 | or stamp collecting!
Mobile: 0408 664435  Home: 9792 6998 |    - Rutherford




More information about the mythtv-dev mailing list