[mythtv-commits] Ticket #5000: null check of m_priv unnecessary in OpenGLContext dtor

MythTV mythtv at cvs.mythtv.org
Thu Mar 20 17:41:46 UTC 2008


#5000: null check of m_priv unnecessary in OpenGLContext dtor
---------------------------------------------+------------------------------
 Reporter:  Erik Hovland <erik at hovland.org>  |        Owner:  danielk 
     Type:  defect                           |       Status:  assigned
 Priority:  trivial                          |    Milestone:  0.22    
Component:  mythtv                           |      Version:  head    
 Severity:  low                              |   Resolution:          
  Mlocked:  0                                |  
---------------------------------------------+------------------------------
Changes (by danielk):

  * owner:  ijr => danielk
  * status:  new => assigned
  * milestone:  unknown => 0.22


Comment:

 Erik, I think the null pointer check is prudent, since if the new fails
 the class will hopefully be destructed soon afterward. But I see your
 point about MakeCurrent(), I will fix this.

 PS On Linux new always succeeds unless the default behavior has been
 changed by the user, but this is not the case on other operating systems,
 like some of the BSD's.

-- 
Ticket URL: <http://svn.mythtv.org/trac/ticket/5000#comment:1>
MythTV <http://www.mythtv.org/>
MythTV


More information about the mythtv-commits mailing list