Re: TortoiseIDiff suggestions

Stefan,

On 26/09/06, Stefan Küng <tortoisesvn@gmail.com> wrote:

> <snip>

Suggestion for the interface of such an image handler:
> IImageHandler
> // resets the enumeration of image types which can be handled
> bool IImageHandler::StartEnumImageTypes()
> // returns an image type in 'extension'. If false is returned, the
> // enumeration is finished
> bool IImageHandler::EnumImageTypes(TCHAR * extension)
>
> // returns true if the image can be handled
> bool IImageHandler::CanHandleImageType(const TCHAR * extension)
>
> // for the next interface functions, please refer to the CPicture class
> // we already have
> bool IImageHandler::LoadImage(const TCHAR * path)
> bool IImageHandler::Show(HDC hDC, RECT rect)
> bool SetInterpolationMode(InterpolationMode ip)
> float IImageHandler::GetHorizontalResolution()
> float IImageHandler::GetVerticalResolution()
> UINT IImageHandler::GetHeigth()
> UINT IImageHandler::GetWidth()
> UINT IImageHandler::GetColorDepth()
> UINT IImageHandler::GetNumberOfFrames()
> long IImageHandler::SetActiveFrame()

I would suggest one (or a few) generic get/setProperty( propname, propvalue)
methods. This will keep the number of methods low (and facilitates
get/setting specialized properties without having to update the CPicture
class.

-- 
Regards,
Jean-Marc
----------------
    ___
   // \\  @@     "De Chelonian Mobile"
  / \_/ \/._)    TortoiseSVN
<\_/_\_/ /      The coolest Interface to (Sub)Version Control
  /_/   \_\      Check out http://tortoisesvn.org