[vlc-devel] DASH access plugin

[Christopher Müller]

Hi,

I have formatted the source as git patches as discussed and fixed the minor
issues that you have mentioned.

> > 	Hello,
> >
> > Le dimanche 20 février 2011 15:46:36 Christopher Müller, vous avez écrit
:
> > > > Are you too shy to ask for a proper code review and submit the code?
> > > > Or should I infer that you are permanently done with your project
> > > > and you won't be updating the code yourselves any further?
> > >
> > > Sorry but I thought that this is the right place for submitting new
> > > source code?
> >
> > Oh sure, this would be the right place. From your wording, it sounded
> > that you expected some other developers to take your code and merge it
> to VLC.
> >
> > If you want to submit the code, you should format it as a (set of)
> > git- formatted patches on top of VLC 1.2.0.
> >
> > In any case, there are a few minor issues in my opinion:
> >
> > * Seek returns VLC_SUCCESS even though it is not implemented.
> >
> > * Preprocessor constants should not have trailing semi-colons.
> >
> > * Indentation does not follow the VLC convention (4 white spaces).
> >
> > * Source code files should not be executable.
> >
> > * "https" scheme is registered, but apparently not implemented??
> >
> > * You should really use net_ConnectTCP(), net_Read(), net_Write() and
> > net_Close() from <vlc_network.h> instead of custom IPv4-only non-
> > Windows blocking-mode socket code.
> >
> >
> > And last, I don't know how practical, but you should try to play nice
> > with
> the
> > "normal" HTTP plugin.
Unfortunately this is not possible because we need HTTP Pipelining.
> >
> Ok this should not be that hard to fix, unfortunately I have no time this
week
> but I will do it in the first week of March and inform you when it's done.
> 

Best Regards
Christopher
-------------- next part --------------
A non-text attachment was scrubbed...
Name: dash_access.patch.tar.gz
Type: application/x-gzip
Size: 18068 bytes
Desc: not available
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20110302/a8fe1de0/attachment.bin>