-
Notifications
You must be signed in to change notification settings - Fork 284
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
Add some const qualifiers #1296
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, module one nit.
toxcore/tox.c
Outdated
@@ -330,7 +330,7 @@ bool tox_version_is_compatible(uint32_t major, uint32_t minor, uint32_t patch) | |||
|
|||
static State_Load_Status state_load_callback(void *outer, const uint8_t *data, uint32_t length, uint16_t type) | |||
{ | |||
Tox *tox = (Tox *)outer; | |||
const Tox *tox = (Tox *)outer; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can also change the (Tox *)
to (const Tox *)
for consistency.
Codecov Report
@@ Coverage Diff @@
## master #1296 +/- ##
========================================
- Coverage 83.2% 83% -0.3%
========================================
Files 82 82
Lines 14978 14962 -16
========================================
- Hits 12469 12425 -44
- Misses 2509 2537 +28
Continue to review full report at Codecov.
|
(Implementing suggestions of sudden6)
(Implementing suggestions of sudden6)
This change is