Ok, the assert sort of makes sense then, but perhaps fix the formatting,
and add a specific comment.

Den 10.7.2013 kl. 13:58 skrev tobiatesan <notifications@github.com>:

In engines/wintermute/graphics/transparent_surface.cpp:

> @@ -383,16 +497,68 @@ void TransparentSurface::doBlitAlpha(byte *ino, byte* outo, uint32 width, uint32
>  	return retSize;
>  }
>
> -TransparentSurface *TransparentSurface::scale(uint16 newWidth, uint16 newHeight) const {
> +TransparentSurface *TransparentSurface::rotoscale(const TransformStruct &transform) const {
> +	
> +	assert (transform._angle != 0); // This would not be ideal

 Asserts are function calls, so no space.
Oh, right.
For some reason I was blind to this, sorry.
But, why assert here at all? If the angle is 0, then simply skip the roto.

Well, I thought we agreed that we should never end up here with a zero
angle, because of the various if()'ing done before.

—
Reply to this email directly or view it on
GitHub<https://github.com/scummvm/scummvm/pull/354/files#r5112906>
.