use flif namespace
create a flif namespace to avoid name conflicts. For instance Image is a widely used class name.
This should ease the integration of the decoder in another project
Well, all exported functions and struct of the library are prefixed with flif, so that's done already. And for the internal classes I don't see any conflicts.
I suppose if you want to integrate the decoder by copying the code (instead of linking to the library), then having a namespace would help. Not sure why you would want to integrate the decoder that way though...
The OP gave an example of a class / struct that is not namespaced nor prefixed: Image.
(I haven't checked whether that's true)
Well the story is simple: I experimented with the decoder to see if it can be used in our program (we are currently using huge png files). Once all my images were converted to flif I loaded them in our program using the flif_decode function. Our program also uses its own Image class (which fortunately is protected by a namespace) so it was kind of messy. This issue is mostly for convenience and cleanness of the API.
I think the inclusion of the decoder in another project should be as seamless as possible to not scare away potential users. (this also apply to #318 )
I wholeheartedly agree with k-brac.
I also, some prefix, like "FLIF_*" should be added to the #defines
And last, but not least, you really should look into the bunch of warnings the code generates.