cpp/openttd-patchpack/source Changeset - r23375:ca293b1be7a0

Changeset - r23375:ca293b1be7a0

Parent rev.

Child rev.

[Not reviewed]

master

0 2 0

PeterN - 5 years ago 2019-03-02 08:06:02
peter@fuzzle.org

Codechange: Make std::stack use std::vector container in string formatting/drawing. (#7305)

This is a very minor performance increase which can add up during operations such
as sorting. Performance impact my be platform/compiler dependent.

2 files changed with 3 insertions and 2 deletions:

src/gfx_layout.h

src/strings.cpp

0 comments (0 inline, 0 general)

src/gfx_layout.h

➞

Show inline comments

@@ @@ -19,6 +19,7 @@ @@
 #include <map>
 #include <string>
 #include <stack>
 #include <vector>
 #ifdef WITH_ICU_LAYOUT
 #include "layout/ParagraphLayout.h"
@@ @@ -35,7 +36,7 @@ struct FontState { @@
 	FontSize fontsize;       ///< Current font size.
 	TextColour cur_colour;   ///< Current text colour.
 	std::stack<TextColour> colour_stack; ///< Stack of colours to assist with colour switching.
+	std::stack<TextColour, std::vector<TextColour>> colour_stack; ///< Stack of colours to assist with colour switching.
 	FontState() : fontsize(FS_END), cur_colour(TC_INVALID) {}
 	FontState(TextColour colour, FontSize fontsize) : fontsize(fontsize), cur_colour(colour) {}

src/strings.cpp

➞

Show inline comments

@@ @@ -791,7 +791,7 @@ static char *FormatString(char *buff, co @@
 	WChar b = '\0';
 	uint next_substr_case_index = 0;
 	char *buf_start = buff;
 	std::stack<const char *> str_stack;
+	std::stack<const char *, std::vector<const char *>> str_stack;
 	str_stack.push(str_arg);
 	for (;;) {

0 comments (0 inline, 0 general)