[PATCH] "dvdnav_jump_to_sector" as an alternative to "dvdnav_time_search" (REV 3: fix: jumping in cell 1 would use admap, not tmap)
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
On Thu, Nov 17, 2011 at 12:12:16AM -0500, gnosygnu wrote:
[...]
I assume you were trying to be helpful, but compressing patches is just an annoyance. The saved space is nothing compared to the convenience of seeing it directly in your mail client. Diego
participants (2)
-
Diego Biurrun -
gnosygnu