Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix too many letters #1585

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

MarkOates
Copy link
Contributor

Allegro has inefficient naming conventions. Specifically, typing ALLEGRO_* for every constant is a waste of precious key tying time.

This PR fixes this by replacing all ALLEGRO_* constants with A50_*.

Since "ALLEGRO_" takes takes 9 keystrokes (including SHIFT) and "A5O_" takes 6 (including a second re-press of SHIFT), this is a productivity increase of a whopping 33.33% for ALL Allegro users.

@pedro-w
Copy link
Contributor

pedro-w commented Sep 29, 2024

This is great, and very insightful. Similarly, long words such as 'bitmap' or 'display' could be replaced by 'b' and 'd' respectively for greater efficiency improvement.

@Iniquitatis
Copy link

This is great, and very insightful. Similarly, long words such as 'bitmap' or 'display' could be replaced by 'b' and 'd' respectively for greater efficiency improvement.

Don't forget about s for semaphore and m for mutex! 🧐

@pmprog
Copy link
Contributor

pmprog commented Oct 17, 2024

Allegro has inefficient naming conventions. Specifically, typing ALLEGRO_* for every constant is a waste of precious key tying time.

This PR fixes this by replacing all ALLEGRO_* constants with A50_*.

Since "ALLEGRO_" takes takes 9 keystrokes (including SHIFT) and "A5O_" takes 6 (including a second re-press of SHIFT), this is a productivity increase of a whopping 33.33% for ALL Allegro users.

Seems a really odd question, but you start by mentioned A50, then you go to A5O, with a letter O rather than a zero.
Is there much need for either? Surely A5_ would be sufficient enough?
I'm in favour :)

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.

4 participants