[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