[vlc-commits] [Git][videolan/vlc][master] 23 commits: core: move main playlist initialisation

Thu Oct 10 15:09:41 UTC 2024

Thomas Guillem pushed to branch master at VideoLAN / VLC

Commits:
e6d41d20 by Thomas Guillem at 2024-10-10T14:46:20+00:00
core: move main playlist initialisation

No functional changes.

- - - - -
7accc911 by Thomas Guillem at 2024-10-10T14:46:20+00:00
playlist: hide vlc_playlist_Preparse()

Not used externally, let's the auto preparser do its job.

- - - - -
46cf3d1a by Thomas Guillem at 2024-10-10T14:46:20+00:00
playlist: merge Preparse and AutoPreparse

- - - - -
d6b182b9 by Thomas Guillem at 2024-10-10T14:46:20+00:00
playlist: rename and expose vlc_playlist_recursive_parsing

Will be used in public in next commits.

- - - - -
c3346925 by Thomas Guillem at 2024-10-10T14:46:20+00:00
preparser: configure threads and timeout externally

- - - - -
6b6f985e by Thomas Guillem at 2024-10-10T14:46:20+00:00
playlist: create its own preaser

A playlist should not use the main preparser by default, but a private
preparser. This is not an issue right now since there is only one used
playlist instance.

- - - - -
e0afa8fe by Thomas Guillem at 2024-10-10T14:46:20+00:00
macosx: release the parser before creating a new one

- - - - -
a8c4e108 by Thomas Guillem at 2024-10-10T14:46:20+00:00
macosx: remove preparseInputItem

Use parseInputItem instead, that does the same thing but using the
parser directly.

- - - - -
e8e74186 by Thomas Guillem at 2024-10-10T14:46:20+00:00
macosx: fix deadlock in case of failure

- - - - -
57d44e33 by Thomas Guillem at 2024-10-10T14:46:20+00:00
macosx: add getNetworkPreparser()

- - - - -
3e6effea by Thomas Guillem at 2024-10-10T14:46:20+00:00
macosx: use the network preparser for media sources

- - - - -
a22a9bdd by Thomas Guillem at 2024-10-10T14:46:20+00:00
qt: rename getPreparser()

This preparser will only be used for network browsing.

- - - - -
752e267a by Thomas Guillem at 2024-10-10T14:46:20+00:00
qt: use a specific preparser for network browsing

In order to not conflict the with the playlist parsing, specially
since network browsing can take a long time (it has no timeout but can
be cancelled by the user).

- - - - -
7196b738 by Thomas Guillem at 2024-10-10T14:46:20+00:00
qt: use the parser directly from the info dialog

No need to use the main preparser (from an executor thread) for parsing a
specific file. This will make the dialog parsing more reactive if the
main preparser is in use.

- - - - -
c627a491 by Thomas Guillem at 2024-10-10T14:46:20+00:00
qt: simplify PlayerController::requestArtUpdate

Since it is always called with b_forced = true and a valid item.

- - - - -
67b5ae75 by Thomas Guillem at 2024-10-10T14:46:20+00:00
qt: reindent after previous commit

No functional changes.

- - - - -
0d7ff30e by Thomas Guillem at 2024-10-10T14:46:20+00:00
qt: use a specific preparser for art fetching

But do not create it systematically as this code is only triggered with
a specific user action.

- - - - -
32e2a714 by Thomas Guillem at 2024-10-10T14:46:20+00:00
qt: use a default timeout for art fetching

Since it's not cancellable by the user, it's better to put a timeout.

- - - - -
b3fb32b0 by Thomas Guillem at 2024-10-10T14:46:20+00:00
test: use vlc_preparser_New directly

- - - - -
e863a52d by Thomas Guillem at 2024-10-10T14:46:20+00:00
lib: use an internal preparser

Do a lazy initialisation as the preparser is not necessarily required by
the user.

This commit is temporary and will be replaced by the future
libvlc_parser_t API.

It is done now to remove libvlc_GetMainPreparser().

- - - - -
62ed634f by Thomas Guillem at 2024-10-10T14:46:20+00:00
core: remove libvlc_GetMainPreparser()

- - - - -
32bd642b by Thomas Guillem at 2024-10-10T14:46:20+00:00
lib: use an internal thumbnailer

Do a lazy initialisation as the thumbnailer is not necessarily required by
the user.

This commit is temporary and will be replaced by the future
libvlc_thumbnailer_t API.

It is done now to remove the core thumbnailer.

- - - - -
6210ebe9 by Thomas Guillem at 2024-10-10T14:46:20+00:00
core: remove the internal thumbnailer

- - - - -

30 changed files:

- include/vlc_playlist.h
- include/vlc_preparser.h
- lib/core.c
- lib/libvlc_internal.h
- lib/media.c
- modules/gui/macosx/library/VLCInputItem.h
- modules/gui/macosx/library/VLCInputItem.m
- modules/gui/macosx/library/media-source/VLCMediaSource.h
- modules/gui/macosx/library/media-source/VLCMediaSource.m
- modules/gui/macosx/library/media-source/VLCMediaSourceBaseDataSource.m
- modules/gui/macosx/library/media-source/VLCMediaSourceProvider.m
- modules/gui/macosx/main/VLCMain.h
- modules/gui/macosx/main/VLCMain.m
- modules/gui/macosx/panels/VLCInformationWindowController.m
- modules/gui/qt/dialogs/dialogs_provider.cpp
- modules/gui/qt/dialogs/dialogs_provider.hpp
- modules/gui/qt/maininterface/mainctx.cpp
- modules/gui/qt/maininterface/mainctx.hpp
- modules/gui/qt/network/networkmediamodel.cpp
- modules/gui/qt/player/player_controller.cpp
- modules/gui/qt/player/player_controller.hpp
- modules/gui/qt/player/player_controller_p.hpp
- modules/gui/qt/widgets/native/interface_widgets.cpp
- src/interface/interface.c
- src/libvlc.c
- src/libvlc.h
- src/libvlccore.sym
- src/playlist/playlist.c
- src/playlist/playlist.h
- src/playlist/preparse.c

The diff was not included because it is too large.

View it on GitLab: https://code.videolan.org/videolan/vlc/-/compare/bbc9223fcd06884409dad1597104bd6d8021332d...6210ebe9805100f5ca2b4cd5be61b52a6d1f5955

-- 
View it on GitLab: https://code.videolan.org/videolan/vlc/-/compare/bbc9223fcd06884409dad1597104bd6d8021332d...6210ebe9805100f5ca2b4cd5be61b52a6d1f5955
You're receiving this email because of your account on code.videolan.org.

VideoLAN code repository instance