mirror
/
OpenTTD
mirror of https://github.com/OpenTTD/OpenTTD
1
0
Fork 0
Code Issues

Codechange: Determine array length using 'std::size' instead of using a separate constant.

pull/14031/head
frosch 2025-04-17 14:37:44 +02:00 committed by frosch
parent 7cbf4f5560
commit 84e175ce63
1 changed files with 6 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
src/video/allegro_v.cpp
View File

@ -40,14 +40,13 @@ static FVideoDriver_Allegro iFVideoDriver_Allegro;
static BITMAP *_allegro_screen;
#define MAX_DIRTY_RECTS 100
static PointDimension _dirty_rects[MAX_DIRTY_RECTS];
static int _num_dirty_rects;
static PointDimension _dirty_rects[100];
static size_t _num_dirty_rects;
static Palette _local_palette; ///< Current palette to use for drawing.
void VideoDriver_Allegro::MakeDirty(int left, int top, int width, int height)
{
if (_num_dirty_rects < MAX_DIRTY_RECTS) {
if (_num_dirty_rects < std::size(_dirty_rects)) {
_dirty_rects[_num_dirty_rects].x = left;
_dirty_rects[_num_dirty_rects].y = top;
_dirty_rects[_num_dirty_rects].width = width;
@ -60,16 +59,16 @@ void VideoDriver_Allegro::Paint()
{
PerformanceMeasurer framerate(PFE_VIDEO);
int n = _num_dirty_rects;
size_t n = _num_dirty_rects;
if (n == 0) return;
_num_dirty_rects = 0;
if (n > MAX_DIRTY_RECTS) {
if (n > std::size(_dirty_rects)) {
blit(_allegro_screen, screen, 0, 0, 0, 0, _allegro_screen->w, _allegro_screen->h);
return;
}
for (int i = 0; i < n; i++) {
for (size_t i = 0; i < n; i++) {
blit(_allegro_screen, screen, _dirty_rects[i].x, _dirty_rects[i].y, _dirty_rects[i].x, _dirty_rects[i].y, _dirty_rects[i].width, _dirty_rects[i].height);
}
}