[DVDnav-discuss] [PATCH] "dvdnav_jump_to_sector" as an alternative to "dvdnav_time_search" (REV 3: fix: jumping in cell 1 would use admap, not tmap)

gnosygnu gnosygnu at gmail.com
Thu Nov 17 06:12:16 CET 2011


Two changes:
1) fix for jumping in first cell
Roger found an embarrassing bug wherein jumping in the first cell
would only use the admap, not the tmap.
It raised the "cell not found" errors in his earlier trace.

Specifically, I changed the following line in dvdnav_cell_find

old:
    else if (find_mode == CELL_FIND_INDEX) {
      if (find_val == CELL_FIND_INDEX) {

new:
    else if (find_mode == CELL_FIND_INDEX) {
      if (find_val == cur_cell_idx) {

2) CELL_FIND_ prefixes
Erik suggested adding CELL_FIND_ prefixes to the cell_find_mode_t enum

I've attached revision 3 above.
Thanks
-------------- next part --------------
A non-text attachment was scrubbed...
Name: jump_to_time.rev3.patch.zip
Type: application/zip
Size: 5565 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/dvdnav-discuss/attachments/20111117/e7428900/attachment.zip>


More information about the DVDnav-discuss mailing list