Menu
Search

Forums

Forum Index

Thread overview
[Issue 22947] sysErrorString throws Exception instead of WindowsException
Mar 27, 2022
Dlang Bot
Mar 31, 2022
Dlang Bot
March 27, 2022
Gravatar of moonlightsentinel@disroot.orgPosted by moonlightsentinel@disroot.orgReply
moonlightsentinel@disroot.org
Gravatar of moonlightsentinel@disroot.org


Reply
https://issues.dlang.org/show_bug.cgi?id=22947

moonlightsentinel@disroot.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|sysErrorString throws a     |sysErrorString throws
                   |Exception instead           |Exception instead of
                   |WindowsException            |WindowsException

--
March 27, 2022
Gravatar of Dlang BotPosted by Dlang BotReply
Dlang Bot
Gravatar of Dlang Bot


Reply
https://issues.dlang.org/show_bug.cgi?id=22947

Dlang Bot <dlang-bot@dlang.rocks> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |pull

--- Comment #1 from Dlang Bot <dlang-bot@dlang.rocks> ---
@MoonlightSentinel created dlang/phobos pull request #8422 "Fix 22947 - Use wenforce in sysErrorString" fixing this issue:

- Fix 22947 - Use wenforce in sysErrorString

  This ensures that the method throws a `WindowsException` initialised
  with the proper error code. Also avoids unrelated exceptions that
  could suppress the error which caused the call to `sysErrorString`.

https://github.com/dlang/phobos/pull/8422

--
March 31, 2022
Gravatar of Dlang BotPosted by Dlang BotReply
Dlang Bot
Gravatar of Dlang Bot


Reply
https://issues.dlang.org/show_bug.cgi?id=22947

Dlang Bot <dlang-bot@dlang.rocks> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #2 from Dlang Bot <dlang-bot@dlang.rocks> ---
dlang/phobos pull request #8422 "Fix 22947 - Use wenforce in sysErrorString" was merged into master:

- 5c60f05dc94be988f69d3e7b025205cf657f80b1 by MoonlightSentinel:
  Fix 22947 - Use wenforce in sysErrorString

  This ensures that the method throws a `WindowsException` initialised
  with the proper error code. Also avoids unrelated exceptions that
  could suppress the error which caused the call to `sysErrorString`.

https://github.com/dlang/phobos/pull/8422

--