Rafaël Carré [Wed, 10 Oct 2007 23:04:09 +0000 (23:04 +0000)]
services_discovery: storing the category & onelevel playlist items with the services_discovery_t is not needed anymore, simplify that.
put the services_discovery_t** in the public playlist_t to save one unchecked malloc() and the corresponding free()
Qt4 - use a docked playlist, remove dead code, don't reinvent the wheel, remove the behaviour where media library hides the video. Don't split the menu between two dialogs, this is weird and anti sane GUI guidelines.
Use enums when you can.
Add the popup menu to the always video cone.
And more...
Bernie Purcell [Wed, 10 Oct 2007 03:41:17 +0000 (03:41 +0000)]
Attempt to stop subtitles going off screen in some formats, when
width and height are specified in the subtitles stream. This may
result in chunkier text in some instances, but it should get it
back on screen at least. Will look into smoothing it out next.
macosx: Multiple GUI improvements follow j-b's and my approach to synchronise the QT4 and OSX intfs, more to come
- "Use DVD Menus" is inverted now, so it's "No DVD Menus"
- 3 dots at the end of a menu-item's caption mean that the entry will trigger a window (following Apple's HIGL)
- removed the entries for "Report a Bug" and "License". The first isn't necessary as we got enough of them and users doing useable reports know how to do, the latter will re-appear in the to-be-re-designed About panel.
- additionally, removed useless occurences of vlc_bool_t in favour of BOOL
- preparations to add fps to Statistics
Qt4 - no more timeElapsed/timeReamining menu for a boolean option. Right and left click toggle it, while double click opens the goto one. kudos to the KDE team.
Jean-Paul Saman [Mon, 8 Oct 2007 20:46:23 +0000 (20:46 +0000)]
Run FontBuilder thread after Freetype has been initialized and no error paths are can be taken. This should solve the crash reported by several people with the video title (or meta-name) appearing at start of the movie.
The freetype module starts another thread called FontBuilder and depending on timing this creates a crash or works allright. The crash is triggered by the font not being found. This patch tries to load the font first before the FontBuilder thread is started.
I was unable to reproduce the crash with the patch applied. Without the patch the crash is reproducable.
As discussed on the mailinglist the FontBuilder thread doesn't depend on the freetype initialisation and rearanging the order should do no harm.
Laurent Aimar [Mon, 8 Oct 2007 18:43:16 +0000 (18:43 +0000)]
Let the input handle *extra* meta data and attachments for flac audio.
(We still parse the flac meta data as a safety, in case taglib support is not there)
Laurent Aimar [Mon, 8 Oct 2007 18:30:11 +0000 (18:30 +0000)]
Es out now takes care of settings the audio replay gain from meta data
if the demuxer has not set them.
Added a demux2 control DEMUX_HAS_UNSUPPORTED_META.
A demuxer should return true only when the format support extra meta
data (id3/ap2/...) but it does not read them itself. The input will then
call a "meta reader" (demuxer modifications on the way).
Rafaël Carré [Mon, 8 Oct 2007 00:36:36 +0000 (00:36 +0000)]
audioscrobbler: use condition variables to be signaled when there are songs to be submitted.
checks if playlist has been stopped in the input "state" callback
holds the p_sys mutex only when writing/reading audioscrobbler_song_t data
Rafaël Carré [Sun, 7 Oct 2007 22:07:07 +0000 (22:07 +0000)]
audioscrobbler: rewrite for new protocol 1.2
allocate as much as we can on the stack, to avoid numerous malloc() & free()
fix memory leaks by the same way
metadata reading is now performed exclusively from the callbacks
don't use linked lists anymore, but a queue of a fixed size of 50 songs, i.e. the maximum we can transmit in one submission.
properly unloads the module when encountering a fatal error, like no username/pw, no memory, VLC being banned by last.fm servers ...
Rafaël Carré [Sun, 7 Oct 2007 17:54:41 +0000 (17:54 +0000)]
fix youtube lua probing, avoid infinite loops
homepage: use %? to represent '?'
other: probe is not successfull anymore for urls like http://10.0.0.1/get_video?xxxx&origin=youtube.com