Use ranged-for loops

src/SFML/Audio/SoundRecorder.cpp

@@ -90,9 +90,9 @@ const char** sfSoundRecorder_getAvailableDevices(size_t* count)
 
     if (cstringDevices.empty() && !stringDevices.empty())
     {
-        for (std::vector<std::string>::const_iterator it = stringDevices.begin(); it != stringDevices.end(); ++it)
+        for (const auto& stringDevice : stringDevices)
         {
-            cstringDevices.push_back(it->c_str());
+            cstringDevices.push_back(stringDevice.c_str());
         }
     }
 

src/SFML/Network/FtpStruct.h

@@ -81,8 +81,8 @@ struct sfFtpListingResponse
 
     ~sfFtpListingResponse()
     {
-        for (std::vector<const char*>::iterator it = Filenames.begin(); it != Filenames.end(); ++it)
+        for (const auto& filename : Filenames)
-            delete[] *it;
+            delete[] filename;
     }
 
     sf::Ftp::ListingResponse This;

src/SFML/Window/VideoMode.cpp

@@ -51,13 +51,13 @@ const sfVideoMode* sfVideoMode_getFullscreenModes(size_t* count)
     // Populate the array on first call
     if (modes.empty())
     {
-        const std::vector<sf::VideoMode>& SFMLModes = sf::VideoMode::getFullscreenModes();
+        const std::vector<sf::VideoMode>& sfmlModes = sf::VideoMode::getFullscreenModes();
-        for (std::vector<sf::VideoMode>::const_iterator it = SFMLModes.begin(); it != SFMLModes.end(); ++it)
+        for (const auto& sfmlMode : sfmlModes)
         {
             sfVideoMode mode;
-            mode.width        = it->width;
+            mode.width        = sfmlMode.width;
-            mode.height       = it->height;
+            mode.height       = sfmlMode.height;
-            mode.bitsPerPixel = it->bitsPerPixel;
+            mode.bitsPerPixel = sfmlMode.bitsPerPixel;
             modes.push_back(mode);
         }
     }