48bpp PNGs not supported in cover art

[justindhill]

Guess we should look into an image solution other than GDI+.

[HTTPSERVER] POST request: /api/cover/420?v=1&c=iSub&u=test&p=test&size=120

** (process:38658): WARNING **: PNG images with 48bpp aren't supported by libgdiplus.
[HTTPSERVER(1)] System.ArgumentException: A null reference or invalid value was found [GDI+ status: InvalidParameter]
  at System.Drawing.GDIPlus.CheckStatus (Status status) [0x0009d] in /private/tmp/monobuild/build/BUILD/mono-2.10.9/mcs/class/System.Drawing/System.Drawing/gdipFunctions.cs:220 
  at System.Drawing.Image.get_Width () [0x0000e] in /private/tmp/monobuild/build/BUILD/mono-2.10.9/mcs/class/System.Drawing/System.Drawing/Image.cs:770 
  at (wrapper remoting-invoke-with-check) System.Drawing.Image:get_Width ()
  at WaveBox.ApiHandler.Handlers.CoverArtApiHandler.ResizeImage (System.Drawing.Image imgToResize, Size size) [0x00000] in /Users/justin/Development/WaveBox/WaveBox/src/ApiHandler/Handlers/CoverArtApiHandler.cs:153 
  at WaveBox.ApiHandler.Handlers.CoverArtApiHandler.Process () [0x001e1] in /Users/justin/Development/WaveBox/WaveBox/src/ApiHandler/Handlers/CoverArtApiHandler.cs:96 
  at WaveBox.Http.HttpProcessor.HandlePOSTRequest () [0x000fe] in /Users/justin/Development/WaveBox/WaveBox/src/HttpServer/HttpProcessor.cs:218 
  at WaveBox.Http.HttpProcessor.process () [0x00072] in /Users/justin/Development/WaveBox/WaveBox/src/HttpServer/HttpProcessor.cs:77 

[justindhill]

A potential solution: http://stackoverflow.com/questions/4674453/cannot-load-png-in-c-sharp-on-mac-osx-running-mono

[einsteinx2]

Is this still an issue now that we're using ImageMagick?

For consistency, we'll get ImageMagick DLLs in for Windows as well and will be using mkbundle to bundle the Mono runtime even on Windows, so it's not a concern whether the Windows GDI works or not.

[justindhill]

It probably isn't, but I can't verify. Most likely, people won't have 48bpp images.

Justin Hill

On Wednesday, May 8, 2013 at 5:00 PM, Ben Baron wrote:

Is this still an issue now that we're using ImageMagick?
For consistency, we'll get ImageMagick DLLs in for Windows as well and will be using mkbundle to bundle the Mono runtime even on Windows, so it's not a concern whether the Windows GDI works or not.

—
Reply to this email directly or view it on GitHub (#18 (comment)).