From: Xavi Artigas <xavierartigas@yahoo.es>
Date: Thu, 1 Aug 2019 11:27:51 +0000 (+0200)
Subject: eolian_mono: Fix whitespace in generated cs files
X-Git-Tag: submit/tizen/20190805.083058~47
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=250b60148c4cd134fdea47d2d2fe9724d6e11a3a;p=platform%2Fupstream%2Fefl.git

eolian_mono: Fix whitespace in generated cs files

Pet peeve of mine.
---

diff --git a/src/bin/eolian_mono/eolian/mono/events.hh b/src/bin/eolian_mono/eolian/mono/events.hh
index ef4f5f4..fc623d0 100644
--- a/src/bin/eolian_mono/eolian/mono/events.hh
+++ b/src/bin/eolian_mono/eolian/mono/events.hh
@@ -247,12 +247,12 @@ struct event_argument_wrapper_generator
 
       std::string evt_name = name_helpers::managed_event_name(evt.name);
 
-      return as_generator("///<summary>Event argument wrapper for event <see cref=\""
+      return as_generator("/// <summary>Event argument wrapper for event <see cref=\""
                           << join_namespaces(evt.klass.namespaces, '.', managed_namespace)
                           << klass_interface_name(evt.klass) << "." << evt_name << "\"/>.</summary>\n"
                           << "[Efl.Eo.BindingEntity]\n"
                           << "public class " << name_helpers::managed_event_args_short_name(evt) << " : EventArgs {\n"
-                          << scope_tab << "///<summary>Actual event payload.</summary>\n"
+                          << scope_tab << "/// <summary>Actual event payload.</summary>\n"
                           << scope_tab << "public " << type << " arg { get; set; }\n"
                           << "}\n"
                  ).generate(sink, *etype, context);
@@ -404,7 +404,7 @@ struct event_definition_generator
       auto library_name = context_find_tag<library_context>(context).actual_library_name(klass.filename);
       std::string upper_c_name = utils::to_uppercase(evt.c_name);
       if (!as_generator(
-            scope_tab << "///<summary>Method to raise event "<< event_name << ".</summary>\n"
+            scope_tab << "/// <summary>Method to raise event "<< event_name << ".</summary>\n"
             << scope_tab << "public void On" << event_name << "(" << event_args_type << " e)\n"
             << scope_tab << "{\n"
             << scope_tab << scope_tab << "var key = \"_" << upper_c_name << "\";\n"
diff --git a/src/bin/eolian_mono/eolian/mono/klass.hh b/src/bin/eolian_mono/eolian/mono/klass.hh
index 46215b9..80d4deb 100644
--- a/src/bin/eolian_mono/eolian/mono/klass.hh
+++ b/src/bin/eolian_mono/eolian/mono/klass.hh
@@ -459,7 +459,7 @@ struct klass
      auto inherit_name = name_helpers::klass_concrete_name(cls);
 
      if(!as_generator(
-                scope_tab << "///<summary>Pointer to the native class description.</summary>\n"
+                scope_tab << "/// <summary>Pointer to the native class description.</summary>\n"
                 << scope_tab << "public override System.IntPtr NativeClass\n"
                 << scope_tab << "{\n"
                 << scope_tab << scope_tab << "get\n"
diff --git a/src/bin/eolian_mono/eolian/mono/struct_definition.hh b/src/bin/eolian_mono/eolian/mono/struct_definition.hh
index c949783..d358dc7 100644
--- a/src/bin/eolian_mono/eolian/mono/struct_definition.hh
+++ b/src/bin/eolian_mono/eolian/mono/struct_definition.hh
@@ -274,7 +274,7 @@ struct struct_internal_definition_generator
      if (!as_generator
          (
           indent << "#pragma warning disable CS1591\n\n"
-          << indent << "///<summary>Internal wrapper for struct " << string << ".</summary>\n"
+          << indent << "/// <summary>Internal wrapper for struct " << string << ".</summary>\n"
           << indent << "[StructLayout(LayoutKind.Sequential)]\n"
           << indent << "public struct " << string << "\n"
           << indent << "{\n"
@@ -295,7 +295,7 @@ struct struct_internal_definition_generator
                               || regular->base_type == "stringshare"
                               || regular->base_type == "any_value_ptr")))
             {
-               if (!as_generator(indent << scope_tab << "///<summary>Internal wrapper for field " << field_name << "</summary>\n"
+               if (!as_generator(indent << scope_tab << "/// <summary>Internal wrapper for field " << field_name << "</summary>\n"
                                  << indent << scope_tab << "public System.IntPtr " << field_name << ";\n")
                    .generate(sink, nullptr, context))
                  return false;
@@ -303,7 +303,7 @@ struct struct_internal_definition_generator
           else if (regular && !(regular->base_qualifier & efl::eolian::grammar::attributes::qualifier_info::is_ref)
                    && regular->base_type == "bool")
             {
-               if (!as_generator(indent << scope_tab << "///<summary>Internal wrapper for field " << field_name << "</summary>\n"
+               if (!as_generator(indent << scope_tab << "/// <summary>Internal wrapper for field " << field_name << "</summary>\n"
                                  << indent << scope_tab << "public System.Byte " << field_name << ";\n")
                    .generate(sink, nullptr, context))
                  return false;
@@ -311,7 +311,7 @@ struct struct_internal_definition_generator
           else if (regular && !(regular->base_qualifier & efl::eolian::grammar::attributes::qualifier_info::is_ref)
                    && regular->base_type == "char")
             {
-               if (!as_generator(indent << scope_tab << "///<summary>Internal wrapper for field " << field_name << "</summary>\n"
+               if (!as_generator(indent << scope_tab << "/// <summary>Internal wrapper for field " << field_name << "</summary>\n"
                                  << indent << scope_tab << "public System.Byte " << field_name << ";\n")
                    .generate(sink, nullptr, context))
                  return false;
@@ -337,7 +337,7 @@ struct struct_internal_definition_generator
 
      // to internal
      if (!as_generator(
-           indent << scope_tab << "///<summary>Implicit conversion to the internal/marshalling representation.</summary>\n"
+           indent << scope_tab << "/// <summary>Implicit conversion to the internal/marshalling representation.</summary>\n"
            << indent << scope_tab << "public static implicit operator " << string << "(" << string << " _external_struct)\n"
            << indent << scope_tab << "{\n"
            << indent << scope_tab << scope_tab << "var _internal_struct = new " << string << "();\n"
@@ -356,7 +356,7 @@ struct struct_internal_definition_generator
 
      // to managed
      if (!as_generator(
-           indent << scope_tab << "///<summary>Implicit conversion to the managed representation.</summary>\n"
+           indent << scope_tab << "/// <summary>Implicit conversion to the managed representation.</summary>\n"
            << indent << scope_tab << "public static implicit operator " << string << "(" << string << " _internal_struct)\n"
            << indent << scope_tab << "{\n"
            << indent << scope_tab << scope_tab << "var _external_struct = new " << string << "();\n"
@@ -425,7 +425,7 @@ struct struct_definition_generator
      // those 'mini-amd64.c condition fields not met' crashes.
      if (struct_.fields.size() == 0)
        {
-           if (!as_generator(indent << scope_tab << "///<summary>Placeholder field</summary>\n"
+           if (!as_generator(indent << scope_tab << "/// <summary>Placeholder field</summary>\n"
                              << indent << scope_tab << "public IntPtr field;\n").generate(sink, nullptr, context))
              return false;
        }
@@ -433,7 +433,7 @@ struct struct_definition_generator
        {
           // Constructor with default parameters for easy struct initialization
           if(!as_generator(
-                      indent << scope_tab << "///<summary>Constructor for " << string << ".</summary>\n"
+                      indent << scope_tab << "/// <summary>Constructor for " << string << ".</summary>\n"
                       << indent << scope_tab << "public " << string << "(\n"
                       << ((indent << scope_tab << scope_tab << field_argument_default) % ",\n")
                       << indent << scope_tab << ")\n"
@@ -445,8 +445,8 @@ struct struct_definition_generator
        }
 
      if(!as_generator(
-            indent << scope_tab << "///<summary>Implicit conversion to the managed representation from a native pointer.</summary>\n"
-            << indent << scope_tab << "///<param name=\"ptr\">Native pointer to be converted.</param>\n"
+            indent << scope_tab << "/// <summary>Implicit conversion to the managed representation from a native pointer.</summary>\n"
+            << indent << scope_tab << "/// <param name=\"ptr\">Native pointer to be converted.</param>\n"
             << indent << scope_tab << "public static implicit operator " << struct_name << "(IntPtr ptr)\n"
             << indent << scope_tab << "{\n"
             << indent << scope_tab << scope_tab << "var tmp = (" << struct_name << ".NativeStruct)Marshal.PtrToStructure(ptr, typeof(" << struct_name << ".NativeStruct));\n"