All new file i/o documentation.

ppapi/c/ppb_file_ref.h

Index: ppapi/c/ppb_file_ref.h
===================================================================
--- ppapi/c/ppb_file_ref.h	(revision 91561)
+++ ppapi/c/ppb_file_ref.h	(working copy)
@@ -11,72 +11,179 @@
 #include "ppapi/c/pp_resource.h"
 #include "ppapi/c/pp_var.h"
 
+/**
+ * @file
+ * This file defines the API to create a file reference or "weak pointer" to a
+ * file in a file system.
+ */
+
 struct PP_CompletionCallback;
 
 #define PPB_FILEREF_INTERFACE_0_9 "PPB_FileRef;0.9"
 #define PPB_FILEREF_INTERFACE PPB_FILEREF_INTERFACE_0_9
 
-// A FileRef is a "weak pointer" to a file in a file system.  It contains a
-// PP_FileSystemType identifier and a file path string.
+/**
+ * @addtogroup Structs
+ * @{
+ */
+/**
+ * The <code>PPB_FileRef</code> struct represents a "weak pointer" to a file in
+ * a file system.  This struct contains a <code>PP_FileSystemType</code>
+ * identifier and a file path string.
+ */
 struct PPB_FileRef {
-  // Creates a weak pointer to a file in the given filesystem. File paths are
-  // POSIX style.  Returns 0 if the path is malformed.
+  /**
+   * Create() creates a weak pointer to a file in the given file system. File
+   * paths are POSIX style.
+   *
+   * @param[in] resource A <code>PP_Resource</code> corresponding to a file
+   * system.
+   * @param[in] path A path to the file.
+   *
+   * @return A <code>PP_Resource</code> corresponding to a file reference if
+   * successful or 0 if the path is malformed.
+   */
   PP_Resource (*Create)(PP_Resource file_system, const char* path);
-
-  // Returns PP_TRUE if the given resource is a FileRef. Returns PP_FALSE if the
-  // resource is invalid or some type other than a FileRef.
+  /**
+   * IsFileRef() determines if the provided resource is a file reference.
+   *
+   * @param[in] resource A <code>PP_Resource</code> corresponding to a file
+   * reference.
+   *
+   * @return <code>PP_TRUE</code> if the resource is a
+   * <code>PPB_FileRef</code>, <code>PP_FALSE</code> if the resource is
+   * invalid or some type other than <code>PPB_FileRef</code>.
+   */
   PP_Bool (*IsFileRef)(PP_Resource resource);
 
-  // Returns the file system identifier of this file, or
-  // PP_FILESYSTEMTYPE_INVALID if the file ref is invalid.
+  /**
+   * GetFileSystemType() returns the type of the file system.
+   *
+   * @param[in] file_ref A <code>PP_Resource</code> corresponding to a file
+   * reference.
+   *
+   * @return A <code>PP_FileSystemType</code> with the file system type if
+   * valid or <code>PP_FILESYSTEMTYPE_INVALID</code> if the provided resource
+   * is not a valid file reference.
+   */
   PP_FileSystemType (*GetFileSystemType)(PP_Resource file_ref);
 
-  // Returns the name of the file. The value returned by this function does not
-  // include any path component (such as the name of the parent directory, for
-  // example). It is just the name of the file. To get the full file path, use
-  // the GetPath() function.
+  /**
+   * GetName() returns the name of the file.
+   *
+   * @param[in] file_ref A <code>PP_Resource</code> corresponding to a file
+   * reference.
+   *
+   * @return A <code>PP_Var</code> containing the name of the file.  The value
+   * returned by this function does not include any path components (such as
+   * the name of the parent directory, for example). It is just the name of the
+   * file. Use GetPath() get the full file path.

[sanga, 2011/07/06 23:20:59]

"to get"

[jond, 2011/07/07 15:11:15]

Done.

+   */
   struct PP_Var (*GetName)(PP_Resource file_ref);
 
-  // Returns the absolute path of the file.  This method fails if the file
-  // system type is PP_FileSystemType_External.
+  /**
+   * GetPath() returns the absolute path of the file.
+   *
+   * @param[in] file_ref A <code>PP_Resource</code> corresponding to a file
+   * reference.
+   *
+   * @return A <code>PP_Var</code> containing the absolute path of the file.
+   * This function fails if the file system type is
+   * <code>PP_FileSystemType_External</code>.
+   */
   struct PP_Var (*GetPath)(PP_Resource file_ref);
 
-  // Returns the parent directory of this file.  If file_ref points to the root
-  // of the filesystem, then the root is returned.  This method fails if the
-  // file system type is PP_FileSystemType_External.
+  /**
+   * GetParent() returns the parent directory of this file.  If
+   * <code>file_ref</code> points to the root of the filesystem, then the root
+   * is returned.
+   *
+   * @param[in] file_ref A <code>PP_Resource</code> corresponding to a file
+   * reference.
+   *
+   * @return A <code>PP_Resource</code> containing the parent directory of the
+   * file. This function fails if the file system type is
+   * <code>PP_FileSystemType_External</code>.
+   */
   PP_Resource (*GetParent)(PP_Resource file_ref);
 
-  // Makes a new directory in the filesystem as well as any parent directories
-  // if the make_ancestors parameter is PP_TRUE.  It is not valid to make a
-  // directory in the external filesystem.  Fails if the directory already
-  // exists or if ancestor directories do not exist and make_ancestors was not
-  // passed as PP_TRUE.
+  /**
+   * MakeDirectory() makes a new directory in the file system as well as any
+   * parent directories if the <code>make_ancestors</code> argument is
+   * <code>PP_TRUE</code>.  It is not valid to make a directory in the external
+   * file system.
+   *
+   * @param[in] file_ref A <code>PP_Resource</code> corresponding to a file
+   * reference.
+   * @param[in] make_ancestors A <code>PP_Bool</code> set to
+   * <code>PP_TRUE</code> to make ancestor directories or <code>PP_FALSE</code>
+   * if ancestor directories are not needed.
+   *
+   * @return An int32_t containing an error code from <code>pp_errors.h</code>.
+   * Fails if the directory already exists or if ancestor directories do not
+   * exist and <code>make_ancestors</code> was not passed as
+   * <code>PP_TRUE</code>.
+   */
   int32_t (*MakeDirectory)(PP_Resource directory_ref,
                            PP_Bool make_ancestors,
                            struct PP_CompletionCallback callback);
 
-  // Updates timestamps for a file.  You must have write access to the file if
-  // it exists in the external filesystem.
+  /**
+   * Touch() Updates time stamps for a file.  You must have write access to the
+   * file if it exists in the external filesystem.
+   *
+   * @param[in] file_ref A <code>PP_Resource</code> corresponding to a file
+   * reference.
+   * @param[in] last_access_time The last time the file was accessed.
+   * @param[in] last_modified_time The last time the file was modified.
+   * @param[in] callback A <code>PP_CompletionCallback</code> to be called upon
+   * completion of Touch().
+   *
+   * @return An int32_t containing an error code from <code>pp_errors.h</code>.
+   */
   int32_t (*Touch)(PP_Resource file_ref,
                    PP_Time last_access_time,
                    PP_Time last_modified_time,
                    struct PP_CompletionCallback callback);
 
-  // Delete a file or directory.  If file_ref refers to a directory, then the
-  // directory must be empty.  It is an error to delete a file or directory
-  // that is in use.  It is not valid to delete a file in the external
-  // filesystem.
+  /**
+   * Delete() deletes a file or directory. If <code>file_ref</code> refers to
+   * a directory, then the directory must be empty. It is an error to delete a
+   * file or directory that is in use.  It is not valid to delete a file in
+   * the external file system.
+   *
+   * @param[in] file_ref A <code>PP_Resource</code> corresponding to a file
+   * reference.
+   * @param[in] callback A <code>PP_CompletionCallback</code> to be called upon
+   * completion of Delete().
+   *
+   * @return An int32_t containing an error code from <code>pp_errors.h</code>.
+   */
   int32_t (*Delete)(PP_Resource file_ref,
                     struct PP_CompletionCallback callback);
 
-  // Rename a file or directory.  file_ref and new_file_ref must both refer to
-  // files in the same filesystem.  It is an error to rename a file or
-  // directory that is in use.  It is not valid to rename a file in the
-  // external filesystem.
+  /**
+   * Rename() renames a file or directory.  Arguments <code>file_ref</code> and
+   * <code>new_file_ref</code> must both refer to files in the same file
+   * system. It is an error to rename a file or directory that is in use.  It
+   * is not valid to rename a file in the external file system.
+   *
+   * @param[in] file_ref A <code>PP_Resource</code> corresponding to a file
+   * reference.
+   * @param[in] new_file_ref A <code>PP_Resource</code> corresponding to a new
+   * file reference.
+   * @param[in] callback A <code>PP_CompletionCallback</code> to be called upon
+   * completion of Rename().
+   *
+   * @return An int32_t containing an error code from <code>pp_errors.h</code>.
+   */
   int32_t (*Rename)(PP_Resource file_ref,
                     PP_Resource new_file_ref,
                     struct PP_CompletionCallback callback);
 };
+/**
+ * @}
+ */
 
 #endif  /* PPAPI_C_PPB_FILE_REF_H_ */