[Emms-patches] darcs patch: info-cache-dirty/coding

[Damien Elmes]

Tue Jun  6 01:33:39 JST 2006  Damien Elmes <address@hidden>
  * info-cache-dirty/coding
  
  * mark the info cache as dirty when it's modified, so we don't have to
    write it out all the time
  * save the cache as mule-utf-8 - comments? i'm not sure if this is
    correct

New patches:

[info-cache-dirty/coding
Damien Elmes <address@hidden>**20060605163339
 
 * mark the info cache as dirty when it's modified, so we don't have to
   write it out all the time
 * save the cache as mule-utf-8 - comments? i'm not sure if this is
   correct
] {
hunk ./emms-info.el 94
+(defvar emms-info-cache-dirty nil
+  "True if the cache has been updated since init.")
+
hunk ./emms-info.el 133
-    (if (or (not cached-track)
+    (when (or (not cached-track)
hunk ./emms-info.el 135
-        (puthash name track emms-info-cache))
+        (puthash name track emms-info-cache)
+        (setq emms-info-cache-dirty t))
hunk ./emms-info.el 146
-  (set-buffer (get-buffer-create " emms-info-cache "))
-  (erase-buffer)
-  (maphash (lambda (k v)
-             (insert (format
-                      "(puthash %S '%S emms-info-cache)\n" k v)))
-           emms-info-cache)
-  (write-region (point-min) (point-max) emms-info-cache-file)
-  (kill-buffer (current-buffer)))
+  (when emms-info-cache-dirty
+    (message "Saving emms info cache...")
+    (set-buffer (get-buffer-create " emms-info-cache "))
+    (erase-buffer)
+    (maphash (lambda (k v)
+               (insert (format
+                        "(puthash %S '%S emms-info-cache)\n" k v)))
+             emms-info-cache)
+    (set-buffer-file-coding-system 'mule-utf-8)
+    (write-region (point-min) (point-max) emms-info-cache-file)
+    (kill-buffer (current-buffer))
+    (message "Saving emms info cache...done")
+    (setq emms-info-cache-dirty nil)))
hunk ./emms-info.el 162
-  (load emms-info-cache-file t nil t))
+  (load emms-info-cache-file t nil t)
+  (setq emms-info-cache-dirty nil))
}

Context:

[emms-info caching (thanks to Damien Elmes)
address@hidden 
[Sort file names from `emms-source-file-directory-tree-function'.
address@hidden 
[Add some sources for inserting playlists without inserting their contents, and 
likewise for directories of playlist files.  Exclude some files and directories 
from being added when walking directories.
Michael Olson <address@hidden>**20060604195602] 
[emms-player-mpd: Differentiate between files and URLs when it makes sense to 
do so.
Michael Olson <address@hidden>**20060604195449] 
[Miscellaneous minor cleanups.
Michael Olson <address@hidden>**20060604195311] 
[Make sure we never have an empty track description when inserting a song into 
a playlist buffer.
Michael Olson <address@hidden>**20060604194940] 
[Remove debian-extras package as requested by ftpmasters (debian)
address@hidden 
[Put volume options in their own customize group.
Martin Schoenmakers <address@hidden>**20060601193853
 
 Added a separate emms-volume group for customize and put things there instead
 of in the main thing.
] 
[Make handling of multiple playlist buffers less error-prone.
Michael Olson <address@hidden>**20060531203810] 
[emms-volume.el: Cosmetic stuff, defvar -> defcustom
address@hidden 
[emms-volume.el: Minor cosmetic cleanup
address@hidden 
[emms-volme.el: Add some requires.
address@hidden 
[emms-volume-amixer.el: Provide a way to set the control for amixer
address@hidden 
[AUTHORS: Add Martin Schoenmakers. Welcome! :-)
address@hidden 
[Add emms-volume and emms-volume-amixer.
Martin Schoenmakers <address@hidden>**20060530223500
 
 New files: emms-volume.el provides some general volume changing things,
 including a minor mode to more easily change volume when not in the
 EMMS buffer. emms-volume-amixer.el is a backend using amixer.
 
] 
[emms-streams: Re-add space after prompt and use completion for type.
Michael Olson <address@hidden>**20060530190620] 
[emms-streams: When the user wants emms-streams to play the selected stream 
instead of add it, create our own playlist buffer.  When quitting, if we own 
the current playlist buffer, kill it.
Michael Olson <address@hidden>**20060530144243] 
[allow nonzero ogginfo exit plus some reindenting
Martin Schoenmakers <address@hidden>**20060530130411
 
 When ogginfo gave a nonzero value on exit, any valid data would get tossed
 if there was any. This prevented emms from showing info for files that are
 tagged but a bit odd.
 
 Also reindented emms-info-ogginfo accordingly, which incidentally removed
 some tabs in favour of spaces.
 
] 
[emms-streams: Re-implement yank and kill so that they do the right thing with 
emms-stream-list.
Michael Olson <address@hidden>**20060530045429] 
[emms-streams: Implement kill and yank.
Michael Olson <address@hidden>**20060530040114] 
[emms-streams: Make hitting RET on a URL do the right thing, improve cursor 
movement, and mark the buffer as unmodified after performing a save.
Michael Olson <address@hidden>**20060529030043] 
[emms-player-mpd: Make seek work correctly.
Michael Olson <address@hidden>**20060525033120] 
[emms-player-mpd: Use more robust method of detecting whether we need to 
force-feed MusicPD our playlist.
Michael Olson <address@hidden>**20060525014253] 
[emms-playlist-mode: Make "d" kill the entire line.  This seems to be a good 
compromise of those who use C-k and those who want more standard object-killing 
behavior.
foo**20060524200008] 
[emms-player-mpd: When showing the currently-playing song, prepend the name of 
the radio station, if it exists.
foo**20060524195911] 
[emms-player-mpd: Fix bug that caused unconditional reloading of the entire 
MusicPD playlist whenever the track was changed manually.
Michael Olson <address@hidden>**20060524061655] 
[emms-player-mpd: Overhaul for streamlist support, and fix a few miscellaneous 
issues.
Michael Olson <address@hidden>**20060524055707] 
[emms-player-mpd: Add a few checks to make sure that the given buffer exists 
before trying to do anything with it.
Michael Olson <address@hidden>**20060517035419] 
[emms-source-playlist: Do not expand names of files in playlists, as this can 
cause problems with emms-player-mpd in some configurations.
Michael Olson <address@hidden>**20060516081257] 
[emms-playlist-mode: Implement the option (disabled by default) of opening a 
new EMMS buffer for a playlist, when hitting RET on one.
Michael Olson <address@hidden>**20060510040730] 
[emms-playlist-mode.el: Don't put a period after the mode map. This hangs 21.4 
on display.
address@hidden 
[TAG 2.0
address@hidden 
Patch bundle hash:
33f327789f05884e376bd2ba27e2bcfb3abf4acb