vm team mailing list archive

[Richard Sharman <richard_sharman@xxxxxxxxx>]

Uday Reddy writes:
| Richard, thanks for reporting this.  I believe the problem exists with
| the current version of VM as well.  So this report is valuable.
| However, the specific line that you identified as the cause isn't in the
| current version.   
| So, we will have to hunt it down.  There are a few
| other occurrences of set-buffer-multibyte in vm-mime.el.  We will have
| to check all of them, I suppose.
| 
| The test.jpg file that you attached is presumably the image that was
| attached in the message?
| 

Yes.  It happened to me on several real [and much larger] files so I
thought I'd try a simple file as a test case - and sure enough it did
fail - and so I attached it in case it was useful.  (I was using
electric-eye to display the image and found that a corrupt .png file was
displayed but looked bizarre whearas a bad .gif file typically could not
be displayed at all.)

I looked at the output produced and it started off right then some bytes
were swapped e.g. c0 98 would become 98 c0 and the length would be
wrong;  usually shorter I think.   The buffer seemed to be correct
before the call to set-buffer-multibyte but incorrect after it so I
removed the line and it worked for me but I was really poking in the
dark!

Richard

-- 
"Display using External Viewer" produces corrupt file under emacs 23
https://bugs.launchpad.net/bugs/495089
You received this bug notification because you are a member of VM
development team, which is the registrant for VM.

Status in VM (View Mail) for Emacs: In Progress

Bug description:
The "Display using External Viewer" menu option isn't working under
Emacs-23.  The file produced is corrupted and typically is slightly
smaller than it should be.  This is with vm 7.19.   With emacs 22.1
this problem doesn't happen.

The "Save to File" does work, however.

I found that by removing the call to set-buffer-multibyte in
vm-mime-display-external-generic the problem went away.

% diff -c vm-mime.el.orig vm-mime.el
*** vm-mime.el.orig	Tue Dec  1 11:07:31 2009
--- vm-mime.el	Thu Dec 10 10:12:31 2009
***************
*** 1967,1973 ****
  			    (vm-mime-transfer-decode-region layout
  							    (point-min)
  							    (point-max))
! 			    (set-buffer-multibyte t)
  			    (set-buffer target)
  			    (setq start (point))
  			    (insert-buffer-substring work-buffer)
--- 1967,1973 ----
  			    (vm-mime-transfer-decode-region layout
  							    (point-min)
  							    (point-max))
! 			    ;; (set-buffer-multibyte t)
  			    (set-buffer target)
  			    (setq start (point))
  			    (insert-buffer-substring work-buffer)
% 


Examples of using the "Save to File"  
-rw-------  1 sharman gx5000 3310 Dec 10 10:13 test.gif
-rw-------  1 sharman gx5000 3424 Dec 10 10:13 test.jpeg
sharmanpc 19% 

whearas when using "Display using External Viewer" the temporary file
was a different size
-rw-------  1 sharman gx5000 3302 Dec 10 10:12 test.gif
-rw-------  1 sharman gx5000 3420 Dec 10 10:12 test.jpeg