if (mGridUnitPxEnv != 0) { mLogicalDpi.first = mGridUnitPxEnv * mGridUnitToLogicalDpiMultiplier; mLogicalDpi.second = mGridUnitPxEnv * mGridUnitToLogicalDpiMultiplier; } else { mLogicalDpi.first = mScale * mMirScaleToGridUnitMultiplier * mGridUnitToLogicalDpiMultiplier; mLogicalDpi.second = mScale * mMirScaleToGridUnitMultiplier * mGridUnitToLogicalDpiMultiplier; }
I think this could be simplified; if mScale is by default 1.0 and you'd initialize mGridUnitPxEnv to the default value of mMirScaleToGridUnitMultiplier (8.0), this could become one branch.
« Back to merge proposal
if (mGridUnitPxEnv != 0) {
mLogicalDpi. first = mGridUnitPxEnv * mGridUnitToLogi calDpiMultiplie r;
mLogicalDpi. second = mGridUnitPxEnv * mGridUnitToLogi calDpiMultiplie r;
mLogicalDpi. first = mScale * mMirScaleToGrid UnitMultiplier * mGridUnitToLogi calDpiMultiplie r;
mLogicalDpi. second = mScale * mMirScaleToGrid UnitMultiplier * mGridUnitToLogi calDpiMultiplie r;
} else {
}
I think this could be simplified; if mScale is by default 1.0 and you'd initialize mGridUnitPxEnv to the default value of mMirScaleToGrid UnitMultiplier (8.0), this could become one branch.