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.

Sign up for GitHub

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

Move internals of many C++ Async classes to ref-counted struct #5099

Merged
merged 10 commits into from
Feb 8, 2016
+46 −18
Merged

Move internals of many C++ Async classes to ref-counted struct #5099

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
38004a8
Wrap groups of related CallOpSet's into a ref-counted structure
vjpai Feb 5, 2016
7e3f9b0
Forgot to include these edits in the last commit
vjpai Feb 5, 2016
5014341
Merge branch 'master' into cpp_races
vjpai Feb 5, 2016
5506bea
Undo much of the previous commits so that only CallOpSet groups
vjpai Feb 5, 2016
2b56386
Drop the ref
vjpai Feb 5, 2016
bedf57f
Ref the collection only when it will be used (and later finalized)
vjpai Feb 6, 2016
4d9d4ab
Merge branch 'master' into cpp_races
vjpai Feb 8, 2016
c593ca0
reset the shared_ptr
vjpai Feb 8, 2016
30bf3ea
clang-format
vjpai Feb 8, 2016
4776292
Merge branch 'master' into cpp_races
vjpai Feb 8, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Merge branch 'master' into cpp_races
@vjpai
vjpai committed Feb 8, 2016
commit 4776292fc3053209485443a6a89b279bb2bc5bb5

This merge commit was added into this branch cleanly.

There are no new changes to show, but you can still view the diff.