Opened 9 years ago

Closed 9 years ago

#3363 closed defect (fixed)

Error in Rational divide

Reported by: miguelgarcia Owned by: garycmartin
Priority: High Milestone: Unspecified
Component: Calculate Version: Git as of bugdate
Severity: Major Keywords:
Cc: Distribution/OS: Fedora
Bug Status: Unconfirmed

Description

Bad result with rational division.
Ex: (1/4)/(1/2) should output 0.5 but instead output 8

Attachments (1)

0001-Fix-error-in-Rational-division.patch (773 bytes) - added by miguelgarcia 9 years ago.
patch for rational division

Download all attachments as: .zip

Change History (5)

comment:1 Changed 9 years ago by miguelgarcia

  • Owner changed from rwh to miguelgarcia
  • Status changed from new to accepted

Changed 9 years ago by miguelgarcia

patch for rational division

comment:2 follow-up: Changed 9 years ago by miguelgarcia

  • Owner changed from miguelgarcia to rwh
  • Status changed from accepted to assigned

Hi, i attached a patch for this issue
Regards

comment:3 Changed 9 years ago by godiard

  • Owner changed from rwh to garycmartin

comment:4 in reply to: ↑ 2 Changed 9 years ago by garycmartin

  • Resolution set to fixed
  • Status changed from assigned to closed

Replying to miguelgarcia:

Hi, i attached a patch for this issue
Regards

Many thanks for catching this nasty bug Miguel! Landed the patch in mainline and will build a Calculate-39 release soon:

http://git.sugarlabs.org/calculate/mainline/commit/2690934c0fc7c20b72884307e3759ec71ec1fdc8

Note: See TracTickets for help on using tickets.