[gvfs/wip/udisks2] Reword a couple of comments



commit aacccf64f7042143a7ec1a4baad03d5dbddca174
Author: David Zeuthen <davidz redhat com>
Date:   Fri Jan 20 13:21:44 2012 -0500

    Reword a couple of comments
    
    Signed-off-by: David Zeuthen <davidz redhat com>

 monitor/udisks2/gvfsudisks2volume.c        |    4 ++--
 monitor/udisks2/gvfsudisks2volumemonitor.c |   16 ++++++++--------
 2 files changed, 10 insertions(+), 10 deletions(-)
---
diff --git a/monitor/udisks2/gvfsudisks2volume.c b/monitor/udisks2/gvfsudisks2volume.c
index 00c47e1..cffda96 100644
--- a/monitor/udisks2/gvfsudisks2volume.c
+++ b/monitor/udisks2/gvfsudisks2volume.c
@@ -780,10 +780,10 @@ mount_command_cb (GObject       *source_object,
   gint exit_status;
   gchar *standard_error = NULL;
 
-  /* TODO: for e.g. NFS and CIFS mounts we could do GMountOperation stuff and pipe a
+  /* NOTE: for e.g. NFS and CIFS mounts we could do GMountOperation stuff and pipe a
    * password to mount(8)'s stdin channel
    *
-   * TODO: if this fails because the user is not authorized (e.g. EPERM), we could
+   * NOTE: if this fails because the user is not authorized (e.g. EPERM), we could
    * run it through a polkit-ified setuid root helper
    */
 
diff --git a/monitor/udisks2/gvfsudisks2volumemonitor.c b/monitor/udisks2/gvfsudisks2volumemonitor.c
index 92abc48..434f2e0 100644
--- a/monitor/udisks2/gvfsudisks2volumemonitor.c
+++ b/monitor/udisks2/gvfsudisks2volumemonitor.c
@@ -870,15 +870,15 @@ static gboolean
 should_include_drive (GVfsUDisks2VolumeMonitor *monitor,
                       UDisksDrive              *drive)
 {
-  /* NOTE: For now, we just include all detected drives. This is probably
-   * wrong - non-removable drives without anything visible (such
-   * RAID components) should probably not be shown.
+  /* NOTE: For now, we just include all detected drives. This is
+   * probably wrong - non-removable drives without anything visible
+   * (such RAID components) should probably not be shown. Then again,
+   * the GNOME 3 user interface doesn't really show GDrive instances
+   * except for in the computer:/// location in Nautilus.
    *
-   * Then again, the GNOME 3 user interface doesn't really show GDrive
-   * instances except for in the computer:/// location in Nautilus.
-   *
-   * Idea: if device is non-removable, maybe only show it, if it has
-   * more visible devices (this is the gdu volume monitor behavior)
+   * Therefore, if device is non-removable, maybe only show it, if it
+   * has more visible devices... this is the gdu volume monitor
+   * behavior.
    */
   return TRUE;
 }



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]