Login | Register For Free | Help
Search for: (Advanced)

Mailing List Archive: MythTV: Commits

Ticket #7590: programinfo needs to init isVideo when loading from DB

 

 

MythTV commits RSS feed   Index | Next | Previous | View Threaded


mythtv at cvs

Nov 16, 2009, 6:09 PM

Post #1 of 3 (261 views)
Permalink
Ticket #7590: programinfo needs to init isVideo when loading from DB

#7590: programinfo needs to init isVideo when loading from DB
--------------------------------------------------+-------------------------
Reporter: John Patrick Poet <jppoet@…> | Owner: ijr
Type: defect | Status: new
Priority: minor | Milestone: unknown
Component: MythTV - General | Version: unknown
Severity: low | Mlocked: 0
--------------------------------------------------+-------------------------
{{{
ProgramInfo::ProgramInfo(const MSqlQuery &query, ...)
}}}

Does not initialize isVideo. This causes ProgramInfo::ToMap() to randomly
fill in starttime, endtime, startdate, etc., incorrectly.

Trivial patch attached.

--
Ticket URL: <http://svn.mythtv.org/trac/ticket/7590>
MythTV <http://www.mythtv.org/>
MythTV
_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-commits


mythtv at cvs

Nov 16, 2009, 6:30 PM

Post #2 of 3 (236 views)
Permalink
Re: Ticket #7590: programinfo needs to init isVideo when loading from DB [In reply to]

#7590: programinfo needs to init isVideo when loading from DB
--------------------------------------------------+-------------------------
Reporter: John Patrick Poet <jppoet@…> | Owner: danielk
Type: defect | Status: assigned
Priority: minor | Milestone: unknown
Component: MythTV - General | Version: unknown
Severity: low | Mlocked: 0
--------------------------------------------------+-------------------------
Changes (by danielk):

* owner: ijr => danielk
* status: new => assigned


--
Ticket URL: <http://svn.mythtv.org/trac/ticket/7590#comment:1>
MythTV <http://www.mythtv.org/>
MythTV
_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-commits


mythtv at cvs

Nov 16, 2009, 8:11 PM

Post #3 of 3 (237 views)
Permalink
Re: Ticket #7590: programinfo needs to init isVideo when loading from DB [In reply to]

#7590: programinfo needs to init isVideo when loading from DB
--------------------------------------------------+-------------------------
Reporter: John Patrick Poet <jppoet@…> | Owner: danielk
Type: defect | Status: closed
Priority: minor | Milestone: unknown
Component: MythTV - General | Version: unknown
Severity: low | Resolution: fixed
Mlocked: 0 |
--------------------------------------------------+-------------------------
Changes (by danielk):

* status: assigned => closed
* resolution: => fixed


Comment:

(In [22854]) Fixes #7590. Initialize all ProgramInfo members in ctors.

This fixes the uninitialized variable John discovered, plus a couple other
instances. It also reorders the clone function so as to make it easier to
verify it in the future.

--
Ticket URL: <http://svn.mythtv.org/trac/ticket/7590#comment:2>
MythTV <http://www.mythtv.org/>
MythTV
_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-commits

MythTV commits RSS feed   Index | Next | Previous | View Threaded
 
 


Interested in having your list archived? Contact Gossamer Threads
 
  Web Applications & Managed Hosting Powered by Gossamer Threads Inc.