Add TODO comment.

2008-06-10  Tor Lillqvist  <tml@novell.com>

	* gcontenttype.c (g_content_type_can_be_executable) [Win32]: Add
	TODO comment.

svn path=/trunk/; revision=6979

diff --git a/gio/ChangeLog b/gio/ChangeLog
index ef6f8f2..7b64725 100644 (file)
--- a/gio/ChangeLog
+++ b/gio/ChangeLog
@@ -1,3 +1,8 @@
+2008-06-10  Tor Lillqvist  <tml@novell.com>
+
+       * gcontenttype.c (g_content_type_can_be_executable) [Win32]: Add
+       TODO comment.
+
 2008-06-09  Matthias Clasen  <mclasen@redhat.com>
 
        * xdgmime/Makefile.am: Fix the build

diff --git a/gio/gcontenttype.c b/gio/gcontenttype.c
index fcc7fa2..c5bb618 100644 (file)
--- a/gio/gcontenttype.c
+++ b/gio/gcontenttype.c
@@ -203,6 +203,18 @@ g_content_type_can_be_executable (const char *type)
       strcmp (type, ".com") == 0 ||
       strcmp (type, ".bat") == 0)
     return TRUE;
+
+  /* TODO: Also look at PATHEXT, which lists the extensions for
+   * "scripts" in addition to those for true binary executables.
+   *
+   * (PATHEXT=.COM;.EXE;.BAT;.CMD;.VBS;.VBE;.JS;.JSE;.WSF;.WSH for me
+   * right now, for instance). And in a sense, all associated file
+   * types are "executable" on Windows... You can just type foo.jpg as
+   * a command name in cmd.exe, and it will run the application
+   * associated with .jpg. Hard to say what this API actually means
+   * with "executable".
+   */
+
   return FALSE;
 }