fixes #34

Author: vchuravy

src/ufixed.jl

@@ -80,8 +80,8 @@ sizeof{T<:UFixed}(::Type{T}) = sizeof(rawtype(T))
 abs(x::UFixed) = x
 
 # Arithmetic
-*{T,f}(x::UFixed{T,f}, y::UFixed{T,f}) = UFixed{T,f}((Base.widemul(x[],y[]) + (convert(widen(T), 1) << (f-1) ))>>f,0)
-/{T,f}(x::UFixed{T,f}, y::UFixed{T,f}) = UFixed{T,f}(div(convert(widen(T), x[])<<f, y[]),0)
+*{T<:UFixed}(x::T, y::T) = convert(T,convert(Float32, x)*convert(Float32, y))
+/{T<:UFixed}(x::T, y::T) = convert(T,convert(Float32, x)/convert(Float32, y))
 
 # Functions
 trunc{T<:UFixed}(x::T) = T(div(x[], rawone(T))*rawone(T),0)

test/ufixed.jl

@@ -20,6 +20,7 @@ using FixedPointNumbers, Base.Test
 for T in FixedPointNumbers.UF
     @test zero(T) == 0
     @test one(T) == 1
+    @test one(T) * one(T) == one(T)
     @test typemin(T) == 0
     @test realmin(T) == 0
     @test eps(zero(T)) == eps(typemax(T))
@@ -78,11 +79,7 @@ for T in FixedPointNumbers.UF
     @test_approx_eq((x+y)-x, fy)
     @test_approx_eq((x-y)+y, fx)
     @test_approx_eq(x*y, convert(T, fx*fy))
-    if T == UFixed14
-        @test convert(T, fx/fy) - x/y == eps(T)
-    else
-        @test_approx_eq(x/y, convert(T, fx/fy))
-    end
+    @test_approx_eq(x/y, convert(T, fx/fy))
     @test_approx_eq(x^2, convert(T, fx^2))
     @test_approx_eq(x^2.1f0, fx^2.1f0)
     @test_approx_eq(x^2.1, convert(Float64, x)^2.1)