[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Gnash-commit] gnash ChangeLog libbase/FLVParser.cpp server/vi...
From: |
strk |
Subject: |
Re: [Gnash-commit] gnash ChangeLog libbase/FLVParser.cpp server/vi... |
Date: |
Thu, 6 Sep 2007 14:44:46 +0200 |
On Thu, Sep 06, 2007 at 12:21:06PM +0000, Tomas Groth wrote:
> // Check if there is enough data to parse the body of the frame
> if (!_lt.isPositionConfirmed(_lastParsedPosition+15+bodyLength)) return
> false;
>
> + _lastParsedPosition += 15 + bodyLength;
What's 15 ? Could you use a symbolic name for constants ?
> if (tag[0] == AUDIO_TAG) {
> FLVAudioFrame* frame = new FLVAudioFrame;
...
> FLVVideoFrame* frame = new FLVVideoFrame;
...
> _videoInfo = new FLVVideoInfo(codec, width, height, 0
> /*frameRate*/, 0 /*duration*/);
...
> amf::AMF* amfParser = new amf::AMF();
Please document ownership whenever allocating new resources.
Best if stuffed into an auto_ptr.
--strk;