----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3273/#review10974 -----------------------------------------------------------
/branches/1.8/main/config.c <https://reviewboard.asterisk.org/r/3273/#comment20605> Setting nsec isn't needed. It's calloc'ed to 0 anyway. And #warning's are not used anywhere else in the codebase (except for the obscure cel_tds backend), so I'm not sure we should start adding those. - wdoekes On Feb. 27, 2014, 4:37 p.m., David Lee wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/3273/ > ----------------------------------------------------------- > > (Updated Feb. 27, 2014, 4:37 p.m.) > > > Review request for Asterisk Developers. > > > Repository: Asterisk > > > Description > ------- > > When nanosecond time resolution was added for identifying config file > changes, it didn't cover all of the myriad of ways that one might obtain > nanosecond time resolution off of struct stat. > > Rather than complicate the #if even further figuring out one system from > the next, this patch directly tests for the three struct members I know > about today, and #ifdef's accordingly. > > > Diffs > ----- > > /branches/1.8/main/config.c 409079 > /branches/1.8/include/asterisk/autoconfig.h.in 409079 > /branches/1.8/configure.ac 409079 > /branches/1.8/configure UNKNOWN > > Diff: https://reviewboard.asterisk.org/r/3273/diff/ > > > Testing > ------- > > Compiled on Ubuntu 13.10 and OS X 10.9.2. > > > Thanks, > > David Lee > >
-- _____________________________________________________________________ -- Bandwidth and Colocation Provided by http://www.api-digital.com -- asterisk-dev mailing list To UNSUBSCRIBE or update options visit: http://lists.digium.com/mailman/listinfo/asterisk-dev
