← Back to team overview

desktop-packages team mailing list archive

[Bug 1209326] Re: SRU micro-release (MRE) 3.10.3 from utopic to trusty - calculator fails to do proper calculatoins after CTRL+Z and Enter

 

@Amr Ibrahim: I'm not a sponsor, but I had a quick glance at your patch,
and I'm pretty sure it's not correct. The merge proposal that Leon Liao
prepared for saucy looks much more convincing to me. If you want this to
be dealt with soon, I would suggest that you try to prepare a similar
merge proposal against trusty-proposed. Please note that changes may not
be made to the original source directly, but you need to do it via a
patch in debian/patches.

-- 
You received this bug notification because you are a member of Desktop
Packages, which is subscribed to gnome-calculator in Ubuntu.
https://bugs.launchpad.net/bugs/1209326

Title:
  SRU micro-release (MRE) 3.10.3 from utopic to trusty - calculator
  fails to do proper calculatoins after CTRL+Z and Enter

Status in GNOME Calculator:
  Fix Released
Status in “gnome-calculator” package in Ubuntu:
  Fix Released
Status in “gnome-calculator” source package in Trusty:
  Triaged

Bug description:
  Micro-release 3.10.3 in utopic fixes Bug #1209326 and Bug #1362919.

  The update fixes these issues:
      * Fix crash on super/subscript buttons
      * Fix previous answer on undo --> (Bug #1209326)
      * Fix segfault on repeated undo
      * Update translation

  I've backported the package from utopic to trusty without any changes
  to my PPA
  https://launchpad.net/~amribrahim1987/+archive/ubuntu/ppa/+packages?field.name_filter
  =gnome-calculator&field.status_filter=published&field.series_filter=

  I've attached a diff file. Please test and SRU to trusty.

  [Impact]

  This bug gives wrong results when undo is used.

  [Test Case]

  1. Open calculator and type "5000/60" followed by <Enter>. The
  calculator produces the right answer, 83.333...

  2. Type "/60" followed by <Enter> to divide by 60 again. The
  calculator again produces the right answer, 1.388...

  3. Press <ctrl>-Z to undo the last calculation.

  4. Press <backspace> twice to delete the last "60".

  5. Type "12" followed by <Enter>, to divide by 12 instead.

  The calculator gives the wrong answer.

  [Regression Potential]

  The fixes themselves could introduce new bugs.

  -----------------------------------------------------------------------------

  I found this bug trying to redo a calculation. Do a calculation maybe
  with decimals like I did and then press CTRL+Z and hit enter and you
  should see the bug.

  It's making a bad calculation after these steps.

  ProblemType: Bug
  DistroRelease: Ubuntu 13.04
  Package: gnome-calculator 1:3.8.1-0ubuntu1
  Uname: Linux 3.10.2-031002-generic x86_64
  ApportVersion: 2.9.2-0ubuntu8.1
  Architecture: amd64
  Date: Wed Aug  7 13:08:05 2013
  EcryptfsInUse: Yes
  InstallationDate: Installed on 2013-07-10 (28 days ago)
  InstallationMedia: Ubuntu 13.04 "Raring Ringtail" - Release amd64 (20130424)
  MarkForUpload: True
  SourcePackage: gnome-calculator
  UpgradeStatus: No upgrade log present (probably fresh install)

To manage notifications about this bug go to:
https://bugs.launchpad.net/gnome-calculator/+bug/1209326/+subscriptions