]> granicus.if.org Git - taglib/commitdiff
Amend an outdated comment.
authorTsuda Kageyu <tsuda.kageyu@gmail.com>
Sat, 9 Jan 2016 01:13:07 +0000 (10:13 +0900)
committerTsuda Kageyu <tsuda.kageyu@gmail.com>
Sat, 9 Jan 2016 01:15:42 +0000 (10:15 +0900)
taglib/mpeg/mpegheader.cpp

index 1b93560b1dc26aba94d2a0058667a93382258fc0..e678f15bf95b8c911a98b2654de41fc92b9ede40 100644 (file)
@@ -299,12 +299,14 @@ void MPEG::Header::parse(File *file, long offset, bool checkLength)
   if(d->isPadded)
     d->frameLength += paddingSize[layerIndex];
 
-  // Check if the frame length has been calculated correctly, or the next frame
-  // synch bytes are right next to the end of this frame.
+  if(checkLength) {
 
-  // We read some extra bytes to be a bit tolerant.
+    // Check if the frame length has been calculated correctly, or the next frame
+    // header is right next to the end of this frame.
 
-  if(checkLength) {
+    // The MPEG versions, layers and sample rates of the two frames should be
+    // consistent. Otherwise, we assume that either or both of the frames are
+    // broken.
 
     file->seek(offset + d->frameLength);
     const ByteVector nextData = file->readBlock(4);
@@ -314,10 +316,6 @@ void MPEG::Header::parse(File *file, long offset, bool checkLength)
       return;
     }
 
-    // The MPEG versions, layers and sample rates of the two frames should be
-    // consistent. Otherwise, we assume that either or both of the frames are
-    // broken.
-
     const unsigned int HeaderMask = 0xfffe0c00;
 
     const unsigned int header     = data.toUInt(0, true)     & HeaderMask;