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

Update Query copy/move strategy #171

Merged
merged 1 commit into from Sep 6, 2015

Conversation

Projects
None yet
1 participant
@bassosimone
Member

bassosimone commented Sep 6, 2015

  • forbid copy that will break our code
  • explain why
  • document need to use default move operations
  • document how a simpler implementation would look like

(not reimplementing now, with the pending release and lots of things in the TODO list, due to other priorities. Also this is simple enough to qualify as an hotfix.)

Update Query copy/move strategy
- forbid copy that will break our code

- explain why

- document need to use default move operations

- document how a simpler implementation would look like

(not reimplementing now, with the pending release and lots of things
in the TODO list, due to other priorities.)

@bassosimone bassosimone self-assigned this Sep 6, 2015

@bassosimone bassosimone added this to the measurement-kit 0.1.0 milestone Sep 6, 2015

bassosimone added a commit that referenced this pull request Sep 6, 2015

@bassosimone bassosimone merged commit bd8e1c9 into measurement-kit:master Sep 6, 2015

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@bassosimone bassosimone deleted the bassosimone:feature/dns-query-copy-move branch Sep 6, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment