You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
marx code with with fseeko/ftello if available and fseek/ftell if not. In jdfits however, preprocessor definitions that select one of the two are duplicated in both configure.ac and in _jdfits.h. At first glance, I believe that only one of those is needed. The same is probably true for jdmath and possible marx itself. To be clear, nothing is really broken, but it is nice to reduce the number of compiler warnings and make headers and install scripts easier to read.
The text was updated successfully, but these errors were encountered:
In fact, I can probably remove all of those, since the C standard has evolved. It might be better to replace all fseek/ftell with fgetpos/fsetpos anywany when switching to C99 and remove all the preprocessor stuff, since I don't think we need to support systems that don't supply fseek any more.
marx code with with
fseeko/ftello
if available andfseek/ftell
if not. In jdfits however, preprocessor definitions that select one of the two are duplicated in bothconfigure.ac
and in_jdfits.h
. At first glance, I believe that only one of those is needed. The same is probably true for jdmath and possible marx itself. To be clear, nothing is really broken, but it is nice to reduce the number of compiler warnings and make headers and install scripts easier to read.The text was updated successfully, but these errors were encountered: