[mythtv] Re: MythChannels

Ben Bucksch linux.news at bucksch.org
Wed Jul 30 07:14:19 EDT 2003

+    if (s.freq < 100000) s.freq = s.freq * 1000; // freq was in MHz
+    if (s.symbol_rate < 100000) s.symbol_rate = s.symbol_rate * 1000;

Why do we need those? IMHO, that's a UI thing about how to enter 
numbers, and the DB field should have a simple and well-defined meaning 
like "freq in Hz", not "if it's < 100000, then it's freq in MHz, 
otherwise it's freq in Hz". The latter complicates all code places which 
read or write the field.

More information about the mythtv-dev mailing list