FS#27924 - [pacman 4] 0,00 MiB 'Net Change' should be a non-negative value.

Attached to Project: Pacman
Opened by Karol Błażewicz (karol) - Friday, 13 January 2012, 05:09 GMT
Last edited by Dan McGee (toofishes) - Tuesday, 07 February 2012, 04:14 GMT
Task Type Bug Report
Category General
Status Closed
Assigned To No-one
Architecture All
Severity Very Low
Priority Normal
Reported Version 4.0.1
Due in Version 4.0.2
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

# pacman -Syu

...

Name Old Version New Version Net Change Download Size

chromium 16.0.912.63-2 16.0.912.75-1 -0,00 MiB 22,94 MiB
...
dmenu 4.4.1-1 4.5-1 0,00 MiB 0,01 MiB


What's the difference between 'Net Change' of -0,00 MiB and one of 0,00 MiB?
This task depends upon

Closed by  Dan McGee (toofishes)
Tuesday, 07 February 2012, 04:14 GMT
Reason for closing:  Fixed
Additional comments about closing:  Fixed in e8db984ce599.
Comment also fixed on maint.
Comment by Karol Błażewicz (karol) - Friday, 13 January 2012, 05:11 GMT
Great, forgot the forms don't preserve formatting. Let's try to make it easier to see:

Name_________Net Change
chromium____-0,00 MiB
dmenu________0,00 MiB
Comment by Dan McGee (toofishes) - Friday, 13 January 2012, 05:20 GMT
It's called rounding- it isn't 0 net change, it is somewhere between -0.005 and 0.000.

Patches welcome to round all values in this range to 0.
Comment by Thomas Dziedzic (tomd123) - Thursday, 02 February 2012, 23:02 GMT
attached a patch which should fix this
   0001-fix-fs-27924-don-t-displ... (5.4 KiB)
Comment by Dan McGee (toofishes) - Thursday, 02 February 2012, 23:27 GMT
Made a few minor adjustments, but this will make it in 4.0.2.
   0001-Fix-FS-27924-don-t-displ... (6 KiB)
Comment by Allan McRae (Allan) - Saturday, 04 February 2012, 10:16 GMT
There is a typo in the comment:

+ * @param precision number of decimal places, ensures -0.0.0 gets rounded to
+ * 0.00; -1 if no rounding desired

Loading...