8000 refactor: Pass `this` pointer as first param to s11n callbacks. by iphydf · Pull Request #2575 · TokTok/c-toxcore · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

refactor: Pass this pointer as first param to s11n callbacks. #2575

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

Merged
merged 1 commit into from
Jan 17, 2024

Conversation

iphydf
Copy link
Member
@iphydf iphydf commented Jan 16, 2024

This change is Reviewable

@iphydf iphydf added this to the v0.2.19 milestone Jan 16, 2024
@iphydf iphydf marked this pull request as ready for review January 16, 2024 19:44
@iphydf iphydf force-pushed the this-ptr branch 3 times, most recently from 2691e3c to 1c16b4c Compare January 16, 2024 20:15
@nurupo
Copy link
Member
nurupo commented Jan 16, 2024

Don't you love it when you write review comments, but don't click on the button to "add" them, and then click to submit the review thinking they would get added, like it works in Reviewable, but GitHub just erases all your comments, having to re-write them again as none are saved. Ugh.

(The next worst thing, since in this case the comments are not lost at least, is adding comments by clicking on "Start a review", thinking than you actually initialized the review process and they are visible to anyone, with "pending" meaning that they haven't been resolved yet, but after several days of no one responding to them, you check the issue in a browser you are not logged in and you don't see any of your comments... yeah, fun).

Copy link
codecov bot commented Jan 16, 2024

Codecov Report

Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (e7fb91d) 66.75% compared to head (c650d9d) 66.84%.

Files Patch % Lines
toxcore/bin_pack.c 75.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2575      +/-   ##
==========================================
+ Coverage   66.75%   66.84%   +0.08%     
==========================================
  Files         148      148              
  Lines       30328    30332       +4     
==========================================
+ Hits        20246    20274      +28     
+ Misses      10082    10058      -24     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@pull-request-attention pull-request-attention bot assigned iphydf and unassigned nurupo Jan 16, 2024
@iphydf iphydf force-pushed the this-ptr branch 3 times, most recently from fd5f8cf to b175548 Compare January 16, 2024 23:36
@pull-request-attention pull-request-attention bot assigned nurupo and unassigned iphydf Jan 16, 2024
@iphydf iphydf force-pushed the this-ptr branch 2 times, most recently from fb910e8 to 7439214 Compare January 17, 2024 09:23
@pull-request-attention pull-request-attention bot assigned iphydf and unassigned nurupo Jan 17, 2024
@toktok-releaser toktok-releaser merged commit c650d9d into TokTok:master Jan 17, 2024
@iphydf iphydf deleted the this-ptr branch January 17, 2024 18:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants
0