Thread overview
[Issue 24027] error: instantiated from here: `maxElement!("a.a", A[])`
Jul 03, 2023
FeepingCreature
Jul 03, 2023
FeepingCreature
Jul 03, 2023
Dlang Bot
Jul 03, 2023
Dlang Bot
Jul 15, 2023
Dlang Bot
July 03, 2023
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
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
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
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
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

--