You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@arrow.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2017/09/29 21:39:00 UTC
[jira] [Commented] (ARROW-1624) [C++] Follow up fixes / tweaks to
compiler warnings for Plasma / LLVM 4.0, add to readme
[ https://issues.apache.org/jira/browse/ARROW-1624?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16186465#comment-16186465 ]
ASF GitHub Bot commented on ARROW-1624:
---------------------------------------
GitHub user wesm opened a pull request:
https://github.com/apache/arrow/pull/1148
WIP ARROW-1624: [C++] Fix build on LLVM 4.0, remove some clang warning suppressions
I'm going to quick make a pass through later today and see how many of these warning suppressions I can remove
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/wesm/arrow warning-fixes
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/arrow/pull/1148.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #1148
----
commit b4fe711f796053c6c9c9c0d3c6e1996e3711199c
Author: Wes McKinney <we...@twosigma.com>
Date: 2017-09-29T20:45:33Z
Fix compiler warnings with clang-4.0
Change-Id: I2569fe4b574c7e970aeb2b81b7e2c96ca3c6cfd4
commit d07be6fac16ed9df899949d5247b75552a767ae6
Author: Wes McKinney <we...@twosigma.com>
Date: 2017-09-29T21:35:47Z
Remove some clang warning suppressions, fix warnings
Change-Id: Id1a9ec85ea14be74d8451af5975209aa80029056
----
> [C++] Follow up fixes / tweaks to compiler warnings for Plasma / LLVM 4.0, add to readme
> ----------------------------------------------------------------------------------------
>
> Key: ARROW-1624
> URL: https://issues.apache.org/jira/browse/ARROW-1624
> Project: Apache Arrow
> Issue Type: Bug
> Components: C++
> Reporter: Wes McKinney
> Assignee: Wes McKinney
> Labels: pull-request-available
> Fix For: 0.8.0
>
>
> Not sure why these failures weren't caught in CI
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)