Refine the comments

Author: jserv

extent.c

@@ -3,17 +3,18 @@
 
 #include "simplefs.h"
 
-/* Search for the extent containing the target block.
+/* Search for the extent containing the target block. Binary search is used
+ * for efficiency.
+ *
  * Returns the first unused file index if not found.
  * Returns -1 if the target block is out of range.
- * Binary search is used for efficiency.
  */
 uint32_t simplefs_ext_search(struct simplefs_file_ei_block *index,
                              uint32_t iblock)
 {
-    /* first, find the first unused file index with binary search.
-     * It'll be our right boundary for actual binary search
-     * and return value when file index is not found.
+    /* First, find the first unused file index with binary search.
+     * It will be our right boundary for actual binary search and the return
+     * value when the file index is not found.
      */
     uint32_t start = 0;
     uint32_t end = SIMPLEFS_MAX_EXTENTS - 1;
@@ -62,13 +63,13 @@ uint32_t simplefs_ext_search(struct simplefs_file_ei_block *index,
 
     /* return 'end' if it directs to valid block
      * return 'boundary' if index is not found
-     * and eiblock has remaining space */
+     * and eiblock has remaining space
+     */
     end_block = index->extents[end].ee_block;
     end_len = index->extents[end].ee_len;
-    if (iblock >= end_block && iblock < end_len) {
+    if (iblock >= end_block && iblock < end_len)
         return end;
-    } else if (boundary < SIMPLEFS_MAX_EXTENTS) {
+    if (boundary < SIMPLEFS_MAX_EXTENTS)
         return boundary;
-    }
     return boundary;
-}
+}

inode.c

@@ -96,7 +96,7 @@ struct inode *simplefs_iget(struct super_block *sb, unsigned long ino)
     return ERR_PTR(ret);
 }
 
-/* Searches for a dentry in dir.
+/* Search for a dentry in dir.
  * Fills dentry with NULL if not found in dir, or with the corresponding inode
  * if found.
  * Returns NULL on success, indicating the dentry was successfully filled or

mkfs.c

@@ -155,7 +155,9 @@ static int write_ifree_blocks(int fd, struct superblock *sb)
         goto end;
     }
 
-    /* All ifree blocks except the one containing 2 first inodes */
+    /* All free blocks in the inode bitmap except the one containing the first
+     * two inodes.
+     */
     ifree[0] = 0xffffffffffffffff;
     uint32_t i;
     for (i = 1; i < le32toh(sb->info.nr_ifree_blocks); i++) {