Skip to content

Conversation

@aaronfranke
Copy link
Member

static _FORCE_INLINE_ bool is_ascii_alphabet_char(char32_t c) {
	return (c >= 'a' && c <= 'z') || (c >= 'A' && c <= 'Z');
}

Reasoning: This is the non-numeric version of is_ascii_alphanumeric_char. is_ascii_char can be misleading because it could imply all ASCII characters, including numbers, symbols, and control characters. Since this method explicitly only checks for alphabetical characters, it should be called is_ascii_alphabet_char (or similar).

@akien-mga akien-mga merged commit 659cf33 into godotengine:master Apr 22, 2024
@akien-mga
Copy link
Member

Thanks!

@aaronfranke aaronfranke deleted the is-ascii-alphabet-char branch April 22, 2024 13:00
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.

4 participants