Opened 3 years ago

Closed 3 years ago

#4589 closed Defect (Fixed)

Fix signed/unsigned printf formatting

Reported by: snu Owned by:
Component: Chat Core (General) Version: 2.4 (Mac)
Severity: Major Keywords:
Cc:

Description

Commit r5538 changed some printf formatting but ignored signedness. Most of the %lu (long unsigned) formatting was replaced by %d (signed integer). This causes at least issue #3547.

Please check for all changes in signedness and use correct printf format (%d or %u).

Change History (2)

comment:1 Changed 3 years ago by zach

Good catch. I'm curious why Xcode wasn't warning about using %d when given a long. Thanks for digging into this, I'll adding that commit more closely.

comment:2 Changed 3 years ago by zach

  • Resolution set to Fixed
  • Status changed from new to closed

Done in r6477, thanks again.

Note: See TracTickets for help on using tickets.