Join GitHub today
GitHub is home to over 20 million developers working together to host and review code, manage projects, and build software together.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
Already on GitHub? Sign in to your account
Fix a couple of logcontext leaks #2049
+5
−5
Conversation
Some checks were not successful
Sytest Postgres (Commit)
— Build #2596 origin/rav/logcontext_leaks succeeded in 8 min 26 sec
richvdh
assigned
erikjohnston
Mar 23, 2017
|
lgtm |
richvdh
merged commit b2d40d7
into
develop
Mar 23, 2017
7 of 8 checks passed
Sytest Dendron (Commit)
Build #1774 origin/rav/logcontext_leaks failed in 12 min
Details
Sytest Dendron (Merged PR)
Build finished.
Details
Sytest Postgres (Commit)
Build #2596 origin/rav/logcontext_leaks succeeded in 8 min 26 sec
Details
Sytest Postgres (Merged PR)
Build finished.
Details
Sytest SQLite (Commit)
Build #2669 origin/rav/logcontext_leaks succeeded in 5 min 50 sec
Details
Sytest SQLite (Merged PR)
Build finished.
Details
continuous-integration/travis-ci/pr
The Travis CI build passed
Details
continuous-integration/travis-ci/push
The Travis CI build passed
Details
richvdh
deleted the
rav/logcontext_leaks
branch
Mar 23, 2017
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
richvdh commentedMar 23, 2017
Use preserve_fn to correctly manage the logcontexts around things we don't want
to yield on.