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.

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

fix: inversed return values #2414

Merged
merged 1 commit into from
Nov 7, 2023
Merged

Conversation

JFreegman
Copy link
Member

@JFreegman JFreegman commented Oct 27, 2023

This was causing pings to be sent every 1-2 seconds rather than the expected interval.


This change is Reviewable

@JFreegman JFreegman added the bug Bug fix for the user, not a fix to a build script label Oct 27, 2023
@JFreegman JFreegman added this to the v0.2.19 milestone Oct 27, 2023
@codecov
Copy link

codecov bot commented Oct 27, 2023

Codecov Report

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

Comparison is base (24b5472) 74.43% compared to head (d0304d8) 74.47%.
Report is 1 commits behind head on master.

❗ Current head d0304d8 differs from pull request most recent head 1195271. Consider uploading reports for the commit 1195271 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2414      +/-   ##
==========================================
+ Coverage   74.43%   74.47%   +0.04%     
==========================================
  Files          87       87              
  Lines       26192    26189       -3     
==========================================
+ Hits        19495    19505      +10     
+ Misses       6697     6684      -13     
Files Coverage Δ
toxcore/group_chats.c 69.69% <50.00%> (+0.13%) ⬆️

... and 8 files with indirect coverage changes

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

@JFreegman JFreegman force-pushed the group_ping_interval branch 2 times, most recently from d38cae2 to 5b72599 Compare November 7, 2023 03:55
This was causing pings to be sent every 1-2 seconds rather than
the expected interval.
@JFreegman JFreegman merged commit 1195271 into TokTok:master Nov 7, 2023
50 checks passed
@JFreegman JFreegman deleted the group_ping_interval branch November 7, 2023 19:15
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bug fix for the user, not a fix to a build script
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants