[vlc-devel] vlc-devel Digest, Vol 56, Issue 26
Shelley Horwitz
shelley at sjcomm.com
Fri Jan 6 20:15:09 CET 2012
With respect to Message 3 (highlighted below), VLC Player integrates soundtracks.
Anyway, can you at least create a file in the same folder that has the bookmarks and check for that file?
Shelley
________________________________
From: "vlc-devel-request at videolan.org" <vlc-devel-request at videolan.org>
To: vlc-devel at videolan.org
Sent: Friday, January 6, 2012 3:00 AM
Subject: vlc-devel Digest, Vol 56, Issue 26
Send vlc-devel mailing list submissions to
vlc-devel at videolan.org
To subscribe or unsubscribe via the World Wide Web, visit
http://mailman.videolan.org/listinfo/vlc-devel
or, via email, send a message with subject or body 'help' to
vlc-devel-request at videolan.org
You can reach the person managing the list at
vlc-devel-owner at videolan.org
When replying, please edit your Subject line so it is more specific
than "Re: Contents of vlc-devel digest..."
Today's Topics:
1. Re: [PATCH] LUA: Remove protocol from vlc.path after redirect
( R?mi Denis-Courmont)
2. Re: [PATCH] Change to npapi for giving access to media meta
thru new input.meta(i) method. (jm)
3. Re: Suggestion and Request (Kaarlo R?ih?)
4. Re: [PATCH 3/3] Replace goto with gotoitem. goto is reserved
in lua 5.2. luac will fail to compile the lua source files
without this change. (Antoine Cellerier)
----------------------------------------------------------------------
Message: 1
Date: Fri, 6 Jan 2012 10:44:10 +0200
From: " R?mi Denis-Courmont" <remi at remlab.net>
To: vlc-devel at videolan.org
Subject: Re: [vlc-devel] [PATCH] LUA: Remove protocol from vlc.path
after redirect
Message-ID: <201201061044.12586.remi at remlab.net>
Content-Type: Text/Plain; charset="iso-8859-1"
Le vendredi 6 janvier 2012 01:38:20 Gaurav Narula, vous avez ?crit :
> ---
> modules/access/http.c | 3 ++-
> 1 files changed, 2 insertions(+), 1 deletions(-)
>
> diff --git a/modules/access/http.c b/modules/access/http.c
> index d32a0a9..deaab84 100644
> --- a/modules/access/http.c
> +++ b/modules/access/http.c
> @@ -581,7 +581,8 @@ connect:
> goto error;
> }
> free( p_access->psz_location );
> - p_access->psz_location = strdup( p_sys->psz_location );
> + char *slash = strstr(p_sys->psz_location, "://");
> + p_access->psz_location = (slash) ? strdup( slash + 3 ) :
> p_sys->psz_location; /* Clean up current Open() run */
> vlc_UrlClean( &p_sys->url );
> http_auth_Reset( &p_sys->auth );
This won't work if the redirection alternates between secure and non-secure
modes or vice versa.
--
R?mi Denis-Courmont
http://www.remlab.net/
http://fi.linkedin.com/in/remidenis
------------------------------
Message: 2
Date: Fri, 06 Jan 2012 10:05:05 +0100
From: jm <jean-michel.lambert7 at wanadoo.fr>
To: Mailing list for VLC media player developers
<vlc-devel at videolan.org>
Subject: Re: [vlc-devel] [PATCH] Change to npapi for giving access to
media meta thru new input.meta(i) method.
Message-ID: <4F06B941.201 at wanadoo.fr>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
On 12/30/2011 02:21 AM, Jean-Baptiste Kempf wrote:
> On Wed, Dec 28, 2011 at 11:41:54PM +0100, JM Lambert wrote :
>> +
> Why?
>
>> +
>> +
> Idem?
>
>> - "none",
>> + "meta",
> That doesn't break anything? Instead of adding it?
No to me, the "none" is to be used when there is no method (that was the
case prior to adding the meta() method)
> No, to
>
>> + VlcPlugin* p_plugin = getPrivate<VlcPlugin>();
>> + libvlc_media_player_t *p_md = p_plugin->getMD();
>> + libvlc_media_t * p_media = libvlc_media_player_get_media(p_md);
> No tabs.
> If p_md was NULL, wouldn't this be bad?
Corrected
>
>> + if( !p_md )
>> + RETURN_ON_ERROR;
>> switch( index )
>> {
>> - case ID_none:
>> - return INVOKERESULT_NO_SUCH_METHOD;
>> + case ID_input_meta:
>> + if( argCount == 1)
> tabs
removed
>
>> + {
>> + char *info;
> const?
corrected
>
>> + int i_metaID, i;
> i is used?
removed i
>
> Btw, is this the best way to expose this API?
that's the simplest and does not require rebuild if fields are aded to
meta. Which other way do you recommend ?
>
> Best regards,
>
------------------------------
Message: 3
Date: Fri, 6 Jan 2012 11:54:03 +0200
From: Kaarlo R?ih? <kaarlo.raiha at gmail.com>
To: Mailing list for VLC media player developers
<vlc-devel at videolan.org>
Subject: Re: [vlc-devel] Suggestion and Request
Message-ID:
<CAMKrNtw72TKMxoBztzHSLJE6muKb3e0kb7nJKh2e99HDJEfOrw at mail.gmail.com>
Content-Type: text/plain; charset="iso-8859-1"
2012/1/5 Shelley Horwitz <shelley at sjcomm.com>
> I have one suggestion for improving the User Interface on the VLC Player
> and one request for a new feature.
>
> Suggestion: Please add a Bookmark Button to the controls so I can click
> at a particular place in a playback and DO NOT have to go to the menu and
> click several times to save a bookmark.
>
> Request: Allow me to save a set of bookmarks with my videos (avi, wmv,
> mpg, etc.). When I create a set of bookmarks, I would like to save them
> with my video so I can use them again whenever I play it. I would prefer to
> have the bookmark "file" integrated with the video itself so that I do not
> have to keep track of 2 files when I move videos around my libraries.
>
VLC cannot write metadata to all files (some formats don't support
metadata, and VLC doesn't always have write access to files). And there
isn't a standard for this kind of behavior which would make this quite hard
to do correctly (not break files with other players, and to allow other
players to use those same bookmarks).
>
> Thank for a great product.
>
> Shelley J. Horwitz
> shelley at sjcomm.com
>
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> http://mailman.videolan.org/listinfo/vlc-devel
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20120106/b9c60f87/attachment-0001.html>
------------------------------
Message: 4
Date: Fri, 6 Jan 2012 11:13:13 +0100
From: Antoine Cellerier <dionoea at videolan.org>
To: vlc-devel at videolan.org
Subject: Re: [vlc-devel] [PATCH 3/3] Replace goto with gotoitem. goto
is reserved in lua 5.2. luac will fail to compile the lua source files
without this change.
Message-ID: <20120106101313.GA16619 at chewa.net>
Content-Type: text/plain; charset=utf-8
On Thu, Jan 05, 2012, Rafa?l Carr? wrote:
> I noticed this:
>
> commit 62d4d133a9b5e66bba5545338219499133909a9b
> Author: Antoine Cellerier <antoine at macbook.(none)>
> Date: Mon Dec 14 23:33:17 2009 +0100
>
> repeat is a reserved lua keyword
>
> Provide playlist._repeat() instead (and keep the old .repeat in case
> anybody was using playlist["repeat"] to workaround the reserved keyword
> issue).
>
> IIUC it only applies to the function name though, not to the string
> interpreted by cli.lua
>
> So Kelly, your change shouldn't affect "goto" but only vlc.playlist.goto
It might also be a good idea to still map the function to
vlc.playlist.goto so lua 5.1 users which use that function can continue
doing so ( => keeping backwards compatibility is a good idea). It is
still completely valid to name a function that way. The only invalid
thing is to call it using "vlc.playlist.goto" in lua 5.2 code.
--
Antoine Cellerier
dionoea
------------------------------
_______________________________________________
vlc-devel mailing list
vlc-devel at videolan.org
http://mailman.videolan.org/listinfo/vlc-devel
End of vlc-devel Digest, Vol 56, Issue 26
*****************************************
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20120106/20214f31/attachment.html>
More information about the vlc-devel
mailing list