stream: kill start_pos, remove --sb option

stream.start_pos was needed for optical media only, and (apparently) not
for very good reasons. Just get rid of it.

For stream_dvd, we don't need to do anything. Byte seeking was already
removed from it earlier.

For stream_cdda and stream_vcd, emulate the start_pos by offsetting the
stream pos as seen by the rest of mpv.

The bits in discnav.c and loadfile.c were for dealing with the code
seeking back to the start in demux.c. Handle this differently by
assuming the demuxer is always initialized with the stream at start
position, and instead seek back if initializing the demuxer fails.

Remove the --sb option, which worked by modifying stream.start_pos. If
someone really wants this option, it could be added back by creating a
"slice" stream (actually ffmpeg already has such a thing).

2 files changed, 0 insertions, 3 deletions

diff --git a/options/options.c b/options/options.c
index 3b9347ebb8..7377f666d9 100644
--- a/options/options.c
+++ b/options/options.c
@@ -281,8 +281,6 @@ const m_option_t mp_opts[] = {
     OPT_CHOICE_OR_INT("frames", play_frames, M_OPT_FIXED, 0, INT_MAX,
                       ({"all", -1})),
 
-    // seek to byte/seconds position
-    OPT_INT64("sb", seek_to_byte, 0),
     OPT_REL_TIME("start", play_start, 0),
     OPT_REL_TIME("end", play_end, 0),
     OPT_REL_TIME("length", play_length, 0),
diff --git a/options/options.h b/options/options.h
index 5da933d41d..6b482e2665 100644
--- a/options/options.h
+++ b/options/options.h
@@ -149,7 +149,6 @@ typedef struct MPOpts {
     struct m_rel_time play_length;
     int play_frames;
     double step_sec;
-    int64_t seek_to_byte;
     int position_resume;
     int position_save_on_quit;
     int pause;