search

staticlibs / ccronexpr

Cron expression parsing in ANSI C
Apache License 2.0
150 stars 78 forks source link

Remove warning #7

Closed Tangerino closed 6 years ago

Tangerino commented 6 years ago

Can we just add a simple #ifdef in order to remove a warning such as:

#ifndef _WIN32
struct tm *gmtime_r(const time_t *timep, struct tm *result);
#ifdef CRON_USE_LOCAL_TIME    <-------- THIS ONE
struct tm *localtime_r(const time_t *timep, struct tm *result);
#endif
#endif

I'm compiling it using SystemWorkbench for STM32 Thank you

staticlibs commented 6 years ago

Hi, to declare localtime_r only if we are actually going to use it - looks like a good idea. Will accept a PR, or can add it myself in a day or two.

Tangerino commented 6 years ago

Thanks, no hurry.

On Sun, Feb 25, 2018 at 12:33 PM, staticlibs notifications@github.com wrote:

Hi, to declare localtime_r only if we are actually going to use it - looks like a good idea. Will accept a PR, or can add it myself in a day or two.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/staticlibs/ccronexpr/issues/7#issuecomment-368302042, or mute the thread https://github.com/notifications/unsubscribe-auth/ADBHxBLGc_HpHRHo-fWRwO1ft7CFb4u4ks5tYUUhgaJpZM4SRxpV .

--

Merci beaucoup, Muito obrigado, Thanks

Carlos TangerinoMobile: +33 6 82 05 55 18