VM

vm-mime-charset-completion-alist needs updating

Bug #878471 reported by Uday Reddy
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
VM
Fix Released
Medium
Julian Bradfield

Bug Description

When a plain text file is attached, VM asks for the charset to be used for the file. The menu of choices, coming from the variable vm-mime-charset-completion-alist, seems old. It doesn't contain UTF, for example.

Where do we get an up to date list from?

Related branches

Uday Reddy (reddyuday)
Changed in vm:
status: Triaged → Confirmed
Uday Reddy (reddyuday)
Changed in vm:
milestone: none → 8.2.0
Revision history for this message
Uday Reddy (reddyuday) wrote :

Dear Julian, do you think you can take a look at this bug?

Changed in vm:
assignee: nobody → Julian Bradfield (jcb+vm)
Uday Reddy (reddyuday)
Changed in vm:
milestone: 8.2.0 → 8.2.1
Revision history for this message
Julian Bradfield (jcb+vm) wrote : [Bug 878471] Re: vm-mime-charset-completion-alist needs updating

> vm-mime-charset-completion-alist needs updating
>
>Status in VM (View Mail) for Emacs:
> Confirmed
>
>Bug description:
> When a plain text file is attached, VM asks for the charset to be used
> for the file. The menu of choices, coming from the variable vm-mime-
> charset-completion-alist, seems old. It doesn't contain UTF, for
> example.
>
> Where do we get an up to date list from?

Why give it as an explicit list? Why not compute it from one of the
other alists that have just been listed/computed. E.g

(defconst vm-mime-charset-completion-alist
  (mapcar 'car vm-mime-mule-charset-to-coding-alist))

For FSFmacs, those previously computed values ought to be enough, I
think. It's more complex for XEmacs, but how many of us are there?

Revision history for this message
Julian Bradfield (jcb+vm) wrote :

>(defconst vm-mime-charset-completion-alist
> (mapcar 'car vm-mime-mule-charset-to-coding-alist))

Sorry, that should have read

(defconst vm-mime-charset-completion-alist
  (mapcar (lambda (a) (list (car a)))
    vm-mime-mule-charset-to-coding-alist))

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

Thanks Julian. That did the trick!

Changed in vm:
milestone: 8.2.1 → 8.2.0
status: Confirmed → Fix Committed
Uday Reddy (reddyuday)
Changed in vm:
status: Fix Committed → Fix Released
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.