On 05/21/2014 09:27 AM, Mark Wielaard wrote:
+/* The C++ version of the enum_underlying_base_type langhook.
+   See also cp/semantics.c (finish_underlying_type).  */
+static tree cxx_enum_underlying_base_type (const_tree type)

We usually leave a blank line between the comment and the function. Also, the function name should be at the beginning of the line for etags.

+/* Default implementation of enum_underlying_base_type using type_for_size.  */
+tree
+lhd_enum_underlying_base_type (const_tree enum_type)

Blank line here, too.

   tree (*reconstruct_complex_type) (tree, tree);
+
+  tree (*enum_underlying_base_type) (const_tree);

And please add a comment documenting the hook.

OK with those changes.

Jason

Reply via email to