[Webkit-unassigned] [Bug 105679] [Blackberry] Static code analysis warning fixes

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Jan 3 02:24:21 PST 2013


https://bugs.webkit.org/show_bug.cgi?id=105679





--- Comment #7 from Rob Buis <rwlbuis at gmail.com>  2013-01-03 02:26:21 PST ---
(In reply to comment #6)
> There are two accesses to the var 'page' in the if-stmt so combining the lines wouldn't have the same semantics.
> 
> The other access is the line further down:
> 
> float scaleFactorFudge = 1 / page->deviceScaleFactor();

I don't think that matters, page is visible in the entire scope of the if block. Can you recheck?

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list