Moving CImage interface members up to IImage

CuteAlien · Post by **CuteAlien** » Wed May 07, 2008 12:27 am

I just needed the scaleTo function of CImage when I noticed that it's not part of the IImage interface. Similar there are a lot more functions offered by CImage which can currently not be accessed from the outside. As those all look very useful I wonder if there's a reason not to put them in IImage?

rogerborg · Post by **rogerborg** » Wed May 07, 2008 7:38 am

Uh, the what function? I can't see CImage::scaleTo() in SVN 1344.

I do see some CImage methods that aren't in IImage, and I can't see any good reason why they can't be exposes there. CImage seems to be the only implementor of IImage.

hybrid · Post by **hybrid** » Wed May 07, 2008 8:06 am

Yes, there had been some patches and requests on the tracker already, should be done for 1.5. However, the interface might also require some general clean up and a merge with the ITexture interface in order to remove some inconsistencies...
BTW: the method is called copyToScaling()

CuteAlien · Post by **CuteAlien** » Wed May 07, 2008 1:08 pm

Oops - didn't want to write scaleTo. I actually meant the copyTo which copies without scaling :-)