Opened 4 months ago

Last modified 4 months ago

#15221 new bug

MediaPlayer playlist should not recalculate file metadata in Draw()

Reported by: KapiX Owned by: stippi
Priority: normal Milestone: Unscheduled
Component: Applications/MediaPlayer Version: R1/Development
Keywords: Cc:
Blocked By: Blocking:
Has a Patch: no Platform: All

Description

PlaylistItem tries to cache metadata using attributes, but that doesn't work on read-only volumes. This leads playlist to reopen the file each time it wants to Draw an item, which sometimes blocks playlist window for a few seconds.

Item::Duration() - https://git.haiku-os.org/haiku/tree/src/apps/mediaplayer/playlist/PlaylistItem.cpp#n145

Item::Draw() - https://git.haiku-os.org/haiku/tree/src/apps/mediaplayer/playlist/PlaylistListView.cpp#n137

Change History (1)

comment:1 by leavengood, 4 months ago

Well it should at least cache the duration in memory, that seems like a bad idea to hit the attribute on each draw, let alone reloading the file...

Note: See TracTickets for help on using tickets.