mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Alex Clemmer <>
Subject Review Request 54631: Windows: Disambiguated `os::write` by adding another overload.
Date Sat, 10 Dec 2016 22:51:28 GMT

This is an automatically generated e-mail. To reply, visit:

Review request for mesos, Andrew Schwartzmeyer, Daniel Pravat, and Joseph Wu.

Bugs: MESOS-6706

Repository: mesos


This commit will partially address MESOS-6706, specifically a bug
causing calls to `os::write` to fail on Windows for all tests in

On Windows, calling `os::write` with a `char *` literal (e.g.,
`os::write("your_file", ...)` will erroneously dispatch to the overload
of `os::write(HANDLE)` instead of `os::write(const string&)`, because
`HANDLE` is a typedef'd `void *`.

In this commit, we simply add an overload for `char *`, which resolves
the ambiguity and causes most tests in `files_tests.cpp` to pass.


  3rdparty/stout/include/stout/os/write.hpp 24a69d8f60efd3c2888d464d75164c758b3701a2 




Alex Clemmer

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message