Pointless use of abs(): All uses of the abs() function in the diff appear to be numerically pointless; if (abs (foo)) is the same as: if (foo)
Pointless temporary creation/destruction: 1345 + priv->region = priv->inputRegion = CompRegion (); Should just assign emptyRegion I think.
« Back to merge proposal
Pointless use of abs(): All uses of the abs() function in the diff appear to be numerically pointless;
if (abs (foo))
is the same as:
if (foo)
Pointless temporary creation/ destruction:
1345 + priv->region = priv->inputRegion = CompRegion ();
Should just assign emptyRegion I think.