Thread overview
[Issue 3895] New: Appending a double[] to a float[] no longer works.
Mar 08, 2010
David Simcha
Mar 08, 2010
David Simcha
[Issue 3895] Appending a double[] to a float[] generates wrong code
Feb 21, 2012
yebblies
Apr 26, 2012
Walter Bright
March 08, 2010
http://d.puremagic.com/issues/show_bug.cgi?id=3895

           Summary: Appending a double[] to a float[] no longer works.
           Product: D
           Version: 2.041
          Platform: Other
        OS/Version: Windows
            Status: NEW
          Keywords: wrong-code
          Severity: regression
          Priority: P2
         Component: DMD
        AssignedTo: nobody@puremagic.com
        ReportedBy: dsimcha@yahoo.com


--- Comment #0 from David Simcha <dsimcha@yahoo.com> 2010-03-08 06:22:04 PST ---
import std.stdio;

void main() {
    double[] stuff = [1.,2.,3.,4.,5.];
    float[] otherStuff;
    otherStuff ~= stuff;
    writeln(otherStuff);
}


Output:

0 1.875 0 2 0 2.125 0 2.25 0 2.3125

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
March 08, 2010
http://d.puremagic.com/issues/show_bug.cgi?id=3895


Steven Schveighoffer <schveiguy@yahoo.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |schveiguy@yahoo.com


--- Comment #1 from Steven Schveighoffer <schveiguy@yahoo.com> 2010-03-08 06:34:29 PST ---
When did this work?  I tested back to 2.036 and it still results in the same output (was testing in case it was something I did in my append patch).

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
March 08, 2010
http://d.puremagic.com/issues/show_bug.cgi?id=3895


David Simcha <dsimcha@yahoo.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Severity|regression                  |major


--- Comment #2 from David Simcha <dsimcha@yahoo.com> 2010-03-08 07:50:20 PST ---
You're right.  What I can't figure out is how code that depended on this working seemed to work until now.  It might have something to do with changes to the Appender struct, since the code that I noticed this in actually used an Appender instead of appending using ~=.

Changing status from regression to major.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
February 21, 2012
http://d.puremagic.com/issues/show_bug.cgi?id=3895


yebblies <yebblies@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |accepts-invalid
                 CC|                            |yebblies@gmail.com
           Platform|Other                       |All
            Version|2.041                       |D2
         AssignedTo|nobody@puremagic.com        |yebblies@gmail.com
         OS/Version|Windows                     |All
           Severity|major                       |critical


--- Comment #3 from yebblies <yebblies@gmail.com> 2012-02-21 14:47:14 EST ---
Wow this is awful.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
February 21, 2012
http://d.puremagic.com/issues/show_bug.cgi?id=3895



--- Comment #4 from github-bugzilla@puremagic.com 2012-02-20 21:32:37 PST ---
Commit pushed to master at https://github.com/D-Programming-Language/phobos

https://github.com/D-Programming-Language/phobos/commit/5aa1b47aedf85f9e67132e5fc7a1586d5f1b293a Merge pull request #446 from yebblies/issue3895

This only works because of a bug in dmd, and it was probably supposed to...

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
February 21, 2012
http://d.puremagic.com/issues/show_bug.cgi?id=3895



--- Comment #5 from github-bugzilla@puremagic.com 2012-02-20 21:36:19 PST ---
Commit pushed to master at https://github.com/D-Programming-Language/druntime

https://github.com/D-Programming-Language/druntime/commit/4610715c33531a9d11613ab78eec6af70f0f4851 Merge pull request #156 from yebblies/issue3895

This only works because of a bug in dmd, and it was probably supposed to...

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
April 26, 2012
http://d.puremagic.com/issues/show_bug.cgi?id=3895



--- Comment #6 from github-bugzilla@puremagic.com 2012-04-26 11:42:31 PDT ---
Commit pushed to master at https://github.com/D-Programming-Language/dmd

https://github.com/D-Programming-Language/dmd/commit/f20d0845f24361463acab77d314170b25b2825c4 fix Issue 3895 - Appending a double[] to a float[] generates wrong code

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
April 26, 2012
http://d.puremagic.com/issues/show_bug.cgi?id=3895


Walter Bright <bugzilla@digitalmars.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
                 CC|                            |bugzilla@digitalmars.com
         Resolution|                            |FIXED


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