Bug 17894 – Generalizing std.range.primitives.empty to use .length causes the style_lint target to fail due to std.regex.internal.ir.Bytecode.length being defined as uint

Status
NEW
Severity
blocker
Priority
P1
Component
phobos
Product
D
Version
D2
Platform
All
OS
All
Creation time
2017-10-11T10:41:56Z
Last change time
2024-12-01T16:30:53Z
Assigned to
No Owner
Creator
ZombineDev
Blocks
6409
Moved to GitHub: phobos#9721 →

Comments

Comment #0 by petar.p.kirov — 2017-10-11T10:41:56Z
See https://github.com/dlang/phobos/pull/5726#issuecomment-335768162 for my analysis. I have fix and a PR is imminent.
Comment #1 by petar.p.kirov — 2017-10-11T12:49:28Z
We decided to workaround the problem locally in std.range.primitives.empty by not instantiating `hasLength` at all.
Comment #2 by robert.schadek — 2024-12-01T16:30:53Z
THIS ISSUE HAS BEEN MOVED TO GITHUB https://github.com/dlang/phobos/issues/9721 DO NOT COMMENT HERE ANYMORE, NOBODY WILL SEE IT, THIS ISSUE HAS BEEN MOVED TO GITHUB