VM

wrong-type argument in vm-fill-long-lines-in-reply

Bug #912014 reported by Uday Reddy
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
VM
Fix Committed
Low
Uday Reddy

Bug Description

Matthew Lovell reports (viewmail-bugs, 2012-01-04):

I've setup vm-reply-hook to fill paragraphs in replies prior to my
editing them. This configuration is copied from my previous vm
setup, under xemacs.

(add-hook 'vm-reply-hook 'vm-fill-long-lines-in-reply)

I can also get the identical error to occur by interactively calling
vm-fill-long-lines-in-reply in the composition frame of the reply.

Debugger entered--Lisp error: (wrong-type-argument number-or-marker-p nil)
  vm-forward-paragraph()
  ad-Orig-vm-fill-paragraphs-containing-long-lines(nil 435 679)
  vm-fill-paragraphs-containing-long-lines(nil 435 679)
  vm-fill-long-lines-in-reply()
  run-hooks(vm-reply-hook)
  vm-do-reply(nil t 1)
  vm-reply-include-text(1)
  call-interactively(vm-reply-include-text nil nil)

Related branches

Revision history for this message
Uday Reddy (reddyuday) wrote :

> I've setup vm-reply-hook to fill paragraphs in replies prior to my
> editing them. This configuration is copied from my previous vm
> setup, under xemacs.
>
> (add-hook 'vm-reply-hook 'vm-fill-long-lines-in-reply)

Please note that you don't need to do this. Once you have set the variable
I mentioned (vm-fill-paragraphs-containing-long-lines-in-reply) to a non-nil value, VM will invoke filling automatically.

> I can also get the identical error to occur by interactively calling
> vm-fill-long-lines-in-reply in the composition frame of the reply.

I think it is a good idea to allow the interactive use of this command. So,
I will add a check for the variable.

However, I think it is not a good idea to do automatic filling any more.
You can turn on auto-fill-mode in the message composition buffer. If there
are still long lines when you send, then VM will MIME-encode the message so
that it is legal and the email recipients can use their own viewers to do
filling or wrapping.

The problem with automatic filling is that it removes line breaks which may
be important, e.g., in tables or program code.

Revision history for this message
Uday Reddy (reddyuday) wrote :

Fix committed in rev. 1325.

Changed in vm:
status: Triaged → Fix Committed
Uday Reddy (reddyuday)
Changed in vm:
milestone: 8.2.0 → 8.2.90a
Uday Reddy (reddyuday)
Changed in vm:
milestone: 8.2.90a → 8.2.89a
Uday Reddy (reddyuday)
Changed in vm:
milestone: 8.2.89a → 8.2.0b1
no longer affects: vm/8.2.x
Uday Reddy (reddyuday)
Changed in vm:
milestone: 8.2.1a → 8.2.0
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.