Skip to content

Comments

[Trivial] Replace usage of enforceEx with enforce#2462

Merged
quickfur merged 1 commit intodlang:masterfrom
Geod24:kill-enforceex
Aug 25, 2014
Merged

[Trivial] Replace usage of enforceEx with enforce#2462
quickfur merged 1 commit intodlang:masterfrom
Geod24:kill-enforceex

Conversation

@Geod24
Copy link
Member

@Geod24 Geod24 commented Aug 25, 2014

The only place where it can't be replaced is this alias in std.format:

private alias enforceFmt = enforceEx!FormatException;

@quickfur
Copy link
Member

LGTM

@quickfur
Copy link
Member

Auto-merge toggled on

quickfur pushed a commit that referenced this pull request Aug 25, 2014
[Trivial] Replace usage of enforceEx with enforce
@quickfur quickfur merged commit 3a29f6d into dlang:master Aug 25, 2014
@Geod24 Geod24 deleted the kill-enforceex branch August 25, 2014 20:36
@yebblies
Copy link
Contributor

The only place where it can't be replaced is this alias in std.format:

Should it be fixed to work there?

@Geod24
Copy link
Member Author

Geod24 commented Aug 26, 2014

Should it be fixed to work there?

Since #2288 , AFAIK, there is no reason to use enforceEx over enforce, except for this one.
It looks wrong to have one symbol dedicated to this rare use case, so I'd like to fix it, but I couldn't see any clean solution, besides fixing 9365.

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.

3 participants