This patch adds a function guess_all_extensions to 
mimetypes.py. This function returns all known 
extensions for a given type, not just the first one 
found in the types_map dictionary. guess_extension is 
still present and returns the first from the list.

Logged In: YES 
user_id=21627

What is the role of add_type in this patch?

Logged In: YES 
user_id=89016

The patch adds an inverted mapping (i.e. mapping from type
to a list of extensions). add_type simplifies adding a
type<->ext mapping to both dictionaries. If this method
should not be exposed we could make the name private.
(_add_type)

Logged In: YES 
user_id=21627

I can't see the point of making it private, since it is not
used inside the module. If you plan to use it, that usage
certainly is outside of the module, so the method would be
public.

If it is public, it needs to be exposed on the module level,
and it needs to be documented.

Logged In: YES 
user_id=89016

It *is* used in two spots: The constructor and the readfp 
method. But exposing it at the module level could make 
sense, because it is the atomic method of adding mime type 
information. So should it change the patch to expose it at the 
module level and change the LaTeX documentation 
accordingly?

Logged In: YES 
user_id=21627

I'm in favour of exposing it on the module level. If you are
uncertain, you might want to ask on python-dev.

Logged In: YES 
user_id=89016

OK, I'll change the patch and post the question to python-dev 
next week (I'm on vacation right now).

Logged In: YES 
user_id=89016

diff2.txt adds the global version of add_type and the
documentation in Doc/lib/libmimetypes.tex.

Logged In: YES 
user_id=12800

If add_type() is going to be public, shouldn't it have a
"strict" flag to decide whether to add it to the standard
types dict or the common types dict?

Logged In: YES 
user_id=89016

diff3.txt adds a strict=True to add_type and to all methods
that call add_type (i.e. read() and readfp()). types_map and
common_types are combined into a dict tuple (in the class,
on the module level they are still two dicts, to be
backwards compatible.)

What about adding a guess_all_types(), that returns a list
of all registered mimetypes for an exception? This way we
would be able to handle duplicates.

Logged In: YES 
user_id=21627

Looks good, please apply it. Don't forget to edit Misc/NEWS.

Logged In: YES 
user_id=89016

Checked in as:
Doc/lib/libmimetypes.tex 1.11
Lib/mimetypes.py 1.23
Misc/NEWS 1.481