Skip to content

d.linegraph: Follow-up fixes for typos, dereference pointers, and format string precision specifier - #7722

Merged
echoix merged 4 commits into
mainfrom
ai-findings-autofix/display-d.linegraph-main.c
Jul 15, 2026
Merged

echoix merged 4 commits into
mainfrom
ai-findings-autofix/display-d.linegraph-main.c

Conversation

@echoix

@echoix echoix commented Jul 15, 2026

Copy link
Copy Markdown
Member

A couple simple string typos were fixed, plus two other issues.

tt and tb are pointer parameters (double *tt, double *tb), so tt - tb computes pointer difference (an address offset), not the difference between the pointed-to text box values. The condition should dereference the pointers: (*tt - *tb) > YTIC_DIST.

The format string "%f.0" is incorrect — the .0 precision specifier must be inside the format specifier, not after it. The literal .0 will be appended as plain text to the output. The intended format is "%.0f" to format the value as a float with no decimal places.

echoix and others added 4 commits July 15, 2026 06:56
Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com>
@github-actions github-actions Bot added C Related code is in C module display labels Jul 15, 2026
@echoix
echoix merged commit 4683662 into main Jul 15, 2026
34 checks passed
@echoix
echoix deleted the ai-findings-autofix/display-d.linegraph-main.c branch July 15, 2026 08:14
@github-actions github-actions Bot added this to the 8.6.0 milestone Jul 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C Related code is in C display module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants