Codechange: Make all NWidgetPart arrays constexpr.
This ensures that the arrays are not created at runtime and prevents using non-constexpr values.
This commit is contained in:

committed by
Peter Nelson

parent
71b7ba0daa
commit
7737aa6640
@@ -508,7 +508,7 @@ struct NewGRFParametersWindow : public Window {
|
||||
GRFParameterInfo NewGRFParametersWindow::dummy_parameter_info(0);
|
||||
|
||||
|
||||
static const NWidgetPart _nested_newgrf_parameter_widgets[] = {
|
||||
static constexpr NWidgetPart _nested_newgrf_parameter_widgets[] = {
|
||||
NWidget(NWID_HORIZONTAL),
|
||||
NWidget(WWT_CLOSEBOX, COLOUR_MAUVE),
|
||||
NWidget(WWT_CAPTION, COLOUR_MAUVE, WID_NP_CAPTION),
|
||||
@@ -1808,7 +1808,7 @@ public:
|
||||
const uint NWidgetNewGRFDisplay::MAX_EXTRA_INFO_WIDTH = 150;
|
||||
const uint NWidgetNewGRFDisplay::MIN_EXTRA_FOR_3_COLUMNS = 50;
|
||||
|
||||
static const NWidgetPart _nested_newgrf_actives_widgets[] = {
|
||||
static constexpr NWidgetPart _nested_newgrf_actives_widgets[] = {
|
||||
NWidget(NWID_VERTICAL), SetPIP(0, WidgetDimensions::unscaled.vsep_wide, 0),
|
||||
/* Left side, presets. */
|
||||
NWidget(NWID_VERTICAL),
|
||||
@@ -1863,7 +1863,7 @@ static const NWidgetPart _nested_newgrf_actives_widgets[] = {
|
||||
EndContainer(),
|
||||
};
|
||||
|
||||
static const NWidgetPart _nested_newgrf_availables_widgets[] = {
|
||||
static constexpr NWidgetPart _nested_newgrf_availables_widgets[] = {
|
||||
NWidget(WWT_FRAME, COLOUR_MAUVE), SetDataTip(STR_NEWGRF_SETTINGS_INACTIVE_LIST, STR_NULL), SetPIP(0, WidgetDimensions::unscaled.vsep_wide, 0),
|
||||
/* Left side, available grfs, filter edit box. */
|
||||
NWidget(NWID_HORIZONTAL), SetPIP(0, WidgetDimensions::unscaled.hsep_wide, 0),
|
||||
@@ -1896,7 +1896,7 @@ static const NWidgetPart _nested_newgrf_availables_widgets[] = {
|
||||
EndContainer(),
|
||||
};
|
||||
|
||||
static const NWidgetPart _nested_newgrf_infopanel_widgets[] = {
|
||||
static constexpr NWidgetPart _nested_newgrf_infopanel_widgets[] = {
|
||||
NWidget(NWID_VERTICAL), SetPIP(0, WidgetDimensions::unscaled.vsep_wide, 0),
|
||||
/* Right side, info panel. */
|
||||
NWidget(WWT_PANEL, COLOUR_MAUVE),
|
||||
@@ -1949,7 +1949,7 @@ std::unique_ptr<NWidgetBase> NewGRFDisplay()
|
||||
}
|
||||
|
||||
/* Widget definition of the manage newgrfs window */
|
||||
static const NWidgetPart _nested_newgrf_widgets[] = {
|
||||
static constexpr NWidgetPart _nested_newgrf_widgets[] = {
|
||||
NWidget(NWID_HORIZONTAL),
|
||||
NWidget(WWT_CLOSEBOX, COLOUR_MAUVE),
|
||||
NWidget(WWT_CAPTION, COLOUR_MAUVE), SetDataTip(STR_NEWGRF_SETTINGS_CAPTION, STR_TOOLTIP_WINDOW_TITLE_DRAG_THIS),
|
||||
@@ -2025,7 +2025,7 @@ void ShowNewGRFSettings(bool editable, bool show_params, bool exec_changes, GRFC
|
||||
}
|
||||
|
||||
/** Widget parts of the save preset window. */
|
||||
static const NWidgetPart _nested_save_preset_widgets[] = {
|
||||
static constexpr NWidgetPart _nested_save_preset_widgets[] = {
|
||||
NWidget(NWID_HORIZONTAL),
|
||||
NWidget(WWT_CLOSEBOX, COLOUR_GREY),
|
||||
NWidget(WWT_CAPTION, COLOUR_GREY), SetDataTip(STR_SAVE_PRESET_CAPTION, STR_TOOLTIP_WINDOW_TITLE_DRAG_THIS),
|
||||
@@ -2183,7 +2183,7 @@ static void ShowSavePresetWindow(const char *initial_text)
|
||||
}
|
||||
|
||||
/** Widgets for the progress window. */
|
||||
static const NWidgetPart _nested_scan_progress_widgets[] = {
|
||||
static constexpr NWidgetPart _nested_scan_progress_widgets[] = {
|
||||
NWidget(WWT_CAPTION, COLOUR_GREY), SetDataTip(STR_NEWGRF_SCAN_CAPTION, STR_TOOLTIP_WINDOW_TITLE_DRAG_THIS),
|
||||
NWidget(WWT_PANEL, COLOUR_GREY),
|
||||
NWidget(NWID_VERTICAL), SetPIP(0, WidgetDimensions::unscaled.vsep_wide, 0), SetPadding(WidgetDimensions::unscaled.modalpopup),
|
||||
|
Reference in New Issue
Block a user