8000 `PadStart` should to throw `ArgumentOutOfRangeException` for invalid width · Issue #1028 · morelinq/MoreLINQ · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

PadStart should to throw ArgumentOutOfRangeException for invalid width #1028

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.

Sign up for GitHub

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
Closed
atifaziz opened this issue Oct 28, 2023 · 0 comments
Closed

PadStart should to throw ArgumentOutOfRangeException for invalid width #1028

atifaziz opened this issue Oct 28, 2023 · 0 comments
Assignees
@atifaziz
Milestone
4.0.0

Comments

@atifaziz
Copy link
Member

Like #1027, PadStart should throw ArgumentException when the width argument is invalid. Is should throw ArgumentOutOfRangeException instead.

This is a compatible change.
@atifaziz atifaziz added this to the 4.0 milestone Oct 28, 2023
@atifaziz atifaziz self-assigned this Oct 28, 2023
@atifaziz atifaziz mentioned this issue Oct 28, 2023
Merged
julienasp pushed a commit to julienasp/MoreLINQ that referenced this issue Nov 14, 2023
@atifaziz @julienasp
Fix exception thrown for invalid "Pad"/"PadStart" width
de20099 
This is a squashed merge of PR morelinq#1030 that:

- fixes morelinq#1027
- fixes morelinq#1028
4991
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@atifaziz atifaziz

Labels
None yet
Projects
None yet
Milestone
4.0.0
Development

No branches or pull requests
1 participant
@atifaziz
0