Rename get_plugin_dir() for consistency
authorJoão Valverde <joao.valverde@tecnico.ulisboa.pt>
Mon, 18 Sep 2017 14:45:17 +0000 (15:45 +0100)
committerGuy Harris <guy@alum.mit.edu>
Wed, 20 Sep 2017 01:00:37 +0000 (01:00 +0000)
WS_DLL_PUBLIC const char *get_plugin_dir(void);
WS_DLL_PUBLIC const char *get_plugins_pers_dir(void);

Opt for the plural form consistently (for public functions at least).

Change-Id: I8a5861ad7f90f9c87168bd3275bd9dbc5c83b749
Reviewed-on: https://code.wireshark.org/review/23608
Petri-Dish: João Valverde <j@v6e.pt>
Reviewed-by: Guy Harris <guy@alum.mit.edu>
debian/libwsutil0.symbols
epan/wslua/init_wslua.c
epan/wslua/wslua_dir.c
tshark.c
ui/gtk/about_dlg.c
ui/qt/about_dialog.cpp
wsutil/filesystem.c
wsutil/filesystem.h
wsutil/plugins.c

index 5d1b215dabca58667032a889b70fe014560e1ea3..c737a39bb73e98bf89e9babd1fe3766fa050aa72 100644 (file)
@@ -73,7 +73,7 @@ libwsutil.so.0 libwsutil0 #MINVER#
  get_os_version_info@Base 1.99.0
  get_persconffile_path@Base 1.12.0~rc1
  get_persdatafile_dir@Base 1.12.0~rc1
- get_plugin_dir@Base 1.12.0~rc1
+ get_plugins_dir@Base 2.5.0
  get_plugins_pers_dir@Base 1.12.0~rc1
  get_positive_int@Base 1.99.0
  get_profile_name@Base 1.12.0~rc1
index 0d19a5c4028efdcea71024e7ce7ec12e55610baa..7d996773cb7221cccc71cc7f5bc1e6e0dca2257e 100644 (file)
@@ -501,7 +501,7 @@ static void set_file_environment(const gchar* filename, const gchar* dirname) {
     lua_pop(L, 1);                  /* pop the path string */
     /* prepend the various paths */
     lua_pushfstring(L, "%s" G_DIR_SEPARATOR_S "?.lua;%s" G_DIR_SEPARATOR_S "?.lua;%s" G_DIR_SEPARATOR_S "?.lua;%s",
-                    dirname, get_plugins_pers_dir(), get_plugin_dir(), path);
+                    dirname, get_plugins_pers_dir(), get_plugins_dir(), path);
     lua_setfield(L, -2, "path");    /* set the new string to be the path field of the package table */
     lua_setfield(L, -2, "package"); /* set the package table to be the package field of the global */
 
@@ -664,7 +664,7 @@ int wslua_count_plugins(void) {
     int plugins_counter;
 
     /* count global scripts */
-    plugins_counter = lua_load_plugins(get_plugin_dir(), NULL, NULL, TRUE, FALSE);
+    plugins_counter = lua_load_plugins(get_plugins_dir(), NULL, NULL, TRUE, FALSE);
 
     /* count users init.lua */
     filename = get_persconffile_path("init.lua", FALSE);
@@ -922,7 +922,7 @@ void wslua_init(register_cb cb, gpointer client_data) {
     lua_pop(L,1);  /* pop the getglobal result */
 
     /* load global scripts */
-    lua_load_plugins(get_plugin_dir(), cb, client_data, FALSE, FALSE);
+    lua_load_plugins(get_plugins_dir(), cb, client_data, FALSE, FALSE);
 
     /* check whether we should run other scripts even if running superuser */
     lua_getglobal(L,"run_user_scripts_when_superuser");
index 1dc356ad715e837ce7743e00c76630e86757eb28..3a9d50b28544fd2e803c1f6204c98585a6924716 100644 (file)
@@ -333,7 +333,7 @@ WSLUA_CONSTRUCTOR Dir_global_plugins_path(lua_State* L) {
 
        @since 1.11.3
     */
-    lua_pushstring(L, get_plugin_dir());
+    lua_pushstring(L, get_plugins_dir());
     WSLUA_RETURN(1); /* The pathname for the global plugins directory. */
 }
 
index ad710b8b088d84a22b35503e06aa9c213d01f8bc..4bd07a636b5247c26130a68f1f0ca78ae8e3a27c 100644 (file)
--- a/tshark.c
+++ b/tshark.c
@@ -616,7 +616,7 @@ about_folders(void)
   printf("%-21s\t%s\n", "Personal Plugins:", get_plugins_pers_dir());
 
   /* global plugins */
-  printf("%-21s\t%s\n", "Global Plugins:", get_plugin_dir());
+  printf("%-21s\t%s\n", "Global Plugins:", get_plugins_dir());
 #endif
 
 #ifdef HAVE_GEOIP
index a5d8643f7a4c9f8b5fa43516ca584ddf5aab9bf4..18b1dc5f14d518e19a881ed9b2a6e9c7feac73e0 100644 (file)
@@ -464,7 +464,7 @@ about_folders_page_new(void)
       "dissector plugins");
 
   /* global plugins */
-  about_folders_row(table, "Global Plugins", get_plugin_dir(),
+  about_folders_row(table, "Global Plugins", get_plugins_dir(),
       "dissector plugins");
 #endif
 
index c4b9c3128a1e64502e819caa7c4c98a326cb1ac3..ed48c66a755a7566727f56b71fd488dd5417bb10 100644 (file)
@@ -229,7 +229,7 @@ AboutDialog::AboutDialog(QWidget *parent) :
     message += about_folders_row("Personal Plugins", get_plugins_pers_dir(), "dissector plugins");
 
     /* global plugins */
-    message += about_folders_row("Global Plugins", get_plugin_dir(), "dissector plugins");
+    message += about_folders_row("Global Plugins", get_plugins_dir(), "dissector plugins");
 #endif
 
 #ifdef HAVE_GEOIP
index 9aec7a7ca6baa96b936197c73e9093df8a2f413f..fd0964975a69c401d8e3586efd4d38d457bc8e06 100644 (file)
@@ -1050,7 +1050,7 @@ init_plugin_pers_dir(void)
  * Get the directory in which the plugins are stored.
  */
 const char *
-get_plugin_dir(void)
+get_plugins_dir(void)
 {
 #if defined(HAVE_PLUGINS) || defined(HAVE_LUA)
     if (!plugin_dir) init_plugin_dir();
index 4ccc159ed722888665fc5185f6c6d80edd36cb96..9357a437ad034c7a5976bf6e1ee44456c24c9621 100644 (file)
@@ -53,7 +53,7 @@ WS_DLL_PUBLIC const char *get_progfile_dir(void);
  * before init_progfile_dir() is called, as they might be stored in a
  * subdirectory of the program file directory.
  */
-WS_DLL_PUBLIC const char *get_plugin_dir(void);
+WS_DLL_PUBLIC const char *get_plugins_dir(void);
 
 /*
  * Get the personal plugin dir.
index 98b78db0e5227e61d63e8d9a29e5a17747628b3a..6febafbf07d63400aa535bd6f5be09bfdec755f2 100644 (file)
@@ -297,7 +297,7 @@ scan_plugins(plugin_load_failure_mode mode)
          * out-of-tree build. If we find subdirectories scan those since
          * they will contain plugins in the case of an in-tree build.
          */
-        plugin_dir = get_plugin_dir();
+        plugin_dir = get_plugins_dir();
         if (plugin_dir == NULL)
         {
             /* We couldn't find the plugin directory. */
@@ -342,7 +342,7 @@ scan_plugins(plugin_load_failure_mode mode)
         else
         {
             plugin_dir_path = g_strdup_printf("%s" G_DIR_SEPARATOR_S "%d.%d",
-                                                get_plugin_dir(), VERSION_MAJOR, VERSION_MINOR);
+                                                get_plugins_dir(), VERSION_MAJOR, VERSION_MINOR);
             plugins_scan_dir(plugin_dir_path, mode);
             g_free(plugin_dir_path);
         }