Skip to content
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

[IMP] ODataRow: add copy constructor and methods #306

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

r-mckay
Copy link
Contributor

@r-mckay r-mckay commented Jan 19, 2017

This patch adds a copy constructor to the ODataRow class. It will
allow to easily duplicate an ODataRow object, thanks to the _data of
the first one.

A "remove" method is also added to expose the epomynous method.

Finally the ODataRow internal _data HashMap is set as a private member
of the class.

This patch adds a copy constructor to the ODataRow class. It will
allow to easily duplicate an ODataRow object, thanks to the _data of
the first one.

A "remove" method is also added to expose the epomynous method.

Finally the ODataRow internal _data HashMap is set as a private member
of the class.
@r-mckay
Copy link
Contributor Author

r-mckay commented Feb 6, 2017

Hello @dpr-odoo ,

may I have your opinion on this?

Thank you in advance.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant