Thread overview | |||||||
---|---|---|---|---|---|---|---|
|
July 03, 2023 [Issue 24027] error: instantiated from here: `maxElement!("a.a", A[])` | ||||
---|---|---|---|---|
| ||||
https://issues.dlang.org/show_bug.cgi?id=24027 FeepingCreature <default_357-line@yahoo.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |default_357-line@yahoo.de --- Comment #1 from FeepingCreature <default_357-line@yahoo.de> --- I think this is a dupe of https://issues.dlang.org/show_bug.cgi?id=23993 . Seems to be fixed on master. This was my fault, sorry - the rewrite to use `Rebindable` had to handle array cases separately. See https://github.com/dlang/phobos/pull/8764 . In my defense, that type was designed badly. -- |
July 03, 2023 [Issue 24027] error: instantiated from here: `maxElement!("a.a", A[])` | ||||
---|---|---|---|---|
| ||||
https://issues.dlang.org/show_bug.cgi?id=24027 --- Comment #2 from FeepingCreature <default_357-line@yahoo.de> --- Oh no, I see - this is the other other special case of `Rebindable`. I'll make a PR for stable. -- |
July 03, 2023 [Issue 24027] error: instantiated from here: `maxElement!("a.a", A[])` | ||||
---|---|---|---|---|
| ||||
https://issues.dlang.org/show_bug.cgi?id=24027 Dlang Bot <dlang-bot@dlang.rocks> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |pull --- Comment #3 from Dlang Bot <dlang-bot@dlang.rocks> --- @FeepingCreature created dlang/phobos pull request #8777 "Fix issue 24027: specialcase `Rebindable` for `class` as well." fixing this issue: - Fix issue 24027: specialcase `Rebindable` for `class` as well. Non-mutable `class` is another case where `Rebindable` just silently aliases itself away. Don't sweat that this looks ugly - this is fixed on `master` via the `Rebindable2` rewrite https://github.com/dlang/phobos/pull/8768 https://github.com/dlang/phobos/pull/8777 -- |
July 03, 2023 [Issue 24027] error: instantiated from here: `maxElement!("a.a", A[])` | ||||
---|---|---|---|---|
| ||||
https://issues.dlang.org/show_bug.cgi?id=24027 Dlang Bot <dlang-bot@dlang.rocks> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution|--- |FIXED --- Comment #4 from Dlang Bot <dlang-bot@dlang.rocks> --- dlang/phobos pull request #8777 "Fix issue 24027: Instead of a bunch of special-cases, just check that `Rebindable` did in fact alias itself away." was merged into stable: - 24ec18b2686effb0abbde6de61b590a5941e0609 by FeepingCreature: Fix issue 24027: Instead of a bunch of special-cases, just check that `Rebindable` did in fact alias itself away. This looks ugly, but it's just for the `stable` branch - this is fixed on `master` via the `Rebindable2` rewrite https://github.com/dlang/phobos/pull/8768 https://github.com/dlang/phobos/pull/8777 -- |
July 15, 2023 [Issue 24027] error: instantiated from here: `maxElement!("a.a", A[])` | ||||
---|---|---|---|---|
| ||||
https://issues.dlang.org/show_bug.cgi?id=24027 --- Comment #5 from Dlang Bot <dlang-bot@dlang.rocks> --- dlang/phobos pull request #8782 "merge stable" was merged into master: - dfcbee7056318734aaa88d9051369463eb94c637 by FeepingCreature: Fix issue 24027: Instead of a bunch of special-cases, just check that `Rebindable` did in fact alias itself away. This looks ugly, but it's just for the `stable` branch - this is fixed on `master` via the `Rebindable2` rewrite https://github.com/dlang/phobos/pull/8768 https://github.com/dlang/phobos/pull/8782 -- |
Copyright © 1999-2021 by the D Language Foundation