Merge pull request #96 from alexmercerind/windows-texture-migration

Using flutter::TextureRegistrar for Video on Windows.

Author: alexmercerind

.github/FUNDING.yml

@@ -1,2 +1 @@
-ko_fi: 'alexmercerind'
 custom: ['https://www.buymeacoffee.com/alexmercerind']

CHANGELOG.md

@@ -1,3 +1,10 @@
+## 0.1.2
+
+- Now using `flutter::TextureRegistrar` for performant `Video` on Windows. (#54).
+- Fixed `autoStart` in `Player.open`.
+- Fixed other crashes for Windows.
+- Improved stability.
+
 ## 0.1.1
 
 - Fixed setState being called after dispose (#75) (Finally)

README.md

@@ -320,16 +320,16 @@ sudo dnf install vlc-devel
 
 You can see an example project [here](https://github.com/alexmercerind/dart_vlc/blob/master/example/lib/main.dart).
 
-Windows
-
 ![](https://github.com/alexmercerind/dart_vlc/blob/assets/dart_vlc_6.png?raw=true)
 
+dart_vlc running on Ubuntu Linux.
 
 ## Workings
+
 The repository contains a [C++ wrapper](https://github.com/alexmercerind/dart_vlc/tree/master/dartvlc) based on libVLC++. This makes handling of events and controls a lot easier & has additional features in it.
 I preferred to do majority of handling in C++ itself, thus Dart code is minimal & very slight mapping to it.
 
-This project might seem like a Flutter plugin, but it is based on FFI instead. [Here](https://github.com/alexmercerind/dart_vlc/tree/master/ffi) are the FFI bindings to [C++ wrapper](https://github.com/alexmercerind/dart_vlc/tree/master/dartvlc), which are shared by all platforms & same can be used in Dart CLI apps aswell.
+This project might seem like a Flutter plugin, but it is based on FFI instead. [Here](https://github.com/alexmercerind/dart_vlc/tree/master/ffi) are the FFI bindings to [C++ wrapper](https://github.com/alexmercerind/dart_vlc/tree/master/dartvlc), which are shared by all platforms & same can be used in Dart CLI apps aswell. Platform channel interface is only used for [flutter]
 
 ## Progress
 
@@ -379,6 +379,8 @@ Done
 Under progress or planned features (irrespective of order)...
 
 - Removing [libVLC++](https://github.com/videolan/libvlcpp) dependency. (Maybe).
+- Subtitle control.
+- Audio track control.
 - Writing metadata tags.
 - Making things more efficient.
 - Supporting native volume control/lock screen notifications (Maybe).
@@ -389,9 +391,9 @@ Under progress or planned features (irrespective of order)...
 
 First of all, thanks to the [VideoLAN](https://www.videolan.org) team for creating [libVLC](https://github.com/videolan/vlc) & [libVLC++](https://github.com/videolan/libvlcpp). Really great guys really great at their work.
 
-Massive thanks to [@stuartmorgan](https://github.com/stuartmorgan) from [Flutter](https://flutter.dev) team to my review code & help me fix the loopholes.
+[@jnschulze](https://github.com/jnschulze) for his awesome contributions to Flutter engine like adding texture support.
 
-Thanks to following members of libVLC community to give me bit of look & advice about how things work:
+Thanks to following members of libVLC community (irrespective of the order) to give me bit of look & advice about how things work:
 
 - [@jeremyVignelles](https://github.com/jeremyVignelles)
 - [@chouquette](https://github.com/chouquette)

dartvlc/internal/events.hpp

@@ -86,7 +86,7 @@ class PlayerEvents : public PlayerGetters {
 		);
 		this->mediaPlayer.setVideoFormatCallbacks(
 			[=](char* chroma, uint32_t* w, uint32_t* h, uint32_t* p, uint32_t* l) -> int {
-				strcpy(chroma, "RV32");
+				strcpy(chroma, "RGBA");
 				*w = this->videoWidth;
 				*h = this->videoHeight;
 				*p = pitch;
@@ -95,7 +95,7 @@ class PlayerEvents : public PlayerGetters {
 			},
 			nullptr
 		);
-		this->mediaPlayer.setVideoFormat("RV32", this->videoWidth, this->videoHeight, pitch);
+		this->mediaPlayer.setVideoFormat("RGBA", this->videoWidth, this->videoHeight, pitch);
 	}
 
 protected:

dartvlc/internal/setters.hpp

@@ -19,10 +19,11 @@ class PlayerSetters: public PlayerEvents {
 public:
 	void open(MediaSource* mediaSource, bool autoStart = true) {
 		/* Freed the previous `Media` objects when a new `Playlist` or `Media` is opened. */
+		this->state->isStarted = false;
+		this->stop();
 		for (Media* media: this->state->medias->medias) {
 			delete media;
 		}
-		this->stop();
 		this->state->medias->medias = {};
 		this->mediaList = VLC::MediaList(this->instance);
 		if (mediaSource->mediaSourceType() == "MediaSourceType.media") {
@@ -46,14 +47,20 @@ class PlayerSetters: public PlayerEvents {
 			this->state->isPlaylist = true;
 		}
 		this->_onOpenCallback(this->mediaList.itemAtIndex(0));
-		this->mediaListPlayer.playItemAtIndex(0);
 		this->state->index = 0;
 		this->state->isPlaying = this->mediaListPlayer.isPlaying();
 		this->state->isValid = this->mediaListPlayer.isValid();
+		if (autoStart) this->play();
 	}
 
 	void play() {
-        this->mediaListPlayer.play();
+		if (!this->state->isStarted) {
+			this->mediaListPlayer.playItemAtIndex(0);
+			this->state->isStarted = true;
+		}
+        else {
+			this->mediaListPlayer.play();
+		}
 	}
 
     void pause() {
@@ -63,7 +70,13 @@ class PlayerSetters: public PlayerEvents {
     }
 
 	void playOrPause() {
-		this->mediaListPlayer.pause();
+		if (!this->state->isStarted) {
+			this->mediaListPlayer.playItemAtIndex(0);
+			this->state->isStarted = true;
+		}
+		else {
+			this->mediaListPlayer.pause();
+		}
 	}
 
     void stop() {

dartvlc/internal/state.hpp

@@ -28,6 +28,8 @@ class PlayerState {
 	float rate = 1.0;
 	bool isPlaylist = false;
 	Device* device = nullptr;
+	/* NOTE: Used for autoStart. */
+	bool isStarted = false;
 	/* TODO: Not used yet.
 	Equalizer* equalizer = nullptr;
 	*/

dartvlc/player.hpp

@@ -60,7 +60,6 @@ class Player: public PlayerSetters {
 		this->mediaPlayer.stop();
 		delete this->state;
 		delete this->_videoFrameBuffer;
-		for (size_t i = 0; i < argsSize; i++) delete this->args[i];
 		delete[] this->args;
 	}
 
@@ -81,6 +80,7 @@ class Players {
 
 	void dispose(int id, std::function<void()> callback = []() -> void {}) {
         delete this->players[id];
+		this->players.erase(id);
 		callback();
     }
 

ffi/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 0.1.1
+
+- Removed `Player::onVideo` callbacks for win32.
+
 ## 0.1.0
 
 - Added new native methods for handling with memory leaks.

ffi/README.md

@@ -3,7 +3,7 @@
 <h5 align="center">Written in C++ using libVLC & libVLC++.</h5>
 
 
-FFI bindings & NativePort event callback handling for `dart_vlc.so`.
+FFI bindings & NativePort event callback handling for `dart_vlc.dll`.
 
 Dependency package for [dart_vlc](https:///github.com/alexmercerind/dart_vlc).
 

ffi/lib/src/player.dart

@@ -37,7 +37,7 @@ Map<int, Player> players = {};
 ///
 class Player {
   /// Id associated with the [Player] instance.
-  final int id;
+  int id;
 
   /// Width of the [Video] frames to be extracted. Higher value may lead to degraded performance.
   late int videoWidth;