Decoder works great when used in isolation. Does not play with other decoders (such as the image/png/jpg built-ins)? about tga HOT 6 OPEN

The problems lays in the TGA format itself.
For side-effect-only decoders to work, image.RegisterFormat must be called in the decoder's init(). It accepts the header argument, which allows image package choose the right decoder based on the header.
Unfortunately TGA doesn't have one :)
So the only solution I could come up with is importing as _ "github.com/ftrvxmtrx/tga" BEFORE any other image decoder, so it tries using TGA decoder only after all other decoders.
Not sure if it's possible to return some kind of "not so faulty" error from image decoder, so it actually tries other decoders as well.

from tga.

FWIW, there is a fork that removes the automatic registration of TGA, replacing with a tga.RegisterFormat() function.

https://github.com/dblezek/tga

from tga.

Cheers, good to know! Will give the "import-first" approach a go... in due course ;D

from tga.

FWIW, decoder tests do not pass for me with Go 1.5 on OSX. I suppose the assumptions about the package registration order may have been broken during the compiler refactoring. Registering it as a default fallback sounds brittle to me. If TGA format does not lend itself to sniffing, then you should probably not try to do it.

What about integrating @dblezek changes or something similar?

from tga.

That makes sense to me.

from tga.

@blezek thanks for that fork, solved my problem

from tga.