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;


_______________________________________________
Gnash-commit mailing list
Gnash-commit@gnu.org
http://lists.gnu.org/mailman/listinfo/gnash-commit

Reply via email to