[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [bongo-devel] Re: Stream metadata
From: |
Daniel Brockman |
Subject: |
Re: [bongo-devel] Re: Stream metadata |
Date: |
Mon, 09 Apr 2007 14:49:57 +0200 |
User-agent: |
Gnus/5.11 (Gnus v5.11) Emacs/22.0.92 (gnu/linux) |
address@hidden (Daniel Jensen) writes:
> Daniel Brockman <address@hidden> writes:
>
>> Anyway, please consider the following patch. I believe it
>> is fit for installation, but you may want to make changes.
>
> Good work! I have only commented on a few details below.
Thank you. :-)
>> + (cond ((or stream-name stream-genre stream-part-title)
>> + `((artist . unknown)
>> + (album . unknown)
>> + (stream (uri . ,file-name)
>> + (uri-title . ,uri-title)
>> + (name . ,stream-name)
>> + (genre . ,stream-genre)
>> + (part-title . ,stream-part-title))))
>> + (uri-title
>> + `((artist . unknown)
>> + (album . unknown)
>> + (track (title . ,(or uri-title stream-name)))))
>> + (t
>> + (bongo-infoset-from-file-name file-name)))))
>
> In `bongo-uri-track-infoset'. The stream-name property is nil in the
> uri-title clause, so you can remove it. Or maybe, in case something
> else was intended, this should be reworked?
Nice catch! I used to have only `stream-part-title' in the
first clause, so that's just a leftover.
>> + (when (or stream-name stream-genre stream-part-title)
>> + (bongo-stream-metadata-changed player)))))
>
> In the VLC backend, it should be `bongo-player-metadata-changed' here.
Thanks. The old name appeared in a few other places as well.
>> (with-bongo-buffer
>> - (apply 'bongo-insert-line 'bongo-file-name uri
>> - (when title
>> - (list 'bongo-infoset `((artist . unknown)
>> - (album . unknown)
>> - (track (title . ,title)))))))
>> + (bongo-insert-line 'bongo-file-name uri))
>
> In `bongo-insert-uri', this ignores the user title completely.
Oops!
> I think it's supposed to be
>
> (apply 'bongo-insert-line 'bongo-file-name uri
> (when title
> (list 'bongo-uri-title
> (and (not (equal title "")) title))))
Thanks, I did this:
(apply 'bongo-insert-line 'bongo-file-name uri
(when (and title (not (equal title "")))
(list 'bongo-uri-title title)))
> That's all.
Again, thanks a lot for your corrections!
I'll go ahead and install this now. I think it's a good start.
We can deal with any remaining issues as they come up.
--
Daniel Brockman <address@hidden>
- [bongo-devel] Re: Stream metadata, Daniel Jensen, 2007/04/01
- Re: [bongo-devel] Re: Stream metadata, Daniel Brockman, 2007/04/01
- [bongo-devel] Re: Stream metadata, Daniel Jensen, 2007/04/02
- Re: [bongo-devel] Re: Stream metadata, Daniel Brockman, 2007/04/02
- [bongo-devel] Re: Stream metadata, Daniel Jensen, 2007/04/03
- Re: [bongo-devel] Re: Stream metadata, Daniel Brockman, 2007/04/03
- [bongo-devel] Re: Stream metadata, Daniel Jensen, 2007/04/04
- Re: [bongo-devel] Re: Stream metadata, Daniel Brockman, 2007/04/08
- [bongo-devel] Re: Stream metadata, Daniel Jensen, 2007/04/09
- Re: [bongo-devel] Re: Stream metadata,
Daniel Brockman <=