Bug 22125 – std.process.Config was changed to a struct but miss operator overloads, leading to user code breakage.

Status
RESOLVED
Resolution
FIXED
Severity
regression
Priority
P1
Component
phobos
Product
D
Version
D2
Platform
All
OS
All
Creation time
2021-07-14T20:48:35Z
Last change time
2021-07-16T08:47:52Z
Keywords
pull
Assigned to
No Owner
Creator
Basile-z

Comments

Comment #0 by b2.temp — 2021-07-14T20:48:35Z
the following code reduced from `iz` breaks: --- import std.process; void main() { Config c; c |= Config.detached; } --- with error > /tmp/temp_7F65D79EC950.d:3:45: Error: `c` is not a scalar, it is a `Config` all binary operators used for the bit fields operations should be supported, so `|`, `&`, `~`, `^` + their respective binary assign.
Comment #1 by dlang-bot — 2021-07-14T20:50:57Z
@CyberShadow created dlang/phobos pull request #8166 "Fix Issue 22125 - std.process.Config was changed to a struct but miss…" fixing this issue: - Fix Issue 22125 - std.process.Config was changed to a struct but miss operator overloads https://github.com/dlang/phobos/pull/8166
Comment #2 by dlang-bot — 2021-07-14T20:52:53Z
@CyberShadow created dlang/phobos pull request #8167 "Fix Issue 22125 - std.process.Config was changed to a struct but miss…" fixing this issue: - Fix Issue 22125 - std.process.Config was changed to a struct but miss operator overloads https://github.com/dlang/phobos/pull/8167
Comment #3 by dlang-bot — 2021-07-16T08:47:52Z
dlang/phobos pull request #8167 "Fix Issue 22125 - std.process.Config was changed to a struct but miss…" was merged into stable: - baf6ede6de920f17df6c48aa5eac32b6ea30ca10 by Vladimir Panteleev: Fix Issue 22125 - std.process.Config was changed to a struct but miss operator overloads https://github.com/dlang/phobos/pull/8167