Comment 3 for bug 1101541

Revision history for this message
Ted Gould (ted) wrote : Re: [Bug 1101541] Re: Coverity DIVIDE_BY_ZERO - CID 12435

On Sat, 2013-06-15 at 13:50 +0000, MC Return wrote:

> PrivateCubeScreen::updateGeometry (int,int) is always called like this
> updateGeometry (screen->vpSize ().width (), mInvert);, which means sides
> will never become 0, because screen->vpSize ().width () will always
> return a value >= 1.

Seems like that's the case in current code, but what if someone uses the
function in the future and doesn't realize? Seems to me like you could
check it with an unlikely branch just to protect against an assert
situation.