https://bugs.kde.org/show_bug.cgi?id=291129

Albert Astals Cid <tsdg...@terra.es> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|http://commits.kde.org/okul |http://commits.kde.org/okul
                   |ar/444e6b7b19bee5285f39a440 |ar/5fb937a46aaf4e531ada8166
                   |07cf00f6e0b235d2            |92b1e47fc95b20b0

--- Comment #16 from Albert Astals Cid <tsdg...@terra.es> ---
Git commit 5fb937a46aaf4e531ada816692b1e47fc95b20b0 by Albert Astals Cid.
Committed on 20/06/2012 at 00:33.
Pushed by aacid into branch 'KDE/4.8'.

Fix underflow in getFreeMemory()

It actually serves three purposes:
a) Make sure all the values are there (this should be always true, but doesn't
hurt making sure) because if SwapFree was there but SwapTotal was not, it'd be
a mess
b) add up things in order so we don't underflow, currently the code did process
stuff as it came in the file, and it happens that SwapTotal appears before
SwapFree in /proc/meminfo so it actually did  "MemFree:" + "Buffers:" +
"Cached:" - "SwapTotal:" + "SwapFree:", which can underflow if "MemFree:" +
"Buffers:" + "Cached:" < "SwapTotal:"
c) Do not underflow at all, so if  "MemFree:" + "Buffers:" + "Cached:" + 
"SwapFree:" < "SwapTotal:" we return 0 correctly not a zillion of free memory

Aurélien you might want to update gwenviews copy of this code (there's a few
other bugfixes we did a while ago you didn't update either)

CCMAIL: agat...@kde.org
(cherry picked from commit 444e6b7b19bee5285f39a44007cf00f6e0b235d2)

M  +22   -7    core/document.cpp

http://commits.kde.org/okular/5fb937a46aaf4e531ada816692b1e47fc95b20b0

-- 
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Okular-devel mailing list
Okular-devel@kde.org
https://mail.kde.org/mailman/listinfo/okular-devel

Reply via email to