lvqcl wrote:

> Some debugging tools don't like if a code uses unitialized variables.
> The code in cpu.c uses the value of fxsr.buff[50], so it makes sense
> to explicitely initialize it (because fxsave can fail to do it).

Your change:

                fxsr.buff[50] = 0;

only changes one element in that 128 work buffer. Why explicitly initialize
onky the 50th element and not all of them?

My approach would be:

     memset (fxsr.buff, 0, sizeof (fxsr.buff));

What do you think?

Erik
-- 
----------------------------------------------------------------------
Erik de Castro Lopo
http://www.mega-nerd.com/
_______________________________________________
flac-dev mailing list
[email protected]
http://lists.xiph.org/mailman/listinfo/flac-dev

Reply via email to