Re: Review Request 44080: Windows:[1/2] Implemented `os::gmtime_r`.

2016-04-05 Thread Michael Park
> On March 18, 2016, 5:36 p.m., Michael Park wrote: > > 3rdparty/libprocess/3rdparty/stout/include/stout/windows/os.hpp, line 177 > > > > > > Can we remove the `struct` in `struct tm`? Submitted with this fix. - M

Re: Review Request 44080: Windows:[1/2] Implemented `os::gmtime_r`.

2016-03-31 Thread Alex Clemmer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44080/ --- (Updated March 31, 2016, 4:40 p.m.) Review request for mesos, Alex Naparu, Dani

Re: Review Request 44080: Windows:[1/2] Implemented `os::gmtime_r`.

2016-03-19 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44080/#review124228 --- 3rdparty/libprocess/3rdparty/stout/include/stout/posix/os.hpp (li

Re: Review Request 44080: Windows:[1/2] Implemented `os::gmtime_r`.

2016-03-14 Thread Yi Sun
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44080/#review123537 --- Ship it! Ship It! - Yi Sun On March 14, 2016, 8:48 p.m., Ale