www.digitalmars.com         C & C++   DMDScript  

digitalmars.D - Recent regression with std.algorithm.maxElement and BigInt

reply "H. S. Teoh" <hsteoh qfbox.info> writes:
https://issues.dlang.org/show_bug.cgi?id=23993

This broke one of my projects. :-(

Basically, maxElement no longer works with ranges of BigInt, including
built-in arrays. That's a pretty major breakage IMO.


T

-- 
Tech-savvy: euphemism for nerdy.
Jun 15 2023
next sibling parent FeepingCreature <feepingcreature gmail.com> writes:
On Thursday, 15 June 2023 at 16:22:38 UTC, H. S. Teoh wrote:
 https://issues.dlang.org/show_bug.cgi?id=23993

 This broke one of my projects. :-(

 Basically, maxElement no longer works with ranges of BigInt, 
 including built-in arrays. That's a pretty major breakage IMO.


 T
Aw crud, this one is my fault. Big sorry. Lemme have a look.
Jun 15 2023
prev sibling parent reply FeepingCreature <feepingcreature gmail.com> writes:
On Thursday, 15 June 2023 at 16:22:38 UTC, H. S. Teoh wrote:
 https://issues.dlang.org/show_bug.cgi?id=23993

 This broke one of my projects. :-(

 Basically, maxElement no longer works with ranges of BigInt, 
 including built-in arrays. That's a pretty major breakage IMO.


 T
Sorry about that! PR is up. Should hopefully be fixed in the next patch release.
Jun 15 2023
next sibling parent Walter Bright <newshound2 digitalmars.com> writes:
On 6/15/2023 10:08 AM, FeepingCreature wrote:
 Sorry about that! PR is up. Should hopefully be fixed in the next patch
release.
Thanks for the quick action.
Jun 15 2023
prev sibling parent "H. S. Teoh" <hsteoh qfbox.info> writes:
On Thu, Jun 15, 2023 at 05:08:50PM +0000, FeepingCreature via Digitalmars-d
wrote:
 On Thursday, 15 June 2023 at 16:22:38 UTC, H. S. Teoh wrote:
 https://issues.dlang.org/show_bug.cgi?id=23993
 
 This broke one of my projects. :-(
 
 Basically, maxElement no longer works with ranges of BigInt,
 including built-in arrays. That's a pretty major breakage IMO.
[...]
 Sorry about that! PR is up. Should hopefully be fixed in the next
 patch release.
Thanks for the prompt response! T -- Ph.D. = Permanent head Damage
Jun 15 2023