[vlc-devel] [PATCH 3/3] dash added resolution interpretation

[Christopher Müller]

> Von: vlc-devel-bounces at videolan.org [mailto:vlc-devel-
> bounces at videolan.org] Im Auftrag von Hugo Beauzée-Luyssen
> Gesendet: Sonntag, 29. Jänner 2012 01:43
> An: Mailing list for VLC media player developers
> Betreff: Re: [vlc-devel] [PATCH 3/3] dash added resolution
> interpretation
> 
> Hello,
> 
> On Sat, Jan 28, 2012 at 10:54 PM, Frederic YHUEL <fyhuel at viotech.net>
> wrote:
> > Hello Christopher,
> >
> > I'm working to provide a solution to the problem of resolution
> > switching, which includes the support of fragmented MP4 into the VLC
> > demux (see my patch on this very mailing list).
> >
> > But the VLC MP4 demux needs to seek. Hugo told me he might add the
> > seeking capability to DASH, but I don't know if he actually have the
> > time / energy to do it. Hugo?
> >
> 
> I have started but it's not finished yet. Should be working within next
> week depending on my available time.
> 
> > Anyway... I just wanted to underline the fact that it would be useful
> > addition to the DASH module.
> >
> > Best Regards,
> >
> > PS: Just checking... you did not send your first patch (Parsing and
> > management of the isoffmain profile) on purpose, or is it a git typo?
> >
> 
> About the patch:
> I'm not sure camel case is used anywhere else for variable names, dash-
> prefwidth should be used instead of prefWidth Also, texts should be
> externalised and declared using N_ macro, in order to be translated.
> As Frederic said, it seems the IsofMainParser class patch is missing.
> Adding const qualifier to getter would be nice, but despite this, I've
> nothing more to say!
> 

Thx changed.

BR
Chris
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0003-dash-added-resolution-interpretation.patch
Type: application/octet-stream
Size: 25909 bytes
Desc: not available
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20120129/27e69aa2/attachment.obj>