Codechange: Pass by reference to UpdateWidgetSize.

[PeterN]

Motivation / Problem

Parameters are passed to UpdateWidgetSize() by pointer, but they are not optional and are always present.

Description

Replace pointers with references instead.

Limitations

Checklist for review

Some things are not automated, and forgotten often. This list is a reminder for the reviewers.

• The bug fix is important enough to be backported? (label: 'backport requested')
• This PR touches english.txt or translations? Check the guidelines
• This PR affects the save game format? (label 'savegame upgrade')
• This PR affects the GS/AI API? (label 'needs review: Script API')
  • ai_changelog.hpp, game_changelog.hpp need updating.
  • The compatibility wrappers (compat_*.nut) need updating.
• This PR affects the NewGRF API? (label 'needs review: NewGRF')
  • newgrf_debug_data.h may need updating.
  • PR must be added to API tracker

[LordAro]

void function with a "primary" out param? Why not return it? (or even a tuple!)

[PeterN]

As mentioned, it's 3 in,out parameters. I've updated the doxygen to note this.

[rubidium42]

I'm also seeing changes to padding happening in several places, but not everywhere. So what's up with that? Should all the padding stay, or should all the padding go? The two places I put the comments to are definitely not the only places where that happened.