csharp: Rename register_event_proxies.
authorLauro Moura <lauromoura@expertisesolutions.com.br>
Sun, 17 Mar 2019 15:09:20 +0000 (16:09 +0100)
committerYeongjong Lee <yj34.lee@samsung.com>
Tue, 2 Apr 2019 03:45:17 +0000 (12:45 +0900)
Summary:
To better adhere to C# conventions and state it is for generated code
only.

Ref T7702

Reviewers: vitor.sousa, segfaultxavi

Reviewed By: segfaultxavi

Subscribers: cedric, #reviewers, #committers

Tags: #efl

Maniphest Tasks: T7702

Differential Revision: https://phab.enlightenment.org/D8369

src/bin/eolian_mono/eolian/mono/events.hh
src/bin/eolian_mono/eolian/mono/klass.hh

index d8dc043..7289386 100644 (file)
@@ -262,7 +262,7 @@ struct event_definition_generator
       if (!generate_event_trigger(sink, wrapper_evt_name, wrapper_args_type, wrapper_args_template, context))
         return false;
 
-      // Store the delegate for this event in this instance. This is initialized in register_event_proxies()
+      // Store the delegate for this event in this instance. This is initialized in RegisterEventProxies()
       // We can't initialize them directly here as they depend on the member methods being valid (i.e.
       // the constructor being called).
       if (!as_generator(scope_tab << "Efl.EventCb evt_" << wrapper_evt_name << "_delegate;\n").generate(sink, attributes::unused, context))
index 9afb6aa..e2f7909 100644 (file)
@@ -237,7 +237,7 @@ struct klass
              << scope_tab << "public " << concrete_name << "(System.IntPtr raw)" << (root ? "" : " : base(raw)") << "\n"
              << scope_tab << "{\n"
              << scope_tab << scope_tab << (root ? "handle = raw;\n" : "")
-             << scope_tab << scope_tab << "register_event_proxies();\n"
+             << scope_tab << scope_tab << "RegisterEventProxies();\n"
              << scope_tab << "}\n"
             )
             .generate(sink, attributes::unused, concrete_cxt))
@@ -537,7 +537,7 @@ struct klass
                      << scope_tab << "public " << inherit_name << "(System.IntPtr raw)" << (root ? "" : " : base(raw)") << "\n"
                      << scope_tab << "{\n"
                      << scope_tab << scope_tab << (root ? "handle = raw;\n" : "")
-                     << scope_tab << scope_tab << "register_event_proxies();\n"
+                     << scope_tab << scope_tab << "RegisterEventProxies();\n"
                      << scope_tab << "}\n"
                  ).generate(sink, std::make_tuple(constructors, constructors, constructors), context))
          return false;
@@ -563,7 +563,7 @@ struct klass
              << scope_tab << scope_tab << scope_tab << "actual_klass = Efl.Eo.ClassRegister.GetInheritKlassOrRegister(base_klass, ((object)this).GetType());\n"
              << scope_tab << scope_tab << "}\n"
              << scope_tab << scope_tab << "handle = Efl.Eo.Globals.instantiate_start(actual_klass, parent);\n"
-             << scope_tab << scope_tab << "register_event_proxies();\n"
+             << scope_tab << scope_tab << "RegisterEventProxies();\n"
              << scope_tab << "}\n"
 
              << scope_tab << "protected void FinishInstantiation()\n"
@@ -632,7 +632,8 @@ struct klass
 
      // Event proxy registration
      if (!as_generator(
-            scope_tab << (is_inherit_context(context) || !root ? "protected " : "") << virtual_modifier << "void register_event_proxies()\n"
+            scope_tab << "///<summary>Register the Eo event wrappers making the bridge to C# events. Internal usage only.</summary>\n"
+            << scope_tab << (is_inherit_context(context) || !root ? "protected " : "") << virtual_modifier << "void RegisterEventProxies()\n"
             << scope_tab << "{\n"
          )
          .generate(sink, NULL, context))
@@ -641,7 +642,7 @@ struct klass
      // Generate event registrations here
 
      if (!root)
-       if (!as_generator(scope_tab << scope_tab << "base.register_event_proxies();\n").generate(sink, NULL, context))
+       if (!as_generator(scope_tab << scope_tab << "base.RegisterEventProxies();\n").generate(sink, NULL, context))
          return false;
 
      // Assigning the delegates