Skip to content

Update sdlgamecontroller.inc to 2.0.22 - #53

Merged
Free-Pascal-meets-SDL-Website merged 18 commits into
PascalGameDevelopment:masterfrom
suve:update-sdlgamecontroller-to-2.0.22
Jul 31, 2022
Merged

Free-Pascal-meets-SDL-Website merged 18 commits into
PascalGameDevelopment:masterfrom
suve:update-sdlgamecontroller-to-2.0.22

Conversation

@suve

@suve suve commented May 22, 2022

Copy link
Copy Markdown
Collaborator

This patch updates sdlgamecontroller.inc to match SDL_gamecontroller.h as of SDL 2.0.22.

@Free-Pascal-meets-SDL-Website

Copy link
Copy Markdown
Collaborator

Impressive update! Thank you!

Just a minor thing: Could you lower-case the U in line 456+457 and 482+483 for consistency, hence cUint32 gets cuint32?

@suve

suve commented May 25, 2022

Copy link
Copy Markdown
Collaborator Author

I'm not a fan of all-lowercase identifiers, but I guess that's something we can add to the "style guide" discussion. I'll add a commit to change those.

@Free-Pascal-meets-SDL-Website

Free-Pascal-meets-SDL-Website commented Jul 11, 2022 •

Copy link
Copy Markdown
Collaborator

I consider merging this PR soon if you don't mind, as it is too important to be delayed too long and the minor issue is of cosmetic nature only. This can still be fixed in the future.

@Free-Pascal-meets-SDL-Website
Free-Pascal-meets-SDL-Website merged commit 4df9dcf into PascalGameDevelopment:master Jul 31, 2022
@Free-Pascal-meets-SDL-Website

Copy link
Copy Markdown
Collaborator

Merged - thanks again, great PR!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants