emacs-lisp: add missing '\' in template 96/19696/1
authorDave Barach <dave@barachs.net>
Sat, 18 May 2019 18:52:54 +0000 (14:52 -0400)
committerDave Barach <dave@barachs.net>
Sat, 18 May 2019 18:53:23 +0000 (14:53 -0400)
Fixes the generated setup_message_id_table(...)
implementation. Without a level of quoting in the emacs lisp skeleton,
the underscore macro used to generate calls to
vl_msg_api_add_msg_name_crc produces "fooCRC" instead of "foo_CRC".
That confuses the Python language binding.

Punctuation saves lives: "Let's eat Grandma" vs. "Let's eat, Grandma."
Or some such.

Change-Id: Ibecc20337092199d6757165ac96b70c830511bb3
Signed-off-by: Dave Barach <dave@barachs.net>
extras/emacs/plugin-main-skel.el

index 14606c6..31b258e 100644 (file)
@@ -104,8 +104,8 @@ int " plugin-name "_enable_disable (" plugin-name "_main_t * " main-p ", u32 sw_
                                sw_if_index, enable_disable, 0, 0);
 
   /* Send an event to enable/disable the periodic scanner process */
-  vlib_process_signal_event (" main-p "->vlib_main, " plugin-name"_periodic_node.index, 
-                            " PLUGIN-NAME"_EVENT_PERIODIC_ENABLE_DISABLE, 
+  vlib_process_signal_event (" main-p "->vlib_main, " plugin-name"_periodic_node.index,
+                            " PLUGIN-NAME"_EVENT_PERIODIC_ENABLE_DISABLE,
                             (uword)enable_disable);
 
   return rv;
@@ -122,7 +122,7 @@ static clib_error_t *
 
   int rv;
 
-  while (unformat_check_input (input) != UNFORMAT_END_OF_INPUT) 
+  while (unformat_check_input (input) != UNFORMAT_END_OF_INPUT)
     {
       if (unformat (input, \"disable\"))
         enable_disable = 0;
@@ -138,7 +138,7 @@ static clib_error_t *
 
   rv = " plugin-name "_enable_disable (" main-p ", sw_if_index, enable_disable);
 
-  switch(rv) 
+  switch(rv)
     {
   case 0:
     break;
@@ -160,7 +160,7 @@ static clib_error_t *
 }
 
 /* *INDENT-OFF* */
-VLIB_CLI_COMMAND (" plugin-name "_enable_disable_command, static) = 
+VLIB_CLI_COMMAND (" plugin-name "_enable_disable_command, static) =
 {
   .path = \"" plugin-name " enable-disable\",
   .short_help =
@@ -210,7 +210,7 @@ static void
 setup_message_id_table (" plugin-name "_main_t * " main-p ", api_main_t * am)
 {
 #define _(id,n,crc) \
-  vl_msg_api_add_msg_name_crc (am, #n "_" #crc, id + " main-p "->msg_id_base);
+  vl_msg_api_add_msg_name_crc (am, #n \"_\" #crc, id + " main-p "->msg_id_base);
   foreach_vl_msg_name_crc_" plugin-name" ;
 #undef _
 }
@@ -252,7 +252,7 @@ VNET_FEATURE_INIT (" plugin-name ", static) =
 /* *INDENT-ON */
 
 /* *INDENT-OFF* */
-VLIB_PLUGIN_REGISTER () = 
+VLIB_PLUGIN_REGISTER () =
 {
   .version = VPP_BUILD_VER,
   .description = \"" plugin-name " plugin description goes here\",
@@ -267,4 +267,3 @@ VLIB_PLUGIN_REGISTER () =
  * End:
  */
 ")
-