Opened 11 years ago

Closed 11 years ago

#6688 closed defect (fixed)

Channel scanner includes logging to cerr

Reported by: Nick Morrott <knowledgejunkie (at) gmail (dot) com> Owned by: danielk
Priority: minor Milestone: unknown
Component: MythTV - Channel Scanner Version: head
Severity: medium Keywords: cerr verbose logging
Cc: Ticket locked: no

Description

The following scanner methods include logging calls to cerr. Might it be useful to have them (or at least the first method whose cerr logging does not appear to be serious) log to a log target instead (perhaps VB_CHANSCAN)?

bool ChannelScanSM::TestNextProgramEncryption(void)


void ChannelScannerCLI::HandleEvent(const ScannerEvent *scanEvent)
void ChannelScannerCLI::InformUser(const QString &error)

Change History (2)

comment:1 Changed 11 years ago by Dibblah

For CLI - As far as I know, cerr is intended - It's a command line interface to the channel scanner, so is expected to output to cerr.

comment:2 Changed 11 years ago by danielk

Resolution: fixed
Status: newclosed

(In [20794]) Fixes #6688. Remove errant cerr debugging in channel scan signal monitor encryption testing code.. Note: The cerr output in the command line scanner are intentional so I've left those in place.

Note: See TracTickets for help on using tickets.