'cppcheck' noted that some variables in the SDL2 port could be more narrowly scoped (part of issue #376).

Author: Bill-Gray

sdl2/pdcdisp.c

@@ -28,8 +28,6 @@ static int foregr = -2, backgr = -2; /* current foreground, background */
 
 void PDC_update_rects(void)
 {
-    int i;
-
     if (rectcount)
     {
         /* if the maximum number of rects has been reached, we're
@@ -41,6 +39,7 @@ void PDC_update_rects(void)
         {
             int w = pdc_screen->w;
             int h = pdc_screen->h;
+            int i;
 
             for (i = 0; i < rectcount; i++)
             {

sdl2/pdcscrn.c

@@ -181,15 +181,14 @@ void PDC_scr_free(void)
 
 int _get_displaynum(void)
 {
-    SDL_Rect size;
-    int i, xpos, ypos, displays;
-
-    displays = SDL_GetNumVideoDisplays();
+    const int displays = SDL_GetNumVideoDisplays();
 
     if (displays > 1)
     {
-        SDL_GetGlobalMouseState(&xpos, &ypos);
+        int i, xpos, ypos;
+        SDL_Rect size;
 
+        SDL_GetGlobalMouseState(&xpos, &ypos);
         for (i = 0; i < displays; i++)
         {
             SDL_GetDisplayBounds(i, &size);