Change: Display more useful information in sprite aligner than sprite ID.

[PeterN]

Motivation / Problem

Sprite IDs are not useful information given they change don't refer to anything outside the loaded game.

They may give (and has given) the false impression that Action A can be used with these IDs, when in fact anything outside of the original baseset is dynamic.

It also isn't very useful to terms of finding out where each sprite is within its file.

Description

Instead of showing the internal SpriteID, include the filename and nfo line at minimum, and include action A or action 5 sprite replacement information if applicable.

This splits of Action5Type information so that it is available to newgrf_debug_gui.cpp as well as newgrf.cpp -- this avoids duplicating the information (which my first version did...)

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