Skip to content

LibGUI: Include limits.h in Window - #27018

Open
ADKaster wants to merge 1 commit into
SerenityOS:masterfrom
ADKaster:random-loom-fixes
Open

LibGUI: Include limits.h in Window#27018
ADKaster wants to merge 1 commit into
SerenityOS:masterfrom
ADKaster:random-loom-fixes

Conversation

@ADKaster

@ADKaster ADKaster commented Sep 8, 2026

Copy link
Copy Markdown
Member
  1. Add an extra include to LibGUI/Window.cpp, since we were using INT_MAX without the proper header. This broke on Fedora 44 with gcc-16.

2. Reformat the default cursor png with ImageMagick to fix a premultiplied alpha problem. See commit for details.

Edit: png alphas handled in Loom code.

We use INT_MAX in a few places, but didn't explicitly load limits.h,
expecting that other system headers would pull it in.
@github-actions github-actions Bot added the 馃憖 pr-needs-review PR needs review from a maintainer or community member label Sep 8, 2026
@ADKaster ADKaster changed the title LibGUI+Base: A few fixes for my Loom project LibGUI: Include limits.h in Window Sep 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

馃憖 pr-needs-review PR needs review from a maintainer or community member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant