revise parameter name for readability

diff --git a/include/Plugin.h b/include/Plugin.h
index 4d98c83..d120ccd 100644 (file)
--- a/include/Plugin.h
+++ b/include/Plugin.h
@@ -42,28 +42,28 @@ namespace ModeSupervisorNamespace {
                        return name;
                }
 
-               virtual int set(const std::string &key, int val, PluginAction **pluginAction)
+               virtual int set(const std::string &key, int val, PluginAction **piAction)
                {
                        return MODES_ERROR_NOT_SUPPORTED;
                }
-               virtual int set(const std::string &key, double val, PluginAction **pluginAction)
+               virtual int set(const std::string &key, double val, PluginAction **piAction)
                {
                        return MODES_ERROR_NOT_SUPPORTED;
                }
-               virtual int set(const std::string &key, bool val, PluginAction **pluginAction)
+               virtual int set(const std::string &key, bool val, PluginAction **piAction)
                {
                        return MODES_ERROR_NOT_SUPPORTED;
                }
-               virtual int set(const std::string &key, const std::string &val, PluginAction **pluginAction)
+               virtual int set(const std::string &key, const std::string &val, PluginAction **piAction)
                {
                        return MODES_ERROR_NOT_SUPPORTED;
                }
-               virtual int undo(PluginAction *pluginAction)
+               virtual int undo(PluginAction *piAction)
                {
-                       if (NULL == pluginAction)
+                       if (NULL == piAction)
                                return MODES_ERROR_INVALID_PARAMETER;
-                       int ret = pluginAction->undo();
-                       delete pluginAction;
+                       int ret = piAction->undo();
+                       delete piAction;
 
                        return ret;
                }
@@ -71,13 +71,13 @@ namespace ModeSupervisorNamespace {
                {
                        return nullptr;
                }
-               virtual int serializeAction(PluginAction *pluginAction, std::string *output)
+               virtual int serializeAction(PluginAction *piAction, std::string *output)
                {
                        return MODES_ERROR_NOT_SUPPORTED;
                }
-               void freeAction(PluginAction *pluginAction)
+               void freeAction(PluginAction *piAction)
                {
-                       delete pluginAction;
+                       delete piAction;
                }
                virtual int getInt(const std::string &key)
                {

diff --git a/plugin/TestPlugin.cpp b/plugin/TestPlugin.cpp
index ab22bcd..e944731 100644 (file)
--- a/plugin/TestPlugin.cpp
+++ b/plugin/TestPlugin.cpp
@@ -29,14 +29,14 @@ public:
        TestPlugin();
        ~TestPlugin();
 
-       int set(const std::string &key, int val, PluginAction **pluginAction) override;
-       int set(const std::string &key, double val, PluginAction **pluginAction) override;
-       int set(const std::string &key, bool val, PluginAction **pluginAction) override;
-       int set(const std::string &key, const std::string &val, PluginAction **pluginAction) override;
+       int set(const std::string &key, int val, PluginAction **piAction) override;
+       int set(const std::string &key, double val, PluginAction **piAction) override;
+       int set(const std::string &key, bool val, PluginAction **piAction) override;
+       int set(const std::string &key, const std::string &val, PluginAction **piAction) override;
 
-       int undo(PluginAction *pluginAction) override;
+       int undo(PluginAction *piAction) override;
        PluginAction* getUndoAction(const std::string &key, const std::string &info) override;
-       int serializeAction(PluginAction *pluginAction, std::string *output) override;
+       int serializeAction(PluginAction *piAction, std::string *output) override;
 
        int getInt(const std::string &key) override;
        double getDouble(const std::string &key) override;
@@ -67,56 +67,56 @@ TestPlugin::~TestPlugin()
 {
 }
 
-int TestPlugin::set(const std::string &key, int val, PluginAction **pluginAction)
+int TestPlugin::set(const std::string &key, int val, PluginAction **piAction)
 {
        DBG("TestPlugin::set int ( %s, %d )", key.c_str(), val);
 
-       if (pluginAction) {
+       if (piAction) {
                TestPluginAction *testAction = new TestPluginAction();
-               *pluginAction = testAction;
+               *piAction = testAction;
        }
        return MODES_ERROR_NONE;
 }
 
-int TestPlugin::set(const std::string &key, double val, PluginAction **pluginAction)
+int TestPlugin::set(const std::string &key, double val, PluginAction **piAction)
 {
        DBG("TestPlugin::set double ( %s, %lf )", key.c_str(), val);
 
-       if (pluginAction) {
+       if (piAction) {
                TestPluginAction *testAction = new TestPluginAction();
-               *pluginAction = testAction;
+               *piAction = testAction;
        }
        return MODES_ERROR_NONE;
 }
 
-int TestPlugin::set(const std::string &key, bool val, PluginAction **pluginAction)
+int TestPlugin::set(const std::string &key, bool val, PluginAction **piAction)
 {
        DBG("TestPlugin::set bool ( %s, %s )", key.c_str(), val ? "true" : "false");
 
-       if (pluginAction) {
+       if (piAction) {
                TestPluginAction *testAction = new TestPluginAction();
-               *pluginAction = testAction;
+               *piAction = testAction;
        }
        return MODES_ERROR_NONE;
 }
 
-int TestPlugin::set(const std::string &key, const std::string &val, PluginAction **pluginAction)
+int TestPlugin::set(const std::string &key, const std::string &val, PluginAction **piAction)
 {
        DBG("TestPlugin::set string ( %s, %s )", key.c_str(), val.c_str());
 
-       if (pluginAction) {
+       if (piAction) {
                TestPluginAction *testAction = new TestPluginAction();
-               *pluginAction = testAction;
+               *piAction = testAction;
        }
        return MODES_ERROR_NONE;
 }
 
-int TestPlugin::undo(PluginAction *pluginAction)
+int TestPlugin::undo(PluginAction *piAction)
 {
-       if (pluginAction) {
-               DBG("TestPlugin::undo(Action:%s)", pluginAction->getName().c_str());
+       if (piAction) {
+               DBG("TestPlugin::undo(Action:%s)", piAction->getName().c_str());
 
-               delete pluginAction;
+               delete piAction;
        }
 
        return MODES_ERROR_NONE;