MSVC compilation fixes

David Bateman David.Bateman at motorola.com
Thu Jul 10 06:56:57 CDT 2008


Michael Goffioul wrote:
> On Wed, Jul 9, 2008 at 5:10 PM, John W. Eaton <jwe at bevo.che.wisc.edu> wrote:
>   
>> diff -r 7683fa6c06a2 src/OPERATORS/op-int.h
>> --- a/src/OPERATORS/op-int.h    Mon Feb 11 10:11:56 2008 +0100
>> +++ b/src/OPERATORS/op-int.h    Fri Jun 20 12:40:27 2008 +0200
>> @@ -227,13 +227,13 @@
>>   octave_value \
>>   xpow (const octave_ ## T1& a, float b) \
>>   { \
>> -    return pow (a, b); \
>> +    return pow (a, static_cast<double> (b)); \
>>   } \
>>
>> Shouldn't we use powf here instead?  Does MSVC provide powf, or do we
>> need a replacement in lo-specfun.cc (or lo-mappers.cc; I'm not sure
>> now why these files aren't combined)?
>>     
>
> Maybe David should comment on it. MSVC has powf
> (but it simply defined to (float)pow(a,b) on non-64bits architectures)).
>
> Michael.
>
>   
I'd say use powf.. If I didn't it was just an oversight..

D.

-- 
David Bateman                                David.Bateman at motorola.com
Motorola Labs - Paris                        +33 1 69 35 48 04 (Ph) 
Parc Les Algorithmes, Commune de St Aubin    +33 6 72 01 06 33 (Mob) 
91193 Gif-Sur-Yvette FRANCE                  +33 1 69 35 77 01 (Fax) 

The information contained in this communication has been classified as: 

[x] General Business Information 
[ ] Motorola Internal Use Only 
[ ] Motorola Confidential Proprietary



More information about the Octave-maintainers mailing list