Fix silly logic inversion in add_todo_entry_dir()
authorSteve McIntyre <steve@einval.com>
Sun, 15 May 2011 02:58:43 +0000 (03:58 +0100)
committerSteve McIntyre <steve@einval.com>
Sun, 15 May 2011 02:58:43 +0000 (03:58 +0100)
When checking if a file is a flac or not (for ignoring purposes),
forgot to invert the test.

Added extra debug for the selection logic here.

C/fuse-music.c

index 3b798a4..57f4387 100644 (file)
@@ -529,20 +529,30 @@ static int add_todo_entry_dir(const char *flac_dir, int keep)
         }
 
         if (should_encode && !S_ISREG(st.st_mode))
+        {
             /* Not a file - don't encode it */
             should_encode = 0;
+            MAINLOG1((logfile, "%s: dirent %s (%s) is not a file, ignore it\n", __func__, de->d_name, flac_path));
+        }
         
-        if (should_encode && is_flac(de->d_name))
+        if (should_encode && !is_flac(de->d_name))
+        {
             /* Not a flac file - don't encode it */
             should_encode = 0;
-
+            MAINLOG1((logfile, "%s: dirent %s (%s) is not a flac, ignore it\n", __func__, de->d_name, flac_path));
+        }
+        
         if (should_encode && !keep && (0 == encoded_size(flac_path, &enc_size)))
+        {
             /* We've been called for readdir(), so we have nokeep. The
              * file already exists in the size cache validly at this
              * point, so there's no need to re-encode it *again* just
              * for a readdir lookahead at this point. */
             should_encode = 0;
-
+            MAINLOG1((logfile, "%s: dirent %s (%s) does not need re-encoding for readdir, ignore it\n",
+                      __func__, de->d_name, flac_path));
+        }
+        
         if (should_encode)
         {
             pthread_t thread;