-
Notifications
You must be signed in to change notification settings - Fork 9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixed copy semantic of arrays #244
Conversation
list_arr3 = list_arr; | ||
CHECK_EQ(list_arr3, list_arr); | ||
} | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is no "move" test for this one ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not yet, because that would require implementing the move semantic in all the arrays, while I think that with my incoming "refactoring" of the arrow_proxy class, we can actually get it for free. I'll add them in the same PR.
test/test_list_array.cpp
Outdated
@@ -266,7 +331,7 @@ namespace sparrow | |||
} | |||
} | |||
} | |||
delete schema.format; | |||
//delete schema.format; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To delete
No description provided.