You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@mesos.apache.org by Akash Gupta <ak...@hotmail.com> on 2018/03/01 09:45:51 UTC
Re: Review Request 65839: Windows: Removed stout W* signal macros in
`windows.hpp`.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/65839/
-----------------------------------------------------------
(Updated March 1, 2018, 9:45 a.m.)
Review request for mesos, Andrew Schwartzmeyer and Joseph Wu.
Changes
-------
split commit into three
Summary (updated)
-----------------
Windows: Removed stout W* signal macros in `windows.hpp`.
Repository: mesos
Description
-------
`windows.hpp` defined a few macros for the status value of `waitpid`
that were either unused in the code base or were only used in gtest.
These macros handled signal logic, so they were removed since they
aren't used on Windows.
Diffs (updated)
-----
3rdparty/stout/include/stout/gtest.hpp ce6e4de6a9f69ea121ff0ca6048316d678c8c857
3rdparty/stout/include/stout/windows.hpp b35e6b94ba6709254450be9429b6f48f2d276689
Diff: https://reviews.apache.org/r/65839/diff/2/
Changes: https://reviews.apache.org/r/65839/diff/1-2/
Testing
-------
Thanks,
Akash Gupta
Re: Review Request 65839: Windows: Removed stout W* signal macros in
`windows.hpp`.
Posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/65839/#review198566
-----------------------------------------------------------
Ship it!
Ship It!
- Andrew Schwartzmeyer
On March 1, 2018, 1:45 a.m., Akash Gupta wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/65839/
> -----------------------------------------------------------
>
> (Updated March 1, 2018, 1:45 a.m.)
>
>
> Review request for mesos, Andrew Schwartzmeyer and Joseph Wu.
>
>
> Repository: mesos
>
>
> Description
> -------
>
> `windows.hpp` defined a few macros for the status value of `waitpid`
> that were either unused in the code base or were only used in gtest.
> These macros handled signal logic, so they were removed since they
> aren't used on Windows.
>
>
> Diffs
> -----
>
> 3rdparty/stout/include/stout/gtest.hpp ce6e4de6a9f69ea121ff0ca6048316d678c8c857
> 3rdparty/stout/include/stout/windows.hpp b35e6b94ba6709254450be9429b6f48f2d276689
>
>
> Diff: https://reviews.apache.org/r/65839/diff/2/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Akash Gupta
>
>