Re: TODO [was Re: Looking for a new Emacs maintainer or team]

[Dan Nicolaescu <dann@ics.uci.edu>]

"V.Rao" <mail.vjrao@yahoo.com> writes:

  > Please let me know if anyone is already working on one of these tasks.
  > 
  > ** make emacsclient accept -nw as a synonym to -t.
  > 
  > ** Replace some uses of the preprocessor code in Makefile.in with the
  > equivalent autoconf.
  > 
  > ** Make "emacs --daemon" start emacs without showing any frame.
  > Use emacsclient later to open frames.

AFAIK nobody has publicly announced that is working on any of these.

Do you have a copyright assignment on file?  If not, it would make sense
to get started on that as soon as possible, so that it is ready by the
time you finish writing the code.


  > ** Make vc-checkin avoid reverting the buffer if has not changed after
  >    the checkin. Comparing (md5 BUFFER) to (md5 FILE) should be enough.

I have a patch for this one.

But it needs a few more eyes on it from people that know the code
involved very well to make sure it is correct and the right thing to do.
The patch itself is not too complicated.

--- vc.el       10 Oct 2007 10:34:53 -0700      1.464
+++ vc.el       11 Oct 2007 12:18:02 -0700      
@@ -1328,7 +1327,19 @@
     (save-excursion
       ;; t means don't call normal-mode;
       ;; that's to preserve various minor modes.
-      (revert-buffer arg no-confirm t))
+      (if (string=
+          (with-temp-buffer
+            ;; Insert the file on disk in a temporary buffer and compute the md5 there.
+            (let ((coding-system-for-read 'binary))
+              (insert-file-contents file)
+              (md5 (current-buffer))))
+          (md5 (current-buffer)))  ;; md5 for the current buffer
+         (let ((writable (file-writable-p (buffer-file-name)))) ;; Try to set the read-only state.
+           (unless (eq buffer-read-only writable)
+             (setq buffer-read-only writable))
+           (message "not reverting"))
+       (message "reverting :-(")
+       (revert-buffer arg no-confirm t)))
     (vc-restore-buffer-context context)))
 
 (defun vc-buffer-sync (&optional not-urgent)