[jira] [Comment Edited] (ARROW-6405) [Python] Add std::move wrapper for use in Cython

["Krisztian Szucs (Jira)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/ARROW-6405?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16942071#comment-16942071 ] 

Krisztian Szucs edited comment on ARROW-6405 at 10/1/19 3:28 PM:
-----------------------------------------------------------------

I've actually have a pending PR about the Dataset bindings, including cymove and its license note: https://github.com/apache/arrow/pull/5237/files#diff-e9651a80565f8c6b1085943b393a7f72R32

We can factor it out to a standalone PR.


was (Author: kszucs):
I've actually have a pending PR about the Dataset bindings, including cymove and its license note: https://github.com/apache/arrow/pull/5237/files#diff-e9651a80565f8c6b1085943b393a7f72R32

> [Python] Add std::move wrapper for use in Cython
> ------------------------------------------------
>
>                 Key: ARROW-6405
>                 URL: https://issues.apache.org/jira/browse/ARROW-6405
>             Project: Apache Arrow
>          Issue Type: Improvement
>          Components: Python
>            Reporter: Wes McKinney
>            Priority: Major
>             Fix For: 1.0.0
>
>
> [~bkietz] pointed out this to me
> https://github.com/ozars/cymove
> This is small enough that we should simply copy this code into our codebase (MIT-licensed) and fix the couple of places where we have {{std::move}}-related workarounds.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)