Skip to content

/ nodatime

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

Jump to bottom

Instant bounds checking bypassed #979

Closed
jskeet opened this issue Oct 4, 2017 · 0 comments
Closed

Instant bounds checking bypassed #979

jskeet opened this issue Oct 4, 2017 · 0 comments
Assignees
@jskeet
Labels
bug
Milestone
2.2.1

Comments

@jskeet
Copy link
Member

@jskeet jskeet commented Oct 4, 2017

This should throw an exception, but doesn't:

Instant.MinValue.Minus(Duration.FromDays(1))
@jskeet jskeet self-assigned this Oct 4, 2017
@jskeet jskeet added the bug label Oct 4, 2017
@jskeet jskeet mentioned this issue Oct 4, 2017
Closed
jskeet added a commit to jskeet/nodatime that referenced this issue Oct 4, 2017
@jskeet
Fix Instant - Duration not performing validation
da3fbc6 
Fixes nodatime#979.
@jskeet jskeet mentioned this issue Oct 4, 2017
Merged
@jskeet jskeet closed this in #980 Oct 5, 2017
jskeet added a commit that referenced this issue Oct 5, 2017
@jskeet
Fix Instant - Duration not performing validation
Loading status checks…
c1fb0aa 
Fixes #979.
@jskeet jskeet mentioned this issue Oct 5, 2017
Closed
4 of 4 tasks complete
@malcolmr malcolmr added this to the 2.2.1 milestone Oct 9, 2017
jskeet added a commit to jskeet/nodatime that referenced this issue Oct 12, 2017
@jskeet
Fix Instant - Duration not performing validation
765ac2a 
Fixes nodatime#979.
jskeet added a commit to jskeet/nodatime that referenced this issue Oct 12, 2017
@jskeet
Fix Instant - Duration not performing validation
8fed682 
Fixes nodatime#979.
jskeet added a commit that referenced this issue Oct 14, 2017
@jskeet
Fix Instant - Duration not performing validation
ebf8002 
Fixes #979.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jskeet jskeet

Labels
bug
Projects
None yet
Milestone
2.2.1
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
@jskeet @malcolmr