[Issue 5322] New: std.math: version(Sparc) should be SPARC

[d-bugmail at puremagic.com]

http://d.puremagic.com/issues/show_bug.cgi?id=5322

           Summary: std.math: version(Sparc) should be SPARC
           Product: D
           Version: D2
          Platform: Other
        OS/Version: Linux
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Phobos
        AssignedTo: nobody at puremagic.com
        ReportedBy: ibuclaw at ubuntu.com


--- Comment #0 from Iain Buclaw <ibuclaw at ubuntu.com> 2010-12-04 14:52:46 PST ---
That seems to be what's used in druntime as the version identifier for SPARC,
should adhere to the same in phobos (or, at the very least, agree on a naming
convention for all architectures)

ie:
ALPHA, ARM, PPC, MIPS, SPARC, X86, X86_64
vs
Alpha, Arm, Ppc, Mips, Sparc, X86, X86_64
vs
alpha, arm, ppc, mips, sparc, x86, x86_64

I think the first is the most agreeable. :~)

Regards


Patch:

--- phobos-trunk~/phobos/std/math.d    2010-12-03 03:25:12.000000000 +0000
+++ phobos-trunk/phobos/std/math.d    2010-12-04 22:47:59.777486614 +0000
@@ -1948,7 +1948,7 @@
             DIVBYZERO_MASK = 0x020,
             INVALID_MASK   = 0xF80 // PowerPC has five types of invalid
exceptions.
         }
-    } else version(Sparc) { // SPARC FSR is a 32bit register
+    } else version (SPARC) { // SPARC FSR is a 32bit register
              //(64 bits for Sparc 7 & 8, but high 32 bits are uninteresting).
         enum : int {
             INEXACT_MASK   = 0x020,

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