May 2012 - DVDnav-discuss - lists.mplayerhq.hu

[PATCH] Make sure we get nav packets for all cells in multi angle
by Joakim Plate 09 Dec '12

09 Dec '12

Hi, This is a resend of an old patch that we still keep in xbmc's version of libdvdnav. The problem is that libdvdnav jumps over NAV packets when playing multi angle features. I don't remember all the exact details on this patch anymore since it was many years since i tracked it down. But it is very obvious in xbmc since we use nav packet end/start times to correct timestamps. Without the patch we git gaps in nav packets that doesn's exist in the streams. I think i still have a dvd sample that shows the issue. /Joakim

3 5

libdvdnav 4.1.4
by Jean-Jacques Sarton 09 Dec '12

09 Dec '12

-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Hello, On Fedora 14 totem and vlc end both with folowing message: libdvdread: Found 1 VTS's libdvdread: Elapsed time 0 libdvdnav: Language 'en' not found, using '' instead libdvdnav: Menu Languages available: libdvdnav: Language 'en' not found, using '' instead libdvdnav: Menu Languages available: libdvdnav: Language 'en' not found, using '' instead libdvdnav: Menu Languages available: libdvdnav: *** pgci_ut handle is NULL *** totem: /builddir/build/BUILD/libdvdnav-4.1.4/src/vm/vm.c:1772: get_ID: Assertion `pgcit != ((void *)0)' failed. Of course tje line totem:... is to be replaced withz vlc:... according to the used player The dvd contain no menu and contain a privat video. If I rename libdvdnav totem/vlc works. Regards, Jean-Jacques Sarton -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.11 (GNU/Linux) iEYEARECAAYFAk0h3bAACgkQM9JbiR3CwQsEEgCggN7qJrKQXspNYBJbCGRh237R +joAn1pQaRWHFN+nASDqRbL6Jgj3CM5S =iNgb -----END PGP SIGNATURE-----

4 6

[PATCH] consuming all memory when scanning dvd "Up"
by John Stebbins 09 Dec '12

09 Dec '12

First a disclaimer. This is a big invasive patch. If you don't like it, my feelings won't be hurt. I will be using it in HandBrake. So if you see problems with it, please let me know. Many dvd's are showing up that have titles that have numerous repeated language unit tables and pgc's. The new release "Up" is an example. I believe "Dark Knight" also exhibits this. When scanning such discs with lsdvd (and other programs that scan all titles), libdvdread will consume many GB of data for these repeated elements and gets very slow (e.g hours to read all titles) on OS X due to reading the duplicate data from uncached raw devices. This patch detects duplicates and reference counts pgc's and pgcit's. When a duplicate is detected, a reference count is incremented instead of allocating new memory and re-reading the data. I opted for reference counting instead of simply returning a failure on duplicate detection because it is perfectly valid to have a title with such duplicates. I could easily imagine titles adding duplicates, then navigating around them.

3 5

cross/mingw compilation (was assertion failure with certain DVD's)
by Roger Pack 19 Oct '12

19 Oct '12

>> $./configure2 >> ./configure2: line 95: cd: /usr/local/: No such file or directory # >> needs to be run in bash I believe > > Nope, you need to set the install prefix with --prefix=blabla Sweet that fixed it, as long as the prefix exists, otherwise same error message different path :) >> Checking if we're big-endian... ./configure2: line 113: mktemp: command not found This meant I didn't have msys installed, only mingw (my bad). >> Done, type 'make install' to build and install >> (appears to set HAVE_DLFCN_H in error) > > Without using autotools, dlfcn.h presence is not detected Is there a way to use autotools with libdvdread? I'm a bit lost, as is evident. though I can work around it for now. This actually makes me wonder how anybody is compiling it for their windows distros today. I know mplayer just uses the src directory and its own configs (which work), dunno about VLC/others. It seems like both msys and cross compiling don't work, or can somebody give me an example of how they do work, somehow? > ./msvc/config.h:#define HAVE_DLFCN_H 1 > > And it is assumed to exist (is this file made for Visual Studio?) I guess so, since it's under the msvc dir. >> Also as a note when it runs strip and ranlib, if it's being cross >> compiled, it would be helpful to call their equivalent: >> $ ./configure2 --cc=i586-mingw32msvc-gcc >> should eventually call i586-mingw32msvc-ranlib >> >> maybe it could use a new configure option for a prefix or what not. > > ./configure --host=i586-mingw32-msvc should work Yes I think it needs an option like that added (configure2 doesn't appear to have one as of yet). -roger-

2 11

[PATCH] Filenames in Unicode decoding issue
by Honza Horak 17 Sep '12

17 Sep '12

Hi all, there's an issue in libdvdread, triggered by DVDs with Unicode filenames (example of such DVD is a movie Thor). Original bug report is available here: https://bugzilla.redhat.com/show_bug.cgi?id=813977 There is also an enhanced Unicodedecode function on Ubuntu forum, which I consider better, than the current implementation is (see the patch attached): http://ubuntuforums.org/showthread.php?p=11254706 Unfortunately, I'm unable to reproduce this issue, since I don't have such DVD, so I cannot verify if the patch fixes the issue. Cheers, Honza

2 3

Error in libdvdnav when reading iso images
by Oncaphillis 14 Jun '12

14 Jun '12

I used to create DVD iso images via k3b and played them via VLC. With isos of relatively newly released DVD this stops to work and VLC fails with the message libdvdnav: demux error! 00 00 00 (should be 0x000001) Playing directly from /dev/dvd still works The distro is Fedora16 + k3b == 2.0.2 + libdvdcss == 1.2.11 ( I also tested 1.2.12) + vlc = 1.1.13 + libdvdnav 4.1.4. I also tried the current SVN build I get the same error when playing with mplayer and I also tried brasero for iso creation. Since direct play works it seems like brasero/k3b have problems to create a proper iso and the problems sits somewhere in the libs. Thank you O.

2 3

Please add dvdnav_dup() and dvdnav_free_dup() functions from the handbrake project
by Fabian Greffrath 25 May '12

25 May '12

Dear libdvdnav devs, please consider applying the following patch originating from the handbrake project: <https://github.com/HandBrake/HandBrake/blob/master/contrib/libdvdnav/A08-dv…> It adds two new functions, dvdnav_dup() and dvdnav_free_dup(), that duplicate a dvdnav_t object and remove a duplicate from memory, respectively - well, at least roughly. We (i.e. the pkg-multimedia-packages team in Debian) are currently trying to get handbrake built cleanly against Debian's system libraries and this patch is currently the only one that's missing to reach this goal. The handbrake build system currently downloads the libdvdnav source code (among others), patches it and builds a private library from it. Adding the functions directly into the handbrake source code (instead of the downloaded library) is impossible, because they need to know sizeof(dvdnav_t), which is not part of libdvdnav's public API - which is in turn reasonable. So, please, having these two functions in the next libdvdnav release would make it possible to build this huge OSS project that is handbrake using only the distribution's libraries (apart from libfaac, of course, which is prepared but still not in Debian, because it is considered non-free by our FTP-Masters). BTW, there are also some other interesting patches in handbrake's repos. ;) Best regards, Fabian

3 6