www.digitalmars.com         C & C++   DMDScript  

digitalmars.D.bugs - [Issue 5089] New: feqrel does not compile for floats

reply d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=5089

           Summary: feqrel does not compile for floats
           Product: D
           Version: D2
          Platform: Other
        OS/Version: Windows
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Phobos
        AssignedTo: nobody puremagic.com
        ReportedBy: ah08010-d yahoo.com



PDT ---
With 2.049, this code:
==========
module scratch;

import std.math;

void foo() {
    float a = 1, b = 1.00000000001;

    auto same = feqrel( a, b ) > 15;
}
==========

Emits this diagnostic:
==========
$ dmd scratch.d
d:\Devel\D\dmd2\windows\bin\..\..\src\phobos\std\math.d(3286): Error: function
std.math.feqrel!(float).feqrel has no return statement, but is expected to
return a value of type int
d:\Devel\D\dmd2\windows\bin\..\..\src\phobos\std\math.d(8): Error: template
instance std.math.feqrel!(float) error instantiating
==========

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Oct 20 2010
next sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=5089


Mike Wey <mike-wey planet.nl> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |mike-wey planet.nl



As far as i know Don wrote both the feqrel function in tango and phobos.
But he has only fixed the one in tango to work with floats.

See: http://www.dsource.org/projects/tango/changeset/4108

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Jul 09 2011
prev sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=5089


Lars T. Kyllingstad <bugzilla kyllingen.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
                 CC|                            |bugzilla kyllingen.net
         Resolution|                            |FIXED



14:04:18 PDT ---
https://github.com/donc/phobos/commit/a8aa8cdcd60d8e8a49a1febb48c079a515365cfc

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