Fix up some comments.
authorguy <guy@f5534014-38df-0310-8fa8-9805f1628bb7>
Wed, 8 Feb 2006 03:25:22 +0000 (03:25 +0000)
committerguy <guy@f5534014-38df-0310-8fa8-9805f1628bb7>
Wed, 8 Feb 2006 03:25:22 +0000 (03:25 +0000)
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@17214 f5534014-38df-0310-8fa8-9805f1628bb7

epan/filesystem.c

index 3acc140537e995e372bc18dc3e07b29f2cbe7eb8..9f4a34f17d78947bd4c20b4a56df826b79494a67 100644 (file)
@@ -791,20 +791,22 @@ gboolean
 files_identical(const char *fname1, const char *fname2)
 {
     /* Two different implementations, because:
-     * - _fullpath is not available on unix 
-     * - the stat inode will not work as expected on Win32,
-     *   so two different implementations.
      *
-     * XXX - will _fullpath work with UNC?
+     * - _fullpath is not available on UN*X, so we can't get full
+     *   paths and compare them (which wouldn't work with hard links
+     *   in any case);
+     *
+     * - st_ino isn't filled in with a meaningful value on Windows.
      */
 #ifdef _WIN32
     char full1[MAX_PATH], full2[MAX_PATH];
 
     /*
      * Get the absolute full paths of the file and compare them.
-     * That won't work if you have hard links, which aren't
+     * That won't work if you have hard links, but those aren't
      * much used on Windows, even though NTFS supports them.
-     * We can't use st_ino on Windows, as it's not supported on FAT.
+     *
+     * XXX - will _fullpath work with UNC?
      */
     if( _fullpath( full1, fname1, MAX_PATH ) == NULL ) {
         return FALSE;