Re: Reduce parameters [was pi program]
On Fri, 2015-09-25 at 12:54 +, John Colvin via Digitalmars-d-learn wrote: > […] > I vastly prefer the UFCS version, but unfortunately reduce has > its arguments the wrong way around for that if you use the > version that takes a seed... In which case the reduce parameter list is wrong, this is a bug and should be fixed. Is there a bug report for this I can connect with? -- Russel. = Dr Russel Winder t: +44 20 7585 2200 voip: sip:russel.win...@ekiga.net 41 Buckmaster Roadm: +44 7770 465 077 xmpp: rus...@winder.org.uk London SW11 1EN, UK w: www.russel.org.uk skype: russel_winder signature.asc Description: This is a digitally signed message part
Re: Reduce parameters [was pi program]
On Saturday, 26 September 2015 at 06:28:22 UTC, Russel Winder wrote: On Fri, 2015-09-25 at 12:54 +, John Colvin via Digitalmars-d-learn wrote: […] I vastly prefer the UFCS version, but unfortunately reduce has its arguments the wrong way around for that if you use the version that takes a seed... In which case the reduce parameter list is wrong, this is a bug and should be fixed. Is there a bug report for this I can connect with? It's been argued about a lot. https://issues.dlang.org/show_bug.cgi?id=8755 https://github.com/D-Programming-Language/phobos/pull/861 https://github.com/D-Programming-Language/phobos/pull/1955 https://github.com/D-Programming-Language/phobos/pull/2033
Re: Reduce parameters [was pi program]
On Sat, 2015-09-26 at 10:46 +, John Colvin via Digitalmars-d-learn wrote: > […] I guess the summary is: it's a breaking change, so do it. No we can't do that it's a breaking change. Seems lame given all the other breaking changes that have been. Sad given that reduce is probably the single most important operation in parallel programming. > It's been argued about a lot. > > https://issues.dlang.org/show_bug.cgi?id=8755 > https://github.com/D-Programming-Language/phobos/pull/861 > https://github.com/D-Programming-Language/phobos/pull/1955 > https://github.com/D-Programming-Language/phobos/pull/2033 -- Russel. = Dr Russel Winder t: +44 20 7585 2200 voip: sip:russel.win...@ekiga.net 41 Buckmaster Roadm: +44 7770 465 077 xmpp: rus...@winder.org.uk London SW11 1EN, UK w: www.russel.org.uk skype: russel_winder signature.asc Description: This is a digitally signed message part
Re: Reduce parameters [was pi program]
On Monday, 28 September 2015 at 11:04:56 UTC, Russel Winder wrote: On Sat, 2015-09-26 at 10:46 +, John Colvin via Digitalmars-d-learn wrote: […] I guess the summary is: it's a breaking change, so do it. No we can't do that it's a breaking change. Seems lame given all the other breaking changes that have been. Sad given that reduce is probably the single most important operation in parallel programming. My thoughts exactly, even though it was partly me that pointed out the breaking changes... I still think we should add fold[lr]? just to fix the ordering.
Re: Reduce parameters [was pi program]
On Mon, 2015-09-28 at 11:37 +, John Colvin via Digitalmars-d-learn wrote: > […] > My thoughts exactly, even though it was partly me that pointed > out the breaking changes... Curses, if no-one had pointed out it was breaking maybe no-one would have noticed, and just made the change? > I still think we should add fold[lr]? just to fix the ordering. Whatever happens in std.algorithm must also happen in std.parallelism. reduce in std.parallelism was constructed to be in harmony with std.algorithm and not UFCS. -- Russel. = Dr Russel Winder t: +44 20 7585 2200 voip: sip:russel.win...@ekiga.net 41 Buckmaster Roadm: +44 7770 465 077 xmpp: rus...@winder.org.uk London SW11 1EN, UK w: www.russel.org.uk skype: russel_winder signature.asc Description: This is a digitally signed message part