gnash-commit
[Top][All Lists]
Advanced

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

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


From: strk
Subject: Re: [Gnash-commit] gnash ChangeLog server/stream.h server/stream.cpp
Date: Fri, 24 Aug 2007 09:55:23 +0200

On Fri, Aug 24, 2007 at 05:45:52AM +0000, Chad Musick wrote:

> +                float   read_short_fixed();

Please document the functions you add, a single line would do.

> +        float   stream::read_short_fixed()
> +        {
> +                m_unused_bits = 0;
> +                return (float) m_input->read_le16();
> +        }
>  
>       void    stream::align() { m_unused_bits = 0; m_current_byte = 0; }

Should you call align() instead of just setting m_unused_bits to 0 ?

--strk;




reply via email to

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