[Issue 6586] feqrel for const values too

2017-07-19 Thread via Digitalmars-d-bugs
https://issues.dlang.org/show_bug.cgi?id=6586

--- Comment #8 from github-bugzi...@puremagic.com ---
Commits pushed to dmd-cxx at https://github.com/dlang/phobos

https://github.com/dlang/phobos/commit/09ebc581ec022d0e1091d47887aab26366df79e0
rework and try to fix Issue 6586

https://github.com/dlang/phobos/commit/62acd72cec052169e7eb56d1c63c860ae92f9ef9
Merge pull request #3017 from 9il/frexp

--


[Issue 6586] feqrel for const values too

2015-06-17 Thread via Digitalmars-d-bugs
https://issues.dlang.org/show_bug.cgi?id=6586

--- Comment #7 from github-bugzi...@puremagic.com ---
Commits pushed to stable at https://github.com/D-Programming-Language/phobos

https://github.com/D-Programming-Language/phobos/commit/09ebc581ec022d0e1091d47887aab26366df79e0
rework and try to fix Issue 6586

https://github.com/D-Programming-Language/phobos/commit/62acd72cec052169e7eb56d1c63c860ae92f9ef9
Merge pull request #3017 from 9il/frexp

--


[Issue 6586] feqrel for const values too

2015-04-11 Thread via Digitalmars-d-bugs
https://issues.dlang.org/show_bug.cgi?id=6586

--- Comment #6 from github-bugzi...@puremagic.com ---
Commit pushed to master at https://github.com/D-Programming-Language/phobos

https://github.com/D-Programming-Language/phobos/commit/cf57c7529a63ac970167c65cb659d5249a3a1ab9
Merge pull request #3017 from 9il/frexp

--


[Issue 6586] feqrel for const values too

2015-02-27 Thread via Digitalmars-d-bugs
https://issues.dlang.org/show_bug.cgi?id=6586

github-bugzi...@puremagic.com changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

--


[Issue 6586] feqrel for const values too

2015-02-27 Thread via Digitalmars-d-bugs
https://issues.dlang.org/show_bug.cgi?id=6586

--- Comment #4 from github-bugzi...@puremagic.com ---
Commits pushed to master at https://github.com/D-Programming-Language/phobos

https://github.com/D-Programming-Language/phobos/commit/09ebc581ec022d0e1091d47887aab26366df79e0
rework and try to fix Issue 6586

https://github.com/D-Programming-Language/phobos/commit/62acd72cec052169e7eb56d1c63c860ae92f9ef9
Merge pull request #3017 from 9il/frexp

[2.067.0-b2][regression] fix Issues 14212 and 6586

--


[Issue 6586] feqrel for const values too

2015-02-27 Thread via Digitalmars-d-bugs
https://issues.dlang.org/show_bug.cgi?id=6586

--- Comment #5 from github-bugzi...@puremagic.com ---
Commit pushed to 2.067 at https://github.com/D-Programming-Language/phobos

https://github.com/D-Programming-Language/phobos/commit/cf57c7529a63ac970167c65cb659d5249a3a1ab9
Merge pull request #3017 from 9il/frexp

[2.067.0-b2][regression] fix Issues 14212 and 6586

--


[Issue 6586] feqrel for const values too

2014-04-08 Thread d-bugmail
https://d.puremagic.com/issues/show_bug.cgi?id=6586


hst...@quickfur.ath.cx changed:

   What|Removed |Added

 CC||hst...@quickfur.ath.cx


--- Comment #3 from hst...@quickfur.ath.cx 2014-04-08 13:23:49 PDT ---
Tested on git HEAD: the original code now compiles, although I did find a
different related failing case:

==Code:==
import std.math, std.stdio;
void main() {
const(double) a=1.0, b=2.0;
writeln(feqrel(a, b));
}
==Compile:==
/usr/src/d/phobos/std/math.d(5394): Error: cannot modify const expression diff
test.d(4): Error: template instance std.math.feqrel!(const(double)) error
instantiating

-- 
Configure issuemail: https://d.puremagic.com/issues/userprefs.cgi?tab=email
--- You are receiving this mail because: ---


[Issue 6586] feqrel for const values too

2011-09-02 Thread d-bugmail
http://d.puremagic.com/issues/show_bug.cgi?id=6586



--- Comment #2 from bearophile_h...@eml.cc 2011-09-02 03:18:39 PDT ---
(In reply to comment #1)

 I'm not really sure why double and const(double) need to be different types. 
 It
 causes a huge amount of template bloat, and I don't think we're getting much
 (if anything) in return.

Tell this again in the main D newsgroup :-)

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
--- You are receiving this mail because: ---


[Issue 6586] feqrel for const values too

2011-09-01 Thread d-bugmail
http://d.puremagic.com/issues/show_bug.cgi?id=6586


Don clugd...@yahoo.com.au changed:

   What|Removed |Added

 CC||clugd...@yahoo.com.au


--- Comment #1 from Don clugd...@yahoo.com.au 2011-09-01 12:06:31 PDT ---
I hit this myself a couple of days ago.
I prefer:

int feqrel(X)(const(X) x, const(X) y) @trusted pure nothrow
if (isFloatingPoint!(X))

which causes less template bloat, and also avoids some unpleasant issues where
x and y are different sizes.

I'm not really sure why double and const(double) need to be different types. It
causes a huge amount of template bloat, and I don't think we're getting much
(if anything) in return.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
--- You are receiving this mail because: ---