[vlc-commits] medialib: Bump contrib & reflect API changes
Hugo Beauzée-Luyssen
git at videolan.org
Mon Aug 24 13:27:49 CEST 2020
vlc | branch: master | Hugo Beauzée-Luyssen <hugo at beauzee.fr> | Wed Jul 1 14:43:38 2020 +0200| [fb68c410d5f52bbcf58557f1a72fa1146b6c6f01] | committer: Hugo Beauzée-Luyssen
medialib: Bump contrib & reflect API changes
> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=fb68c410d5f52bbcf58557f1a72fa1146b6c6f01
---
configure.ac | 2 +-
contrib/src/medialibrary/rules.mak | 4 +-
include/vlc_media_library.h | 10 ++---
.../macosx/library/VLCLibraryCollectionViewItem.m | 2 +-
modules/gui/macosx/library/VLCLibraryController.m | 3 --
modules/gui/macosx/library/VLCLibraryDataTypes.h | 4 +-
modules/gui/macosx/library/VLCLibraryDataTypes.m | 16 +------
.../macosx/library/VLCLibraryInformationPanel.m | 2 +-
.../playlist/VLCPlaybackContinuityController.m | 6 +--
modules/gui/qt/medialibrary/mlvideo.cpp | 9 +---
modules/misc/medialibrary/MetadataExtractor.cpp | 5 ++-
modules/misc/medialibrary/entities.cpp | 1 +
modules/misc/medialibrary/fs/device.cpp | 9 ++--
modules/misc/medialibrary/fs/device.h | 2 +-
modules/misc/medialibrary/fs/file.cpp | 10 +++++
modules/misc/medialibrary/fs/file.h | 3 ++
modules/misc/medialibrary/fs/fs.cpp | 6 +--
modules/misc/medialibrary/fs/fs.h | 2 +-
modules/misc/medialibrary/medialib.cpp | 14 ++----
src/player/input.c | 6 +--
src/player/medialib.c | 51 ++++------------------
src/player/player.h | 1 +
22 files changed, 58 insertions(+), 110 deletions(-)
diff --git a/configure.ac b/configure.ac
index 4f7a46349b..8654e87d74 100644
--- a/configure.ac
+++ b/configure.ac
@@ -4455,7 +4455,7 @@ dnl Libnotify notification plugin
dnl
PKG_ENABLE_MODULES_VLC([NOTIFY], [], [libnotify], [libnotify notification], [auto])
-PKG_ENABLE_MODULES_VLC([MEDIALIBRARY], [medialibrary], [medialibrary >= 0.8.0], (medialibrary support), [auto])
+PKG_ENABLE_MODULES_VLC([MEDIALIBRARY], [medialibrary], [medialibrary >= 0.9.0], (medialibrary support), [auto])
dnl
dnl Endianness check
diff --git a/contrib/src/medialibrary/rules.mak b/contrib/src/medialibrary/rules.mak
index ff6d7c40d5..e2ba1c7a8e 100644
--- a/contrib/src/medialibrary/rules.mak
+++ b/contrib/src/medialibrary/rules.mak
@@ -1,9 +1,9 @@
-MEDIALIBRARY_HASH := cfc889fcd97ee9b1a71e100888715389025b12ea
+MEDIALIBRARY_HASH := b208f5b7f0e7981efbc779018d5ae7f117f331d2
MEDIALIBRARY_VERSION := git-$(MEDIALIBRARY_HASH)
MEDIALIBRARY_GITURL := https://code.videolan.org/videolan/medialibrary.git
PKGS += medialibrary
-ifeq ($(call need_pkg,"medialibrary >= 0.8"),)
+ifeq ($(call need_pkg,"medialibrary >= 0.9"),)
PKGS_FOUND += medialibrary
endif
diff --git a/include/vlc_media_library.h b/include/vlc_media_library.h
index fd82a71a23..aa12175979 100644
--- a/include/vlc_media_library.h
+++ b/include/vlc_media_library.h
@@ -201,6 +201,7 @@ typedef struct vlc_ml_media_t
/* Duration in milliseconds */
int64_t i_duration;
uint32_t i_playcount;
+ float f_progress;
time_t i_last_played_date;
char* psz_title;
@@ -471,7 +472,7 @@ enum vlc_ml_control
VLC_ML_NEW_STREAM, /**< arg1: const char*; arg2(out): vlc_ml_media_t** */
/* Media management */
- VLC_ML_MEDIA_INCREASE_PLAY_COUNT, /**< arg1: media id; can fail */
+ VLC_ML_MEDIA_UPDATE_PROGRESS, /**< arg1: media id; arg2: playback position; can fail */
VLC_ML_MEDIA_GET_MEDIA_PLAYBACK_STATE, /**< arg1: media id; arg2: vlc_ml_playback_state; arg3: char**; */
VLC_ML_MEDIA_SET_MEDIA_PLAYBACK_STATE, /**< arg1: media id; arg2: vlc_ml_playback_state; arg3: const char*; */
VLC_ML_MEDIA_GET_ALL_MEDIA_PLAYBACK_STATES, /**< arg1: media id; arg2(out): vlc_ml_playback_states_all* */
@@ -499,7 +500,6 @@ enum vlc_ml_control
enum vlc_ml_playback_state
{
VLC_ML_PLAYBACK_STATE_RATING,
- VLC_ML_PLAYBACK_STATE_PROGRESS,
VLC_ML_PLAYBACK_STATE_SPEED,
VLC_ML_PLAYBACK_STATE_TITLE,
VLC_ML_PLAYBACK_STATE_CHAPTER,
@@ -521,7 +521,6 @@ enum vlc_ml_playback_state
typedef struct vlc_ml_playback_states_all
{
- float progress;
float rate;
float zoom;
int current_title;
@@ -932,9 +931,10 @@ static inline vlc_ml_media_t* vlc_ml_new_stream( vlc_medialibrary_t* p_ml, const
return res;
}
-static inline int vlc_ml_media_increase_playcount( vlc_medialibrary_t* p_ml, int64_t i_media_id )
+static inline int vlc_ml_media_update_progress( vlc_medialibrary_t* p_ml, int64_t i_media_id,
+ double progress )
{
- return vlc_ml_control( p_ml, VLC_ML_MEDIA_INCREASE_PLAY_COUNT, i_media_id );
+ return vlc_ml_control( p_ml, VLC_ML_MEDIA_UPDATE_PROGRESS, i_media_id, progress );
}
static inline int vlc_ml_media_get_playback_state( vlc_medialibrary_t* p_ml, int64_t i_media_id, int i_state, char** ppsz_result )
diff --git a/modules/gui/macosx/library/VLCLibraryCollectionViewItem.m b/modules/gui/macosx/library/VLCLibraryCollectionViewItem.m
index f6487a17d9..b836d52a7f 100644
--- a/modules/gui/macosx/library/VLCLibraryCollectionViewItem.m
+++ b/modules/gui/macosx/library/VLCLibraryCollectionViewItem.m
@@ -172,7 +172,7 @@ const CGFloat VLCLibraryCollectionViewItemMaximumDisplayedProgress = 0.95;
VLCMediaLibraryTrack *videoTrack = _representedMediaItem.firstVideoTrack;
[self showVideoSizeIfNeededForWidth:videoTrack.videoWidth andHeight:videoTrack.videoHeight];
- CGFloat position = _representedMediaItem.lastPlaybackPosition;
+ CGFloat position = _representedMediaItem.progress;
if (position > VLCLibraryCollectionViewItemMinimalDisplayedProgress && position < VLCLibraryCollectionViewItemMaximumDisplayedProgress) {
_progressIndicator.progress = position;
_progressIndicator.hidden = NO;
diff --git a/modules/gui/macosx/library/VLCLibraryController.m b/modules/gui/macosx/library/VLCLibraryController.m
index b20b48ade8..58a100ad46 100644
--- a/modules/gui/macosx/library/VLCLibraryController.m
+++ b/modules/gui/macosx/library/VLCLibraryController.m
@@ -94,9 +94,6 @@
}
int ret = [[[VLCMain sharedInstance] playlistController] addInputItem:p_inputItem atPosition:-1 startPlayback:playImmediately];
input_item_Release(p_inputItem);
- if (ret == VLC_SUCCESS) {
- [mediaItem increasePlayCount];
- }
return ret;
}
diff --git a/modules/gui/macosx/library/VLCLibraryDataTypes.h b/modules/gui/macosx/library/VLCLibraryDataTypes.h
index c643e5e6c8..b61dfde6a6 100644
--- a/modules/gui/macosx/library/VLCLibraryDataTypes.h
+++ b/modules/gui/macosx/library/VLCLibraryDataTypes.h
@@ -172,6 +172,7 @@ extern const long long int VLCMediaLibraryMediaItemDurationDenominator;
@property (readonly) int64_t duration; /* Duration in milliseconds */
@property (readonly) uint32_t playCount;
@property (readonly) time_t lastPlayedDate;
+ at property (readonly) float progress;
@property (readonly) NSString *title;
@property (readonly, nullable) NSString *smallArtworkMRL;
@@ -185,7 +186,6 @@ extern const long long int VLCMediaLibraryMediaItemDurationDenominator;
@property (readonly, nullable) VLCMediaLibraryAlbumTrack *albumTrack;
@property (readwrite) int rating;
- at property (readwrite) float lastPlaybackPosition;
@property (readwrite) float lastPlaybackRate;
@property (readwrite) int lastTitle;
@property (readwrite) int lastChapter;
@@ -203,8 +203,6 @@ extern const long long int VLCMediaLibraryMediaItemDurationDenominator;
@property (readwrite) int lastSubtitleTrack;
@property (readwrite) int lastSubtitleDelay;
-- (int)increasePlayCount;
-
@end
@interface VLCMediaLibraryEntryPoint : NSObject
diff --git a/modules/gui/macosx/library/VLCLibraryDataTypes.m b/modules/gui/macosx/library/VLCLibraryDataTypes.m
index 167ecaf38e..b9c0520050 100644
--- a/modules/gui/macosx/library/VLCLibraryDataTypes.m
+++ b/modules/gui/macosx/library/VLCLibraryDataTypes.m
@@ -383,6 +383,7 @@ NSString *VLCMediaLibraryMediaItemLibraryID = @"VLCMediaLibraryMediaItemLibraryI
_duration = p_mediaItem->i_duration;
_playCount = p_mediaItem->i_playcount;
_lastPlayedDate = p_mediaItem->i_last_played_date;
+ _progress = p_mediaItem->f_progress;
_title = toNSStr(p_mediaItem->psz_title);
_smallArtworkGenerated = p_mediaItem->thumbnails[VLC_ML_THUMBNAIL_SMALL].psz_mrl != NULL;
if (_smallArtworkGenerated) {
@@ -598,16 +599,6 @@ NSString *VLCMediaLibraryMediaItemLibraryID = @"VLCMediaLibraryMediaItemLibraryI
[self setIntegerPreference:rating forKey:VLC_ML_PLAYBACK_STATE_RATING];
}
-- (float)lastPlaybackPosition
-{
- return [self floatPreferenceForKey:VLC_ML_PLAYBACK_STATE_PROGRESS];
-}
-
-- (void)setLastPlaybackPosition:(float)lastPlaybackPosition
-{
- [self setFloatPreference:lastPlaybackPosition forKey:VLC_ML_PLAYBACK_STATE_PROGRESS];
-}
-
- (float)lastPlaybackRate
{
return [self floatPreferenceForKey:VLC_ML_PLAYBACK_STATE_SPEED];
@@ -768,11 +759,6 @@ NSString *VLCMediaLibraryMediaItemLibraryID = @"VLCMediaLibraryMediaItemLibraryI
[self setIntegerPreference:lastSubtitleDelay forKey:VLC_ML_PLAYBACK_STATE_SUBTITLE_DELAY];
}
-- (int)increasePlayCount
-{
- return vlc_ml_media_increase_playcount(_p_mediaLibrary, _libraryID);
-}
-
@end
@implementation VLCMediaLibraryEntryPoint
diff --git a/modules/gui/macosx/library/VLCLibraryInformationPanel.m b/modules/gui/macosx/library/VLCLibraryInformationPanel.m
index 54b0e0a3e6..f81579c7a9 100644
--- a/modules/gui/macosx/library/VLCLibraryInformationPanel.m
+++ b/modules/gui/macosx/library/VLCLibraryInformationPanel.m
@@ -56,7 +56,7 @@
[textContent appendFormat:@"Duration: %@\n", [NSString stringWithTime:_representedMediaItem.duration / VLCMediaLibraryMediaItemDurationDenominator]];
[textContent appendFormat:@"Play count: %u, last played: %@\n", _representedMediaItem.playCount, [NSDateFormatter localizedStringFromDate:[NSDate dateWithTimeIntervalSince1970:_representedMediaItem.lastPlayedDate] dateStyle:NSDateFormatterShortStyle timeStyle:NSDateFormatterShortStyle]];
[textContent appendFormat:@"Small artwork generated? %@\n", _representedMediaItem.smallArtworkGenerated == YES ? _NS("Yes") : _NS("No")];
- [textContent appendFormat:@"Favorited? %@, Playback progress: %2.f%%\n", _representedMediaItem.smallArtworkGenerated == YES ? _NS("Yes") : _NS("No"), _representedMediaItem.lastPlaybackPosition * 100.];
+ [textContent appendFormat:@"Favorited? %@, Playback progress: %2.f%%\n", _representedMediaItem.smallArtworkGenerated == YES ? _NS("Yes") : _NS("No"), _representedMediaItem.progress * 100.];
NSArray *array = _representedMediaItem.files;
NSUInteger count = array.count;
diff --git a/modules/gui/macosx/playlist/VLCPlaybackContinuityController.m b/modules/gui/macosx/playlist/VLCPlaybackContinuityController.m
index d9a719c6d7..5a490c5699 100644
--- a/modules/gui/macosx/playlist/VLCPlaybackContinuityController.m
+++ b/modules/gui/macosx/playlist/VLCPlaybackContinuityController.m
@@ -187,7 +187,7 @@ static NSString *VLCRecentlyPlayedMediaListKey = @"recentlyPlayedMediaList";
ask:(BOOL)ask
player:(VLCPlayerController *)playerController
{
- float lastPlaybackPosition = libraryMediaItem.lastPlaybackPosition;
+ float lastPlaybackPosition = libraryMediaItem.progress;
int64_t duration = libraryMediaItem.duration;
if (lastPlaybackPosition < MinimumStorePercent || duration < MinimumDuration) {
@@ -293,10 +293,6 @@ BOOL ShouldStorePlaybackPosition(float position, int64_t duration)
// input item isn't part of the library
[self storeLegacyPlaybackPositionForInputItem:inputItem withPlayer:playerController];
}
-
- if (ShouldStorePlaybackPosition(playerController.position, libraryMediaItem.duration)) {
- libraryMediaItem.lastPlaybackPosition = playerController.position;
- }
}
- (void)storeLegacyPlaybackPositionForInputItem:(VLCInputItem *)inputItem
diff --git a/modules/gui/qt/medialibrary/mlvideo.cpp b/modules/gui/qt/medialibrary/mlvideo.cpp
index 3092d9b3ce..e1c3b0d9c5 100644
--- a/modules/gui/qt/medialibrary/mlvideo.cpp
+++ b/modules/gui/qt/medialibrary/mlvideo.cpp
@@ -80,7 +80,7 @@ MLVideo::MLVideo(vlc_medialibrary_t* ml, const vlc_ml_media_t* data, QObject* pa
, m_id( data->i_id, VLC_ML_PARENT_UNKNOWN )
, m_title( QString::fromUtf8( data->psz_title ) )
, m_thumbnail( QString::fromUtf8( data->thumbnails[VLC_ML_THUMBNAIL_SMALL].psz_mrl ) )
- , m_progress( -1.f )
+ , m_progress( data->f_progress )
, m_playCount( data->i_playcount )
, m_thumbnailGenerated( data->thumbnails[VLC_ML_THUMBNAIL_SMALL].b_generated )
, m_ml_event_handle( nullptr, [this](vlc_ml_event_callback_t* cb ) {
@@ -99,13 +99,6 @@ MLVideo::MLVideo(vlc_medialibrary_t* ml, const vlc_ml_media_t* data, QObject* pa
m_mrl = QUrl::fromEncoded(file.psz_mrl);
break;
}
- char *psz_progress;
- if ( vlc_ml_media_get_playback_state( ml, data->i_id, VLC_ML_PLAYBACK_STATE_PROGRESS,
- &psz_progress ) == VLC_SUCCESS && psz_progress != NULL )
- {
- m_progress = atof( psz_progress );
- free( psz_progress );
- }
unsigned int numChannel = 0 , maxWidth = 0 , maxHeight = 0;
for( const vlc_ml_media_track_t& track: ml_range_iterate<vlc_ml_media_track_t>( data->p_tracks ) ) {
diff --git a/modules/misc/medialibrary/MetadataExtractor.cpp b/modules/misc/medialibrary/MetadataExtractor.cpp
index 0896804f45..0d6d969cc9 100644
--- a/modules/misc/medialibrary/MetadataExtractor.cpp
+++ b/modules/misc/medialibrary/MetadataExtractor.cpp
@@ -131,7 +131,8 @@ void MetadataExtractor::addSubtree( ParseContext& ctx, input_item_node_t *root )
for ( auto i = 0; i < root->i_children; ++i )
{
auto it = root->pp_children[i]->p_item;
- auto& subItem = ctx.item.createSubItem( it->psz_uri, i );
+ auto& subItem = ctx.item.createLinkedItem( it->psz_uri,
+ medialibrary::IFile::Type::Main, i );
populateItem( subItem, it );
}
}
@@ -184,7 +185,7 @@ medialibrary::parser::Status MetadataExtractor::run( medialibrary::parser::IItem
return medialibrary::parser::Status::Fatal;
if ( item.fileType() == medialibrary::IFile::Type::Playlist &&
- item.nbSubItems() == 0 )
+ item.nbLinkedItems() == 0 )
return medialibrary::parser::Status::Fatal;
populateItem( item, ctx.inputItem.get() );
diff --git a/modules/misc/medialibrary/entities.cpp b/modules/misc/medialibrary/entities.cpp
index 8c567f954a..2078ab3ba5 100644
--- a/modules/misc/medialibrary/entities.cpp
+++ b/modules/misc/medialibrary/entities.cpp
@@ -236,6 +236,7 @@ bool Convert( const medialibrary::IMedia* input, vlc_ml_media_t& output )
output.i_duration = input->duration();
output.b_is_favorite = input->isFavorite();
output.i_playcount = input->playCount();
+ output.f_progress = input->progress();
output.i_last_played_date = input->lastPlayedDate();
output.psz_title = strdup( input->title().c_str() );
diff --git a/modules/misc/medialibrary/fs/device.cpp b/modules/misc/medialibrary/fs/device.cpp
index 226f158bad..87227d4b5b 100644
--- a/modules/misc/medialibrary/fs/device.cpp
+++ b/modules/misc/medialibrary/fs/device.cpp
@@ -62,10 +62,13 @@ bool SDDevice::isNetwork() const
return m_isNetwork;
}
-const
-std::string &SDDevice::mountpoint() const
+std::vector<std::string> SDDevice::mountpoints() const
{
- return m_mountpoints[0].mrl;
+ std::vector<std::string> res;
+ res.reserve( m_mountpoints.size() );
+ for ( const auto& m : m_mountpoints )
+ res.push_back( m.mrl );
+ return res;
}
void SDDevice::addMountpoint( std::string mrl )
diff --git a/modules/misc/medialibrary/fs/device.h b/modules/misc/medialibrary/fs/device.h
index ba9eb2822a..b4b0804bb0 100644
--- a/modules/misc/medialibrary/fs/device.h
+++ b/modules/misc/medialibrary/fs/device.h
@@ -43,7 +43,7 @@ public:
bool isRemovable() const override;
bool isPresent() const override;
bool isNetwork() const override;
- const std::string &mountpoint() const override;
+ std::vector<std::string> mountpoints() const override;
void addMountpoint( std::string mrl ) override;
void removeMountpoint( const std::string& mrl ) override;
std::tuple<bool, std::string> matchesMountpoint( const std::string& mrl ) const override;
diff --git a/modules/misc/medialibrary/fs/file.cpp b/modules/misc/medialibrary/fs/file.cpp
index 1dd98a5c06..daa77f99f9 100644
--- a/modules/misc/medialibrary/fs/file.cpp
+++ b/modules/misc/medialibrary/fs/file.cpp
@@ -65,5 +65,15 @@ SDFile::size() const
return 0;
}
+IFile::LinkedFileType SDFile::linkedType() const
+{
+ return IFile::LinkedFileType::None;
+}
+
+const std::string &SDFile::linkedWith() const
+{
+ return m_linkedFile;
+}
+
} /* namespace medialibrary */
} /* namespace vlc */
diff --git a/modules/misc/medialibrary/fs/file.h b/modules/misc/medialibrary/fs/file.h
index bb0525b4d9..41584b2476 100644
--- a/modules/misc/medialibrary/fs/file.h
+++ b/modules/misc/medialibrary/fs/file.h
@@ -39,11 +39,14 @@ public:
unsigned int lastModificationDate() const override;
int64_t size() const override;
inline bool isNetwork() const override { return true; }
+ LinkedFileType linkedType() const override;
+ const std::string &linkedWith() const override;
private:
std::string m_mrl;
std::string m_name;
std::string m_extension;
+ std::string m_linkedFile;
};
} /* namespace medialibrary */
diff --git a/modules/misc/medialibrary/fs/fs.cpp b/modules/misc/medialibrary/fs/fs.cpp
index b8d326a3f0..85323cb231 100644
--- a/modules/misc/medialibrary/fs/fs.cpp
+++ b/modules/misc/medialibrary/fs/fs.cpp
@@ -149,8 +149,8 @@ void SDFileSystemFactory::onDeviceMounted(const std::string& uuid,
}
device->addMountpoint(mountpoint);
}
-
- m_callbacks->onDeviceMounted( *device );
+ fprintf(stderr, "device mounted: %s %s\n", uuid.c_str(), mountpoint.c_str());
+ m_callbacks->onDeviceMounted( *device, mountpoint );
}
void vlc::medialibrary::SDFileSystemFactory::onDeviceUnmounted(const std::string& uuid,
@@ -170,7 +170,7 @@ void vlc::medialibrary::SDFileSystemFactory::onDeviceUnmounted(const std::string
return;
}
device->removeMountpoint(mountpoint);
- m_callbacks->onDeviceUnmounted(*device);
+ m_callbacks->onDeviceUnmounted(*device, mountpoint);
}
std::shared_ptr<IDevice> SDFileSystemFactory::deviceByUuid(const std::string& uuid)
diff --git a/modules/misc/medialibrary/fs/fs.h b/modules/misc/medialibrary/fs/fs.h
index 38867c0ca0..30025e6fb6 100644
--- a/modules/misc/medialibrary/fs/fs.h
+++ b/modules/misc/medialibrary/fs/fs.h
@@ -91,7 +91,7 @@ private:
std::shared_ptr<fs::IDevice>
deviceByUuid(const std::string& uuid);
- bool isStarted() const;
+ bool isStarted() const override;
std::shared_ptr<fs::IDevice> deviceByMrl(const std::string& mrl);
diff --git a/modules/misc/medialibrary/medialib.cpp b/modules/misc/medialibrary/medialib.cpp
index b73cd46c55..8eb4a624dc 100644
--- a/modules/misc/medialibrary/medialib.cpp
+++ b/modules/misc/medialibrary/medialib.cpp
@@ -590,7 +590,7 @@ int MediaLibrary::Control( int query, va_list args )
*va_arg( args, vlc_ml_media_t**) = CreateAndConvert<vlc_ml_media_t>( media.get() );
return VLC_SUCCESS;
}
- case VLC_ML_MEDIA_INCREASE_PLAY_COUNT:
+ case VLC_ML_MEDIA_UPDATE_PROGRESS:
case VLC_ML_MEDIA_GET_MEDIA_PLAYBACK_STATE:
case VLC_ML_MEDIA_SET_MEDIA_PLAYBACK_STATE:
case VLC_ML_MEDIA_GET_ALL_MEDIA_PLAYBACK_STATES:
@@ -961,8 +961,6 @@ medialibrary::IMedia::MetadataType MediaLibrary::metadataType( int meta )
{
case VLC_ML_PLAYBACK_STATE_RATING:
return medialibrary::IMedia::MetadataType::Rating;
- case VLC_ML_PLAYBACK_STATE_PROGRESS:
- return medialibrary::IMedia::MetadataType::Progress;
case VLC_ML_PLAYBACK_STATE_SPEED:
return medialibrary::IMedia::MetadataType::Speed;
case VLC_ML_PLAYBACK_STATE_TITLE:
@@ -1053,7 +1051,6 @@ int MediaLibrary::getMeta( const medialibrary::IMedia& media,
vlc_ml_playback_states_all* res )
{
auto metas = media.metadata();
- res->progress = -1.f;
res->rate = .0f;
res->zoom = -1.f;
res->current_title = -1;
@@ -1069,9 +1066,6 @@ int MediaLibrary::getMeta( const medialibrary::IMedia& media,
switch ( meta.first )
{
- case medialibrary::IMedia::MetadataType::Progress:
- res->progress = atof( meta.second.c_str() );
- break;
case medialibrary::IMedia::MetadataType::Speed:
res->rate = atof( meta.second.c_str() );
break;
@@ -1127,8 +1121,6 @@ int MediaLibrary::setMeta( medialibrary::IMedia& media,
{
using MT = medialibrary::IMedia::MetadataType;
std::unordered_map<MT, std::string> metas;
- if ( values->progress >= .0f )
- metas[MT::Progress] = std::to_string( values->progress );
if ( values->rate != .0f )
metas[MT::Speed] = std::to_string( values->rate );
if ( values->zoom != .0f )
@@ -1163,8 +1155,8 @@ int MediaLibrary::controlMedia( int query, va_list args )
return VLC_EGENERIC;
switch( query )
{
- case VLC_ML_MEDIA_INCREASE_PLAY_COUNT:
- if ( m->increasePlayCount() == false )
+ case VLC_ML_MEDIA_UPDATE_PROGRESS:
+ if ( m->setProgress( va_arg( args, double ) ) == false )
return VLC_EGENERIC;
return VLC_SUCCESS;
case VLC_ML_MEDIA_GET_MEDIA_PLAYBACK_STATE:
diff --git a/src/player/input.c b/src/player/input.c
index 5dc311436d..59b4f7345d 100644
--- a/src/player/input.c
+++ b/src/player/input.c
@@ -604,9 +604,9 @@ vlc_player_input_HandleTitleEvent(struct vlc_player_input *input,
if (input->ml.restore == VLC_RESTOREPOINT_POSITION &&
input->ml.states.current_title >= 0 &&
(size_t)input->ml.states.current_title == ev->selected_idx &&
- input->ml.states.progress > .0f)
+ input->ml.pos > .0f)
{
- input_SetPosition(input->thread, input->ml.states.progress, false);
+ input_SetPosition(input->thread, input->ml.pos, false);
}
/* Reset the wanted title to avoid forcing it or the position
* again during the next title change
@@ -945,10 +945,10 @@ vlc_player_input_New(vlc_player_t *player, input_item_t *item)
input->ml.states.current_video_track =
input->ml.states.current_audio_track =
input->ml.states.current_subtitle_track = NULL;
- input->ml.states.progress = -1.f;
input->ml.restore = VLC_RESTOREPOINT_NONE;
input->ml.restore_states = false;
input->ml.delay_restore = false;
+ input->ml.pos = -1.f;
input->thread = input_Create(player, input_thread_Events, input, item,
player->resource, player->renderer);
diff --git a/src/player/medialib.c b/src/player/medialib.c
index a5f8a32bd9..b4deb94e0a 100644
--- a/src/player/medialib.c
+++ b/src/player/medialib.c
@@ -53,7 +53,6 @@ vlc_player_input_RestoreMlStates(struct vlc_player_input* input, bool force_pos)
vlc_ml_release(media);
return;
}
- vlc_ml_release(media);
input->ml.restore = (restore_pos == VLC_PLAYER_RESTORE_PLAYBACK_POS_ALWAYS) ?
VLC_RESTOREPOINT_TITLE : VLC_RESTOREPOINT_NONE;
@@ -64,11 +63,16 @@ vlc_player_input_RestoreMlStates(struct vlc_player_input* input, bool force_pos)
*/
if (restore_pos == VLC_PLAYER_RESTORE_PLAYBACK_POS_ALWAYS &&
input->ml.states.current_title == -1 &&
- input->ml.states.progress > .0f)
- input_SetPosition(input->thread, input->ml.states.progress, false);
+ media->f_progress > .0f)
+ input_SetPosition(input->thread, media->f_progress, false);
else if (restore_pos == VLC_PLAYER_RESTORE_PLAYBACK_POS_ASK &&
- input->ml.states.progress > .0f)
+ media->f_progress > .0f)
+ {
input->ml.delay_restore = true;
+ input->ml.pos = media->f_progress;
+ }
+
+ vlc_ml_release(media);
if (!restore_states)
return;
@@ -119,26 +123,6 @@ vlc_player_input_RestoreMlStates(struct vlc_player_input* input, bool force_pos)
}
}
-static const float beginning_of_media_percent = .05f;
-static const int64_t beginning_of_media_sec = 60;
-
-static int
-beginning_of_media(struct vlc_player_input *input)
-{
- return input->position <= beginning_of_media_percent &&
- input->time < VLC_TICK_FROM_SEC(beginning_of_media_sec);
-}
-
-static const float end_of_media_percent = .95f;
-static const int64_t end_of_media_sec = 60;
-
-static int
-end_of_media(struct vlc_player_input *input)
-{
- return input->position >= end_of_media_percent &&
- input->length - input->time < VLC_TICK_FROM_SEC(end_of_media_sec);
-}
-
static bool
vlc_player_UpdateMediaType(const struct vlc_player_input* input,
vlc_medialibrary_t* ml, vlc_ml_media_t* media)
@@ -214,24 +198,7 @@ vlc_player_UpdateMLStates(vlc_player_t *player, struct vlc_player_input* input)
}
assert(media->i_type != VLC_ML_MEDIA_TYPE_UNKNOWN);
- /* If we reached end of the media, bump the play count & the media in the
- * history */
- if (end_of_media(input))
- vlc_ml_media_increase_playcount(ml, media->i_id);
-
- if (beginning_of_media(input) || end_of_media(input))
- {
- /* Ensure we remove any previously saved position to allow the playback
- * of this media to restart from the begining */
- if (input->ml.states.progress >= .0f )
- {
- vlc_ml_media_set_playback_state(ml, media->i_id,
- VLC_ML_PLAYBACK_STATE_PROGRESS, NULL );
- input->ml.states.progress = -1.f;
- }
- }
- else
- input->ml.states.progress = input->position;
+ vlc_ml_media_update_progress( ml, media->i_id, input->position );
/* If the value changed during the playback, update it in the medialibrary.
* If not, set each state to their "unset" values, so that they aren't saved
diff --git a/src/player/player.h b/src/player/player.h
index 87b9e43033..077086199a 100644
--- a/src/player/player.h
+++ b/src/player/player.h
@@ -114,6 +114,7 @@ struct vlc_player_input
VLC_RESTOREPOINT_POSITION,
VLC_RESTOREPOINT_NONE,
} restore;
+ float pos;
bool restore_states;
bool delay_restore;
} ml;
More information about the vlc-commits
mailing list