chansen / p5-time-moment

Time::Moment represents an exact moment in time.
32 stars 8 forks source link

0.39 does not compile with threaded perls #22

Closed eserte closed 7 years ago

eserte commented 7 years ago

See subject. Here a sample build log on a OS X system:

Moment.xs:944:51: error: too many arguments to function call, expected 3, have 4
        r = moment_compare_precision(self, other, precision);
            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~
src/moment.h:244:50: note: expanded from macro 'moment_compare_precision'
    THX_moment_compare_precision(aTHX_ mt1, mt2, precision)
    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~                 ^~~~~~~~~
src/moment.h:132:1: note: 'THX_moment_compare_precision' declared here
int         THX_moment_compare_precision(const moment_t *mt1, const mome...
^
chansen commented 7 years ago

Thank you for the report! I have shipped 0.40 to CPAN which addresses this issue.