[Webkit-unassigned] [Bug 112354] [BlackBerry] Resolves 3 WebGL conformance suite crashes.
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Thu Mar 14 09:12:32 PDT 2013
https://bugs.webkit.org/show_bug.cgi?id=112354
--- Comment #5 from Rob Buis <rwlbuis at gmail.com> 2013-03-14 09:14:58 PST ---
(In reply to comment #4)
> (In reply to comment #2)
> > (From update of attachment 193130 [details] [details])
> > View in context: https://bugs.webkit.org/attachment.cgi?id=193130&action=review
> >
> > > Source/WebCore/platform/graphics/blackberry/GraphicsContext3DBlackBerry.cpp:427
> > > + sizeofChannels *= 1;
> >
> > I guess this is not needed. Maybe the compiler does not generate the instructions though.
> >
> > > Source/WebCore/platform/graphics/blackberry/GraphicsContext3DBlackBerry.cpp:454
> > > + sizeofChannels *= 1;
> >
> > Ditto.
>
> You're definitely correct that it's needless. I was trying to be more clear about what's going on. Perhaps I could add case statements which simply break immediately without doing a multiply. Is that a fair compromise?
Sounds good :)
--
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