Re: Drag and drop patch for X, please review.

[Jan D.]

> I'd suggest:
>
>   (1) Make the alist entry be one of (DND-TYPE DROP-FUN TEST-FUN) or
>       (DND-TYPE . DROP-FUN) with TEST-FUN being a test function that 
> accepts
>       the same args as DROP-FUN.  Putting the TEST-FUN last makes it 
> easier
>       to omit.
>
>   (2) If TEST-FUN is omitted/nil, default to some common formula like 
> `in a
>       writable text area'
>
>   (3) Export the default test-function as a real function, so the user 
> can
>       call it from his code too (or use it explicitly in the alist, 
> though of
>       course that would be redundant).

I see just one problem with this approach.  The DND-TYPE is not unique.
That is, when I drop a file from say the Gnome file manager, I have
several types to choose from, for example, text/uri-list, text/plain,
UTF8_STRING, and some more.  If a user is going to be able to make
an intelligent choice, I'd assume he wants to look at all acceptable
types as a group.  Also, there are several types out there that a user
can add to the alist that the current suggestion just ignores (say
application/xml or just about any unknown type).  How do we determine
which of these are more specific?

Am I missing something here?

        Jan D.