apt.dat parser: indentation fixes; add a comment

- The first place where indentation is fixed was really misleading, at
  least.

- The added comment "'line' may end with an \r character [...]" just
  clarifies something that has been there for ages, to avoid people
  falling into the trap.

diff --git a/src/Airports/apt_loader.cxx b/src/Airports/apt_loader.cxx
index 9fe89a42a996218a67437a91af0d46bfd3c7cb2f..0dd3512b4520c63608ffafa65a9fda3f8aac84ba 100644 (file)
--- a/src/Airports/apt_loader.cxx
+++ b/src/Airports/apt_loader.cxx
@@ -129,23 +129,24 @@ public:
     throwExceptionIfStreamError(in, "apt.dat", apt_dat);
 
     while ( std::getline(in, line) ) {
+      // 'line' may end with an \r character, see above
       line_num++;
 
       if ( isBlankOrCommentLine(line) )
         continue;
 
-        if ((line_num % 100) == 0) {
-            // every 100 lines
-            unsigned int percent = (line_num * 100) / LINES_IN_APT_DAT;
-            cache->setRebuildPhaseProgress(NavDataCache::REBUILD_AIRPORTS, percent);
-        }
+      if ((line_num % 100) == 0) {
+        // every 100 lines
+        unsigned int percent = (line_num * 100) / LINES_IN_APT_DAT;
+        cache->setRebuildPhaseProgress(NavDataCache::REBUILD_AIRPORTS, percent);
+      }
 
       // Extract the first field into 'line_id'
       line_id = atoi(line.c_str());
 
       if ( line_id == 1 /* Airport */ ||
-                    line_id == 16 /* Seaplane base */ ||
-                    line_id == 17 /* Heliport */ ) {
+           line_id == 16 /* Seaplane base */ ||
+           line_id == 17 /* Heliport */ ) {
         parseAirportLine(simgear::strutils::split(line));
       } else if ( line_id == 10 ) { // Runway v810
         parseRunwayLine810(simgear::strutils::split(line));