Do not play on resume if the video is not loaded for CommonVideoPlayerDesktop

lucas-viva · web-flow · commit 4b5309c5ef45 · 2025-01-16T09:32:06.000-03:00
In the old implementation (where `load` &amp; `play` were done together), `resume` would load and play the video.

This change avoids a `NullPointerException` if calling `resume` before `load`.
diff --git a/gdx-video-desktop/src/com/badlogic/gdx/video/CommonVideoPlayerDesktop.java b/gdx-video-desktop/src/com/badlogic/gdx/video/CommonVideoPlayerDesktop.java
@@ -297,7 +297,9 @@ public void pause () {
 
 	@Override
 	public void resume () {
-		play();
+		if (decoder != null) {
+			play();
+		}
 	}
 
 	@Override

Original file line number	Diff line number	Diff line change
`@@ -297,7 +297,9 @@ public void pause () {`
`297`	`297`
`298`	`298`	`@Override`
`299`	`299`	`public void resume () {`
`300`		`- play();`
	`300`	`+ if (decoder != null) {`
	`301`	`+ play();`
	`302`	`+ }`
`301`	`303`	`}`
`302`	`304`
`303`	`305`	`@Override`