You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The GDA spec says Rem should fail in the same ways as QuoInt, which means literally in the same ways. Since this means we (effectively) cannot optimize QuoInt and Rem to reduce computations, for code clarity we should just merge the two into QuoRem.
The text was updated successfully, but these errors were encountered:
ericlagergren
changed the title
Squash QuoInt and Rem into QuoRem.
Merge QuoInt and Rem into QuoRem.
Jan 29, 2018
Are you saying that you want to eliminate QuoInt and Rem entirely (which would be a breaking API change), or are you saying that the implementations of QuoInt and Rem should be converted to calls to QuoRem with a discarded value?
@kentquirk Yeah, converted to calls to QuoRem with a discarded value.
Looking at #72, I'm not entirely sure this is still necessary. It's been 6 months since I created it so I'll have to reinvestigate. But yeah, no API breakage.
The GDA spec says
Rem
should fail in the same ways asQuoInt
, which means literally in the same ways. Since this means we (effectively) cannot optimizeQuoInt
andRem
to reduce computations, for code clarity we should just merge the two intoQuoRem
.The text was updated successfully, but these errors were encountered: