Mathias Fröhlich <[EMAIL PROTECTED]> writes:

> By the way: the right fix would be:
>
> float
> XDR_decode_float ( const xdr_data_t & f_Val )
> {
>     union {
>       float f;
>       xdr_data_t x;
>     } tmp;
>     tmp.x = XDR_decode_int32 (f_Val);
>     return tmp.f;
> }
>
> Please use this one. And I believe, without looking into the code,
> that there are likely more of them ...

well, this is c++ so the appropriate fix would be

float
XDR_decode_float ( const xdr_data_t & f_Val )
{
    return (static_cast<float> (XDR_decode_int32 (f_Val)));
}

i.e., we should be using static_cast.

--alex--

-- 
| I believe the moment is at hand when, by a paranoiac and active |
|  advance of the mind, it will be possible (simultaneously with  |
|  automatism and other passive states) to systematize confusion  |
|  and thus to help to discredit completely the world of reality. |

_______________________________________________
Flightgear-devel mailing list
Flightgear-devel@flightgear.org
http://mail.flightgear.org/mailman/listinfo/flightgear-devel
2f585eeea02e2c79d7b1d8c4963bae2d

Reply via email to