Skip to content

Comments

[trivial] Fix imports in std.numeric#4636

Merged
9il merged 2 commits intodlang:masterfrom
JackStouffer:numeric
Jul 21, 2016
Merged

[trivial] Fix imports in std.numeric#4636
9il merged 2 commits intodlang:masterfrom
JackStouffer:numeric

Conversation

@JackStouffer
Copy link
Contributor

No description provided.

@9il 9il added the math label Jul 21, 2016
@9il
Copy link
Member

9il commented Jul 21, 2016

Auto-merge toggled on

@9il
Copy link
Member

9il commented Jul 21, 2016

Thanks!

@9il 9il merged commit 600c32c into dlang:master Jul 21, 2016
@PetarKirov
Copy link
Member

Why is enforce used, instead of assert? Most of the use cases seem like logic errors.

@9il
Copy link
Member

9il commented Jul 26, 2016

Why is enforce used, instead of assert? Most of the use cases seem like logic errors.

Yes, and they can be easily checked by users. Asserst looks better for me too for std.numeric. @ZombineDev will you open the PR?

@PetarKirov
Copy link
Member

Yes, I will. I was just checking if my understanding is correct.

@PetarKirov
Copy link
Member

@9il see #4688

@JackStouffer JackStouffer deleted the numeric branch May 22, 2017 17:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants