-
Notifications
You must be signed in to change notification settings - Fork 278
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
various manual cleanups #2278
various manual cleanups #2278
Conversation
Codecov Report
@@ Coverage Diff @@
## main #2278 +/- ##
==========================================
+ Coverage 63.52% 63.53% +0.01%
==========================================
Files 118 118
Lines 19600 19599 -1
Branches 9560 9562 +2
==========================================
+ Hits 12450 12453 +3
+ Misses 5082 5079 -3
+ Partials 2068 2067 -1
Continue to review full report at Codecov.
|
07bdb73
to
8d9553f
Compare
Updated. Additionally removed some lambdas. |
Signed-off-by: Rosen Penev <[email protected]>
Mostly deleted member functions Signed-off-by: Rosen Penev <[email protected]>
Signed-off-by: Rosen Penev <[email protected]>
Removed const ref change. I disagree but w/e. |
Signed-off-by: Rosen Penev <[email protected]>
90c3e90
to
a8d6012
Compare
Signed-off-by: Rosen Penev <[email protected]>
Small cleanup Signed-off-by: Rosen Penev <[email protected]>
Signed-off-by: Rosen Penev <[email protected]>
Signed-off-by: Rosen Penev <[email protected]>
I'd be happy to approve the change if you add a test. Right now that code is untested, which makes me nervous about changing it. |
No description provided.