Migrate WIC image I/O to use msw::ComPtr and add dynamic codec enumeration

- Replace raw COM pointers with msw::ComPtr for automatic lifetime management
- Use releaseAndGetAddressOf() for COM output parameters
- Remove overly restrictive static_assert from ComPtr that prevented compilation with forward-declared types
- Implement dynamic WIC decoder/encoder enumeration to automatically register supported extensions
- WIC now queries installed codecs at registration time instead of using hardcoded extension list

Author: andrewfb

include/cinder/ImageSourceFileWic.h

@@ -25,13 +25,16 @@
 #include "cinder/Cinder.h"
 #include "cinder/ImageIO.h"
 #include "cinder/Exception.h"
+#include "cinder/msw/CinderMsw.h"
 
 #include <guiddef.h>
 
 // WIC forward declarations
 struct IWICImagingFactory;
 struct IWICBitmapFrameDecode;
 struct IWICStream;
+struct IWICComponentEnumerator;
+struct IWICBitmapCodecInfo;
 
 namespace cinder {
 
@@ -52,12 +55,12 @@ class ImageSourceFileWic : public ImageSource {
 
 	bool	processFormat( const ::GUID &guid, ::GUID *convertGUID );
 
-	std::shared_ptr<IWICBitmapFrameDecode>	mFrame;
-	std::shared_ptr<IWICStream>				mStream;
-	ci::BufferRef							mBuffer;
-	bool									mRequiresConversion;
-	int32_t									mRowBytes;
-	::GUID									mPixelFormat, mConvertPixelFormat;
+	msw::ComPtr<IWICBitmapFrameDecode>	mFrame;
+	msw::ComPtr<IWICStream>				mStream;
+	ci::BufferRef						mBuffer;
+	bool								mRequiresConversion;
+	int32_t								mRowBytes;
+	::GUID								mPixelFormat, mConvertPixelFormat;
 };
 
 class ImageSourceFileWicExceptionUnsupportedData : public ImageIoException {

include/cinder/ImageTargetFileWic.h

@@ -24,12 +24,15 @@
 
 #include "cinder/Cinder.h"
 #include "cinder/ImageIo.h"
+#include "cinder/msw/CinderMsw.h"
 
 #ifndef GUID
 typedef struct _GUID GUID;
 #endif
 struct IWICBitmapEncoder;
 struct IWICBitmapFrameEncode;
+struct IWICComponentEnumerator;
+struct IWICBitmapCodecInfo;
 
 namespace cinder {
 
@@ -38,24 +41,24 @@ typedef std::shared_ptr<class ImageTargetFileWic> ImageTargetFileWicRef;
 class ImageTargetFileWic : public ImageTarget {
   public:
 	static ImageTargetRef		create( DataTargetRef dataTarget, ImageSourceRef imageSource, ImageTarget::Options options, const std::string &extensionData );
-	
+
 	void*	getRowPointer( int32_t row ) override;
 	void	finalize() override;
-	
+
 	static void		registerSelf();
-	
+
   protected:
 	ImageTargetFileWic( DataTargetRef dataTarget, ImageSourceRef imageSource, ImageTarget::Options options, const std::string &extensionData );
 
 	void		setupPixelFormat( const GUID &guid );
 
-	std::shared_ptr<uint8_t>	mData;
-	int32_t						mRowBytes;
-	DataTargetRef				mDataTarget;
-	const GUID					*mCodecGUID;
-	
-	std::shared_ptr<IWICBitmapEncoder>			mEncoder;
-	std::shared_ptr<IWICBitmapFrameEncode>		mBitmapFrame;
+	std::shared_ptr<uint8_t>			mData;
+	int32_t								mRowBytes;
+	DataTargetRef						mDataTarget;
+	const GUID							*mCodecGUID;
+
+	msw::ComPtr<IWICBitmapEncoder>		mEncoder;
+	msw::ComPtr<IWICBitmapFrameEncode>	mBitmapFrame;
 };
 
 } // namespace cinder

include/cinder/msw/CinderMsw.h

@@ -56,8 +56,14 @@ CI_API inline vec2 toVec2( const ::POINTFX& p )
 CI_API void ComDelete( void* p );
 
 //! Functor version that calls Release() on a com-managed object
-struct CI_API ComDeleter{ template<typename T> void operator()( T* p ){ if( p ) p->Release();
-}}; // namespace cinder::msw
+struct CI_API ComDeleter {
+	template<typename T>
+	void operator()( T* p )
+	{
+		if( p )
+			p->Release();
+	}
+};
 
 template<typename T>
 using ManagedComRef = std::shared_ptr<T>;
@@ -86,8 +92,6 @@ ManagedComPtr<T> makeComUnique( T* p )
 	- Move operations transfer ownership without ref-counting */
 template<typename T>
 class CI_API ComPtr {
-	static_assert( std::is_base_of<IUnknown, T>::value, "ComPtr<T>: T must derive from IUnknown" );
-
   public:
 	ComPtr() noexcept
 		: ptr( nullptr )