From 50f4cc4317d0237cd83cc39a1332a8e28cbe86db Mon Sep 17 00:00:00 2001
From: Ingo Weinhold <ingo_weinhold@gmx.de>
Date: Mon, 16 Jan 2006 23:47:38 +0000
Subject: [PATCH] Added TODO. Just in case someone wants to tackle it. :-)

git-svn-id: file:///srv/svn/repos/haiku/haiku/trunk@15990 a95241bf-73f2-0310-859d-f6bbb57e9c96
---
 src/kits/storage/mime/UpdateMimeInfoThread.cpp | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/src/kits/storage/mime/UpdateMimeInfoThread.cpp b/src/kits/storage/mime/UpdateMimeInfoThread.cpp
index 9d282e3b68..1e5bc13605 100644
--- a/src/kits/storage/mime/UpdateMimeInfoThread.cpp
+++ b/src/kits/storage/mime/UpdateMimeInfoThread.cpp
@@ -35,6 +35,14 @@ UpdateMimeInfoThread::UpdateMimeInfoThread(const char *name, int32 priority,
 status_t
 UpdateMimeInfoThread::DoMimeUpdate(const entry_ref *entry, bool *entryIsDir)
 {
+// TODO: This implementation is incomplete.
+// For application executables we also need to copy several things from the
+// resources into attributes (basically that is supported by BAppFileInfo,
+// which we can use here, BTW).
+// Furthermore fForce shouldn't be a boolean. The do_mime_update() API
+// functions takes an "int force", which can have three valid values, 0, 1,
+// and 2. 0 is no-force, 1 and 2 correspond to the mimeset flags -f and -F,
+// respectively.
 	status_t err = entry ? B_OK : B_BAD_VALUE;
 	bool doUpdate = true;
 	BNode node;