I think the whole point of the name horizontalAdvance() is that characters can actually be drawn outside of that width. I guess boundingRect().width() is a better port for width() in general.

Heh actually I just read the documentation for width and it agrees with that.

So yes this is a "same behaviour" port, but the whole point of the renaming is to make us realize these things. Sometimes horizontalAdvance is correct (like in kwordwrap.cpp, where they get added together), sometimes boundingRect is what we need, like here when it's to resize something and make sure all the pixels fit.