More multi-version support

ppapi/generators/test_cgen_range/versions.h

Index: ppapi/generators/test_cgen_range/versions.h
===================================================================
--- ppapi/generators/test_cgen_range/versions.h	(revision 0)
+++ ppapi/generators/test_cgen_range/versions.h	(revision 0)
@@ -0,0 +1,42 @@
+/* Copyright (c) 2011 The Chromium Authors. All rights reserved.
+ * Use of this source code is governed by a BSD-style license that can be
+ * found in the LICENSE file.
+ */
+
+/* From test_cgen_range/versions.idl modified Wed Aug 24 10:35:01 2011. */
+
+#ifndef PPAPI_C_TEST_CGEN_RANGE_VERSIONS_H_
+#define PPAPI_C_TEST_CGEN_RANGE_VERSIONS_H_
+
+#include "ppapi/c/pp_macros.h"
+
+/**
+ * @file
+ * File Comment. */

[dmichael (off chromium), 2011/08/24 21:15:29]

Looks like a problem in generating that comment...  shouldn't the */ be on the
next line?

[noelallen1, 2011/08/24 22:43:06]

This is correct based on the input IDL, but I can convert both to make it look
more like our common use case.

+
+
+/**
+ * @addtogroup Structs
+ * @{
+ */
+/* Bogus Struct */

[dmichael (off chromium), 2011/08/24 21:15:29]

Should the comment become a doxygen-style comment? Or did it have to be one in
the idl file for that to happen?

[noelallen1, 2011/08/24 22:43:06]

Yes.  This is correct based on the input IDL.  The input IDL needs to be
formated for Doxygen, if you want to generate it.

+struct PP_Size {
+  /**
+   * Comment for function
+   */
+  int32_t (*Foo)(int32_t x);
+  /**
+   * Comment for function
+   */
+  int32_t (*Foo)(int32_t x, int32_t y);
+  /**
+   * Comment for function
+   */
+  int32_t (*Foo)(int32_t x, int32_t y, int32_t z);

[dmichael (off chromium), 2011/08/24 21:15:29]

Maybe you can throw a comment somewhere (wherever makes sense) that this is a
work in progress...  as you know, the above code isn't right for Chrome or the
public-facing headers yet. I don't want to ask you to fix all that in this CL,
but it would be good to have a quick comment somewhere to that effect.

[noelallen1, 2011/08/24 22:43:06]

The place that makes sense is the CL itself I think.

+};
+/**
+ * @}
+ */
+
+#endif  /* PPAPI_C_TEST_CGEN_RANGE_VERSIONS_H_ */
+
Property changes on: ppapi/generators/test_cgen_range/versions.h
___________________________________________________________________
Added: svn:eol-style
   + LF