update: don't re-read unchanged container files

MPD checks if every flac (possibly other types as well) file contains cuesheet on every update, which produces unneeded I/O. My music collection is on NFS share, so it's quite noticeable. IMHO, it shouldn't re-read unchanged files, so I wrote simple patch to fix it.

update: don't re-read unchanged container files
9d42f4e0 · Igor Kuzmin · Max Kellermann · fd8aa54a · 9d42f4e0 · 9d42f4e0
Commit 9d42f4e0 authored Aug 19, 2009 by Igor Kuzmin Committed by Max Kellermann Aug 19, 2009
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 1 deletion

NEWS NEWS +1 -0

update.c src/update.c +2 -1

No files found.
--- a/NEWS
+++ b/NEWS
 ver 0.15.3 (2009/??/??)
 * output:
  - fix stuttering due to uninitialized variable
+* update: don't re-read unchanged container files
 ver 0.15.2 (2009/08/15)

--- a/src/update.c
+++ b/src/update.c
@@ -502,7 +502,8 @@ update_regular_file(struct directory *directory,
 	{
 		struct song* song = songvec_find(&directory->songs, name);
-		if (plugin->container_scan != NULL)
+		if (!(song != NULL && st->st_mtime == song->mtime) &&
+			plugin->container_scan != NULL)
 		{
 			if (update_container_file(directory, name, st, plugin))
 			{