Skip to content

[asimage] fix compiler warnings#22454

Merged
linev merged 2 commits into
root-project:masterfrom
linev:fix_asimage
Jun 2, 2026
Merged

[asimage] fix compiler warnings#22454
linev merged 2 commits into
root-project:masterfrom
linev:fix_asimage

Conversation

@linev
Copy link
Copy Markdown
Member

@linev linev commented Jun 2, 2026

Eliminate compiler warnings.

linev added 2 commits June 2, 2026 13:37
Fix gcc warning:
variable ‘upscaled_gray’ might be clobbered by ‘longjmp’ or ‘vfork’
xcf.c:80:37: warning: writing 4 bytes into a region of size 0 [-Wstringop-overflow=]

gcc complains that pointer of struct member used in xcf_read32
to read three values. To be on the safe side - just read these values in
temporary array and then copy to correspondent members
@linev linev requested a review from couet June 2, 2026 13:02
@linev linev self-assigned this Jun 2, 2026
@linev linev requested a review from dpiparo as a code owner June 2, 2026 13:02
Copy link
Copy Markdown
Contributor

@guitargeek guitargeek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 2, 2026

Test Results

    22 files      22 suites   3d 14h 5m 54s ⏱️
 3 861 tests  3 861 ✅ 0 💤 0 ❌
77 103 runs  77 103 ✅ 0 💤 0 ❌

Results for commit f24e6c3.

@linev linev merged commit 13c3fdc into root-project:master Jun 2, 2026
40 of 41 checks passed
@linev linev deleted the fix_asimage branch June 2, 2026 19:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants