KImageEffect compile fixes/ bug fixes

Jarosław Staniek js at iidea.pl
Mon Sep 13 17:57:41 BST 2004


> Ian Reinhart Geiser said:
> 
>>Jaros^(3)aw Staniek said:
>>
>>>ok?
>>
>>no not okay.
>>
>>whats it do? tests are useful for this sort of thing.
>>
>>cheers
>>   -ian reinhart geiser
> 
> the zero out the struct is cool, but im not sure what the double cast
> accomplishes... this code is basicly untested, and the limited tests we
> have done on it, cause odd behavior when you do "correct" casts.
> 
> Do you have a test that this in fact fixes anything?

Yes, using win32, execute

nmake

in kdefx/ directory. Without the patch there is compilter error (sqrt() has 
many overloads). With the patch all is ok. Enough?

-- 
regards / pozdrawiam,
   Jaroslaw Staniek / OpenOffice Polska
   Kexi project: http://koffice.org/kexi/ http://www.kexi-project.org/
   Qt-KDE Wrapper project: http://iidea.pl/~js/qkw/




More information about the kde-core-devel mailing list