<html> <head> <base href="https://bugs.freedesktop.org/"> </head> <body> <div> <a class="bz_bug_link bz_status_REOPENED " title="REOPENED - [HSW][regression][bisect] RPG Maker game gives "invalid floating point operation" at startup" href="https://bugs.freedesktop.org/show_bug.cgi?id=95419#c10">Comment # 10</a> on <a class="bz_bug_link bz_status_REOPENED " title="REOPENED - [HSW][regression][bisect] RPG Maker game gives "invalid floating point operation" at startup" href="https://bugs.freedesktop.org/show_bug.cgi?id=95419">bug 95419</a> from <a class="email" href="mailto:pinguin255@gmail.com" title="dnord <pinguin255@gmail.com>"> dnord</a> <pre>(In reply to Matt Turner from <a href="show_bug.cgi?id=95419#c8">comment #8</a>) > Sorry, I had to revert this. It broke various piglit tests. > > The problem is that total_wants and remaining_space are updated between each > calculation, making the transformation your patch made incorrect. I know they are updated, but I don't understand why it's incorrect. I mean look at this: >remaining_space -= vs_additional; >total_wants -= vs_wants; Because vs_additional/vs_wants = remaining_space/total_wants, remaining_space/total_wants should remain the same after update and code should be equivalent, up to rounding errors. Maybe it's just rounding up error and after replace "round" with "floor" it will work? (can you test this?) Anyway I try to look at this more closely.</pre> </div> <hr> You are receiving this mail because: <ul> <li>You are the QA Contact for the bug.</li> </ul> </body> </html>