[mythtv-commits] Ticket #4442: upnpcdsvideo doesn't handle directories correctly

MythTV mythtv at cvs.mythtv.org
Tue Jan 8 01:51:41 UTC 2008


#4442: upnpcdsvideo doesn't handle directories correctly
----------------------------------------------------------+-----------------
 Reporter:  David Asher <david.asher at caviumnetworks.com>  |       Owner:  dblain 
     Type:  defect                                        |      Status:  new    
 Priority:  minor                                         |   Milestone:  unknown
Component:  upnp                                          |     Version:  unknown
 Severity:  medium                                        |     Mlocked:  0      
----------------------------------------------------------+-----------------
 There are at least 3 problems with the current svn head's (15373) handling
 of directories in upnpcdsvideo.cpp:

 1. When ProcessAll is called (for Videos/0), it gets ALL videos instead of
 the videos whose parentid == STARTING_VIDEO_OBJECTID.  Consequently the
 container returned for VideoRoot reports too many children.  I haven't
 provided a patch for this as it looks quite intrusive to fix, possibly
 requiring some refactoring around GetDistinctCount or the handling of the
 root nodes.

 2. In UPnpCDSVideo::AddItem, the child count is never filled in when a
 container is created.  The attached patch fixes this.

 3. It looks like the support for multiple levels of directories is broken.
 A path like <root>/Sub1/Sub2/Vid1 doesn't produce the expected results.
 The IDs get kinda out of whack and extra levels of hierarchy keep getting
 created.  I haven't fully worked this one out yet.  (so no patch)

-- 
Ticket URL: <http://svn.mythtv.org/trac/ticket/4442>
MythTV <http://svn.mythtv.org/trac>
MythTV


More information about the mythtv-commits mailing list