[mythtv-commits] Ticket #9824: [PATCH] MythPlayer: Fix playback of dvb-s radio channels
MythTV
noreply at mythtv.org
Thu Oct 13 08:24:32 UTC 2011
#9824: [PATCH] MythPlayer: Fix playback of dvb-s radio channels
-------------------------------------------------+-------------------------
Reporter: Lawrence Rust <lvr@…> | Owner:
Type: Patch - Bug Fix | Status: new
Priority: minor | Milestone: unknown
Component: MythTV - General | Version:
Severity: medium | 0.24-fixes
Keywords: | Resolution:
| Ticket locked: 0
-------------------------------------------------+-------------------------
Comment (by markk):
Lawrence - I've just started looking at these patches.
As a general observation (and this applies to various patches you've
submitted, not just these), your patches are difficult to review because
they contain various unnecessary and largely cosmetic changes that obscure
what you are really trying to achieve. For example, converting locks to
use QMutexLocker is fine but put them in a different patch.
Given the potential impact of some of these changes, it would also be much
appreciated if you explained what you are doing with various changes.
I'd suggest you try and cut them down to what is absolutely needed.
--
Ticket URL: <http://code.mythtv.org/trac/ticket/9824#comment:1>
MythTV <http://code.mythtv.org/trac>
MythTV Media Center
More information about the mythtv-commits
mailing list