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