[FFmpeg-devel] [PATCH] RDT/Realmedia patches #2

Michael Niedermayer michaelni
Sun Nov 16 19:20:31 CET 2008


On Sun, Nov 16, 2008 at 11:15:56AM -0500, Ronald S. Bultje wrote:
> Hi,
> 
> On Sun, Nov 16, 2008 at 10:32 AM, Michael Niedermayer <michaelni at gmx.at> wrote:
> > On Sat, Nov 15, 2008 at 03:51:17PM -0500, Ronald S. Bultje wrote:
> >> @@ -175,10 +175,10 @@
> >>
> >>  int
> >>  ff_rdt_parse_header(const uint8_t *buf, int len,
> >> -                    int *set_id, int *seq_no, int *stream_id,
> >> +                    int *pset_id, int *seq_no, int *pstream_id,
> >>                      int *is_keyframe, uint32_t *timestamp)
> >
> > if you want to prefix pointers by p in a function, fine but i think
> > it should be done consistently (within that function), that is for all pointer
> > arguments. And in a seperate patch
> 
> Attached.

ok

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

There will always be a question for which you do not know the correct awnser.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20081116/4d9d4193/attachment.pgp>



More information about the ffmpeg-devel mailing list