You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@mesos.apache.org by "Joseph Wu (JIRA)" <ji...@apache.org> on 2016/12/15 23:31:58 UTC
[jira] [Commented] (MESOS-6706) Port `files_tests.cpp`
[ https://issues.apache.org/jira/browse/MESOS-6706?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15752840#comment-15752840 ]
Joseph Wu commented on MESOS-6706:
----------------------------------
{code}
commit c1578bb74929440c37bd9f393af5e55efee3c67d
Author: Alex Clemmer <cl...@gmail.com>
Date: Thu Dec 15 15:08:14 2016 -0800
Windows: Disambiguated `os::write` by adding another overload.
This commit will address test failures on Windows in `files_tests.cpp`.
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.
Review: https://reviews.apache.org/r/54631/
{code}
> Port `files_tests.cpp`
> ----------------------
>
> Key: MESOS-6706
> URL: https://issues.apache.org/jira/browse/MESOS-6706
> Project: Mesos
> Issue Type: Bug
> Components: agent
> Reporter: Alex Clemmer
> Assignee: Alex Clemmer
> Labels: microsoft, windows-mvp
>
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)