An ancient wart remaining in libfpx' fpx/fpxlib.cpp

[mi]

The ReadPage() and ReadPageLine() methods must've been returning int at some point so the calling method tried to analyze the response to turn it into a probable FPXStatus (an enum).

The methods now (and since, at least, 1.2) are already returning FPXStatus themselves, so the analysis is not necessary. Worse, because the checks were comparing FPXStatus with negative values, clang++ complains, because those (-2, -3) aren't valid in the enum.

The patch (a bit too large to just copy-paste here) is now part of the FreeBSD port graphics/libfpx.

[magick]

Thanks for the problem report and patch. We'll update the FPX distribution with the patch by sometime tomorrow.