search

changsanjiang / SJBaseVideoPlayer

video player. rotation, volume, brightness, rate, play, pause, stop, seekToTime, present.
MIT License
128 stars 52 forks source link

endGeneratingDeviceOrientationNotifications 没有调用可能导致 iOS 13.x crash #25

Closed ACFancy closed 2 years ago

ACFancy commented 2 years ago

问题

ACFancy commented 2 years ago

@changsanjiang

changsanjiang commented 2 years ago

我看看

ACFancy commented 2 years ago

@changsanjiang 可以参考下这个库的PR: PR

changsanjiang commented 2 years ago

他这个应该解决的是 多次调用开启通知的问题。 咱们不需要， 开启之前已经做了相应判断了。

ACFancy commented 2 years ago

@changsanjiang 开启前是做了判断，但是begin和end并没有成对出现吧? 如果是自己主动开启的，还是要满足条件自己主动关闭掉吧？

ACFancy commented 2 years ago

他这个应该解决的是 多次调用开启通知的问题。 咱们不需要， 开启之前已经做了相应判断了。上面提及的PR通知只是他们的一部分，还有一部分是begin和end的

changsanjiang commented 2 years ago

通常不需要关闭它， 因为我们一直要监听设备方向变化的， 关闭之后就收不到了。

比方App 会创建很多个播放器， 一旦某个播放器关闭通知， 其他播放器的旋转管理类就收不到通知了。