gnash-commit
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Gnash-commit] gnash ChangeLog server/cxform.cpp server/cxform...


From: strk
Subject: Re: [Gnash-commit] gnash ChangeLog server/cxform.cpp server/cxform...
Date: Wed, 21 May 2008 09:35:23 +0200

On Wed, May 21, 2008 at 07:26:09AM +0000, Zou Lunkai wrote:

>      // TODO: temp hack, should drop!
> +    // why read_xxx(stream *) and read_xxx(stream &) are mixed together?
>      void read_rgb(stream* in) { read_rgb(*in); }
>  
>      // TODO: temp hack, should drop!
> +    // why read_xxx(stream *) and read_xxx(stream &) are mixed together?
>      void read_rgba(stream* in) { read_rgba(*in); }

Old code passed by pointer, but since NULL wouldn't be valid I had new code
pass by ref, dropping the by-pointer should make it easy to find callers
to update (if any).

--strk;




reply via email to

[Prev in Thread] Current Thread [Next in Thread]