Je signale quand-même que le passage par référence non-const est contraire aux principes de style Qt parce que ça cache le fait que le paramètre peut être modifié, l'équipe Qt conseille d'utiliser les pointeurs parce qu'on voit plus clairement ce qui se passe.

De plus, un pointeur, ça permet de passer NULL si on n'a rien à battre d'une sortie (à condition que la fonction vérifie ce cas, évidemment), une référence oblige à passer un objet temporaire bidon (parce qu'on n'a pas le droit d'avoir une référence sur NULL, le comportement est indéfini si on en a une).

Mais comme toute question de style, ça se discute.