Code review comment for lp:~mhr3/zeitgeist/mimetypes

Revision history for this message
Mikkel Kamstrup Erlandsen (kamstrup) wrote :

While we are at improving our mime handling situation we might as well go the extra mile - especially now that we want to make it public API.

We now have interpretations like Interpretation.SPREADSHEET/PAGINATED_TEXT_DOCUMENT so we can do a much finer grained mapping of the mimetypes. This in turn works out beautifully with the queries because we now expand queries to also match all child types.

As an example take a look at my mimetype mappings libzeitgeist: http://bazaar.launchpad.net/~libzeitgeist-developers/libzeitgeist/trunk/annotate/head:/src/zeitgeist-mimetypes.c#L239

Anoter note: The public API should be added to our generated sphinx docs too...

review: Needs Information (visual review)

« Back to merge proposal