Skip to content

Add isCharacter to phobos.sys.traits. - #11089

Merged
LightBender merged 1 commit into
dlang:masterfrom
jmdavis:pv3_traits_ischar
Sep 3, 2026
Merged

Add isCharacter to phobos.sys.traits.#11089
LightBender merged 1 commit into
dlang:masterfrom
jmdavis:pv3_traits_ischar

Conversation

@jmdavis

@jmdavis jmdavis commented Sep 3, 2026

Copy link
Copy Markdown
Member

isCharacter is the Phobos v3 equivalent to std.traits' isSomeChar. However, unlike isSomeChar, isCharacter does not accept enum types with a base type which is a character type (since that's one of the mistakes in Phobos v2 that we're trying to fix). The type must be either char, wchar, or dchar (ignoring qualifiers).

isCharacter is the Phobos v3 equivalent to std.traits' isSomeChar.
However, unlike isSomeChar, isCharacter does not accept enum types with
a base type which is a character type (since that's one of the mistakes
in Phobos v2 that we're trying to fix). The type must be either char,
wchar, or dchar (ignoring qualifiers).
@jmdavis jmdavis added the Phobos 3 The PR/issue is for Phobos V3. label Sep 3, 2026
@jmdavis

jmdavis commented Sep 3, 2026

Copy link
Copy Markdown
Member Author

Lovely, we have an unrelated buildkite failure.

@rikkimax

rikkimax commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Everyone is getting it.

@LightBender

Copy link
Copy Markdown
Contributor

@jmdavis We know what it is, Nic or I can (and will) override.

@jmdavis

jmdavis commented Sep 3, 2026

Copy link
Copy Markdown
Member Author

Everyone is getting it.

Since it's pretty clearly unrelated, I figured. Any idea what's causing the failures?

EDIT: And Adam replied while I was posting my reply... LOL.

@LightBender
LightBender merged commit 982f603 into dlang:master Sep 3, 2026
9 of 10 checks passed
@jmdavis
jmdavis deleted the pv3_traits_ischar branch September 3, 2026 14:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Phobos 3 The PR/issue is for Phobos V3.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants