snd_xmp.c: handle the tricky xmp_set_position() return code better

sezero · sezero · commit d6919462f0e2 · 2026-03-11T17:11:02.000+03:00
diff --git a/engine/h2shared/snd_xmp.c b/engine/h2shared/snd_xmp.c
@@ -158,7 +158,13 @@ static void S_XMP_CodecCloseStream (snd_stream_t *stream)
 
 static int S_XMP_CodecJumpToOrder (snd_stream_t *stream, int to)
 {
-	return xmp_set_position((xmp_context)stream->priv, to);
+	const int err = xmp_set_position((xmp_context)stream->priv, to);
+	switch (err) {
+	case -XMP_ERROR_STATE:
+	case -XMP_ERROR_INVALID:
+		return -1;
+	}
+	return 0;
 }
 
 static int S_XMP_CodecRewindStream (snd_stream_t *stream)

Original file line number	Diff line number	Diff line change
`@@ -158,7 +158,13 @@ static void S_XMP_CodecCloseStream (snd_stream_t *stream)`
`158`	`158`
`159`	`159`	`static int S_XMP_CodecJumpToOrder (snd_stream_t *stream, int to)`
`160`	`160`	`{`
`161`		`- return xmp_set_position((xmp_context)stream->priv, to);`
	`161`	`+ const int err = xmp_set_position((xmp_context)stream->priv, to);`
	`162`	`+ switch (err) {`
	`163`	`+ case -XMP_ERROR_STATE:`
	`164`	`+ case -XMP_ERROR_INVALID:`
	`165`	`+ return -1;`
	`166`	`+ }`
	`167`	`+ return 0;`
`162`	`168`	`}`
`163`	`169`
`164`	`170`	`static int S_XMP_CodecRewindStream (snd_stream_t *stream)`