[Vorbis-dev] vorbis_fpu_setround()

[Gábor Kovács]

Hi,

I think there is a minor bug in vorbis_fpu_setround().
The "temp" is an output argument instead of input.
Intel compiler warning:

lib/os.h(102): warning #592: variable "temp" is used before its value is set
            "fldcw %1\n\t":"=m"(ret):"m"(temp): "dx");

I would recommend something like:
            "fldcw %1\n\t":"=m"(ret),"=m"(temp): : "dx");

Best regards,
Gábor
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.xiph.org/pipermail/vorbis-dev/attachments/20111107/a781d873/attachment.htm