Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(384)

Unified Diff: ppapi/c/pp_var.h

Issue 6297018: Removed Doxygen groupings by ppb_, pp_, and ppp_. Added grouping by construct... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 9 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ppapi/c/pp_var.h
===================================================================
--- ppapi/c/pp_var.h (revision 72639)
+++ ppapi/c/pp_var.h (working copy)
@@ -8,15 +8,17 @@
/**
* @file
* Defines the API ...
- *
- * @addtogroup PP
- * @{
*/
#include "ppapi/c/pp_bool.h"
#include "ppapi/c/pp_macros.h"
#include "ppapi/c/pp_stdint.h"
+/**
+ *
+ * @addtogroup Enums
+ * @{
+ */
typedef enum {
PP_VARTYPE_UNDEFINED,
PP_VARTYPE_NULL,
@@ -26,9 +28,18 @@
PP_VARTYPE_STRING,
PP_VARTYPE_OBJECT
} PP_VarType;
+/**
+ * @}
+ */
+
PP_COMPILE_ASSERT_SIZE_IN_BYTES(PP_VarType, 4);
/**
+ * @addtogroup Structs
+ * @{
+ */
+
+/**
* Do not rely on having a predictable and reproducible
* int/double differentiation.
* JavaScript has a "number" type for holding a number, and
@@ -39,6 +50,7 @@
*
* Your best bet is to have a wrapper for variables
* that always gets out the type you expect, converting as necessary.
+ *
*/
struct PP_Var {
PP_VarType type;
@@ -62,8 +74,16 @@
int64_t as_id;
} value;
};
+/**
+ * @}
+ */
+
PP_COMPILE_ASSERT_STRUCT_SIZE_IN_BYTES(PP_Var, 16);
+/**
+ * @addtogroup Functions
+ * @{
+ */
PP_INLINE struct PP_Var PP_MakeUndefined() {
struct PP_Var result = { PP_VARTYPE_UNDEFINED, 0, {PP_FALSE} };
return result;
@@ -91,10 +111,9 @@
result.value.as_double = value;
return result;
}
-
/**
* @}
- * End addtogroup PP
*/
+
#endif /* PPAPI_C_PP_VAR_H_ */

Powered by Google App Engine
This is Rietveld 408576698