patch: performance meter

[DorpsGek]

skidd13 opened the ticket and wrote:

A while ago Zojj wrote a patch to get a basic information how well his code improvements are. His patch worked only on Window$. At first I made a linux version of it. After a while Zojj assigned the patch to me. KUDr gave advice to me for the C++ conversion.

The patch can be used a FPS counter or as performance indicator.

There must be one thing checked: OS compatibility (at least Win32, Win64 and OSX)

forum topic: http://tt-forums.net/viewtopic.php?f=33&t=33121
svn rev: 10648
savegame bump: not needed

Attachments

• performance_meter_#25.patch (14.33 KiB)

Reported version: trunk
Operating system: Linux

---

This issue was imported from FlySpray: https://bugs.openttd.org/task/1065

[DorpsGek]

TrueBrain wrote:

This patch looks flawed... you have to keep on moving around in order for it to do anything.. any ideas?

---

This comment was imported from FlySpray: https://bugs.openttd.org/task/1065#comment1693

[DorpsGek]

skidd13 wrote:

Updated against trunk. Was a one liner.

svn rev: 10672

Attachments

• performance_meter_#26.patch (14.52 KiB)

---

This comment was imported from FlySpray: https://bugs.openttd.org/task/1065#comment1694

[DorpsGek]

skidd13 wrote:

So here is the update.
Now it is possible to switch between real time and process time.
I splitted also the internal code from the GUI to extra files.

I wrote only the code for SDL based real time.

svn rev: 10684

Attachments

• performance_meter_#27.patch (22.93 KiB)
• openttd.grf (29.26 KiB)

---

This comment was imported from FlySpray: https://bugs.openttd.org/task/1065#comment1707

[DorpsGek]

skidd13 wrote:

Ok here is another update of the patch.

Attachments

• performance_meter_#28.patch (29.04 KiB)

---

This comment was imported from FlySpray: https://bugs.openttd.org/task/1065#comment1713

[DorpsGek]

skidd13 wrote:

And another one.

Attachments

• performance_meter_#29.patch (22.87 KiB)

---

This comment was imported from FlySpray: https://bugs.openttd.org/task/1065#comment1716

[DorpsGek]

skidd13 wrote:

Reverted too much from 28->29 here is the fixed version.

Attachments

• performance_meter_#30.patch (22.87 KiB)

---

This comment was imported from FlySpray: https://bugs.openttd.org/task/1065#comment1717

[DorpsGek]

peter1138 wrote:

What does 'TPS' stand for, anyway?

It also seems rather strange that what is essentially a performance debugging a tool (but not a very useful one) should require a GUI and quite a lot of code.

---

This comment was imported from FlySpray: https://bugs.openttd.org/task/1065#comment1718

[DorpsGek]

skidd13 wrote:

TPS = Ticks Per Second
IMO the performance meter is for getting a basic impression of the current performance.
You could add a non GUI variant if you dislike the GUI.
The code has been written in such a way, because now other performance measurements could be added by using the same interface.
IIRC there are a few code parts in the source into this direction.

---

This comment was imported from FlySpray: https://bugs.openttd.org/task/1065#comment1720

[DorpsGek]

skidd13 wrote:

I extended the performance meter with a new type (CPU cycles per code calls).
It's nearly working as expected ;). The current value isn't drawn right. But I'd like to hear comments about the concept itself.
The idea how it should be used is in town_cmd.cpp

Attachments

• performance_meter_r10814.patch (31.61 KiB)

---

This comment was imported from FlySpray: https://bugs.openttd.org/task/1065#comment1833

[DorpsGek]

skidd13 wrote:

Sry for the double post. Here is the fixed version.

Attachments

• performance_meter_r10822.patch (31.55 KiB)

---

This comment was imported from FlySpray: https://bugs.openttd.org/task/1065#comment1834

[DorpsGek]

skidd13 wrote:

And another update. This fixes the memory leaks (I hope that I didn't missed any) and a few display bugs.

Attachments

• performance_meter_r10827.patch (31.66 KiB)

---

This comment was imported from FlySpray: https://bugs.openttd.org/task/1065#comment1841

[DorpsGek]

skidd13 closed the ticket.

Reason for closing: Out of date

---

This comment was imported from FlySpray: https://bugs.openttd.org/task/1065