Remove duplicated codes 97/208597/3
authorKiseok Chang <kiso.chang@samsung.com>
Wed, 26 Jun 2019 08:34:40 +0000 (17:34 +0900)
committerKiseok Chang <kiso.chang@samsung.com>
Thu, 27 Jun 2019 02:04:12 +0000 (11:04 +0900)
- menufunc_*()& memugetstring_*() had same codes
- remove redundent menufunc_*()

Change-Id: If91a5e4437857270151fb3416fb25118b8983519
Signed-off-by: Kiseok Chang <kiso.chang@samsung.com>
17 files changed:
src/common/GenSettingMenu.cpp
src/common/GenSettingMenu.h
src/common/SettingMenu.h
src/report/sc_report.cpp
src/sc_main.cpp
src/settings/sc_aboutdevice.cpp
src/settings/sc_airplane.cpp
src/settings/sc_apps.cpp
src/settings/sc_battery.cpp
src/settings/sc_bluetooth.cpp
src/settings/sc_datausage.cpp
src/settings/sc_datetime.cpp
src/settings/sc_display.cpp
src/settings/sc_sound.cpp
src/settings/sc_storage.cpp
src/settings/sc_wifi.cpp
template/sc_tmplt_connections_status.txt

index ca26c1f..0e167a2 100644 (file)
@@ -3,8 +3,9 @@
 
 using namespace std;
 
-GenSettingMenu::GenSettingMenu(const char * optstr, const char * namestr, MenuFunc funcptr, MenuPreCreate precreateptr, MenuGetString getstringptr)
-       :strOpt(optstr), strName(namestr), ptrFunc(funcptr), ptrPrecreate(precreateptr), ptrGetString(getstringptr)
+
+GenSettingMenu::GenSettingMenu(const char * optstr, const char * namestr, MenuGetString getstringptr, MenuFunc funcptr, MenuPreCreate precreateptr)
+       :strOpt(optstr), strName(namestr), ptrGetString(getstringptr), ptrFunc(funcptr), ptrPrecreate(precreateptr)
 {
 }
 
@@ -25,6 +26,16 @@ MENURUN_ERROR GenSettingMenu::func(string &title, int argc, char *argv[])
                printf("%s\n\n", title.c_str());
                return (*ptrFunc)(argc, argv, this);
        }
+       else if (ptrGetString) {
+               printf("%s\n\n", title.c_str());
+
+               string resultstr;
+               MENURUN_ERROR ret = (*ptrGetString)(resultstr, argc, argv, this);
+               if (ret != MENURUN_ERR_NONE)
+                       resultstr = "[Failed]";
+               printf("    %s : %s\n", name(), resultstr.c_str());
+               return ret;
+       }
 
        return MENURUN_ERR_NONE;
 }
@@ -35,14 +46,14 @@ MENURUN_ERROR GenSettingMenu::getstring(string &resultstr, int argc, char *argv[
        if (ptrGetString) {
                return (*ptrGetString)(resultstr, argc, argv, this);
        }
-       
+
        return SettingMenu::getstring(resultstr, argc, argv);
 }
 
 
 bool GenSettingMenu::isImplemented()
 {
-       if (ptrFunc == nullptr && subMenus.empty()) return false;
+       if (ptrFunc == nullptr && ptrGetString == nullptr && subMenus.empty()) return false;
        return true;
 }
 
@@ -56,14 +67,14 @@ const char *GenSettingMenu::name()
        return strName;
 }
 
-void GenSettingMenu::addSubmenu(const char * optstr, const char * namestr, MenuFunc funcptr, MenuPreCreate precreateptr, MenuGetString getstringptr)
+void GenSettingMenu::addSubmenu(const char * optstr, const char * namestr, MenuGetString getstringptr, MenuFunc funcptr, MenuPreCreate precreateptr)
 {
-       subMenus.push_back(new GenSettingMenu(optstr, namestr, funcptr, precreateptr, getstringptr));
+       subMenus.push_back(new GenSettingMenu(optstr, namestr, getstringptr, funcptr, precreateptr));
 }
 
 void GenSettingMenu::addSubmenu(MenuItem item)
 {
-       addSubmenu(item.optstr, item.namestr, item.funcptr, item.precreateptr, item.getstringptr);
+       addSubmenu(item.optstr, item.namestr, item.getstringptr, item.funcptr, item.precreateptr);
 }
 
 void GenSettingMenu::addSubmenus(MenuItem *items, int count)
index 81029c1..40637db 100644 (file)
@@ -8,23 +8,23 @@
 
 class GenSettingMenu;
 
+typedef  MENURUN_ERROR(*MenuGetString)(std::string &resultstr, int argc, char *argv[], GenSettingMenu *menu);
 typedef  MENURUN_ERROR (*MenuFunc)(int argc, char *argv[], GenSettingMenu *menu);
-typedef  MENURUN_ERROR(*MenuGetString)(std::string &resultstr,int argc, char *argv[], GenSettingMenu *menu);
 typedef  void (*MenuPreCreate)(GenSettingMenu  *menu);
 
 typedef struct {
        const char *optstr;
        const char *namestr;
+
+       MenuGetString getstringptr;
        MenuFunc funcptr;
        MenuPreCreate precreateptr;
-       MenuGetString getstringptr;
 } MenuItem;
 
-
 class GenSettingMenu : public SettingMenu
 {
 public:
-       GenSettingMenu(const char * optstr, const char * namestr, MenuFunc funcptr = nullptr, MenuPreCreate precreateptr = nullptr, MenuGetString getstringptr = nullptr);
+       GenSettingMenu(const char * optstr, const char * namestr, MenuGetString getstringptr = nullptr, MenuFunc funcptr = nullptr, MenuPreCreate precreateptr = nullptr);
 
 public:
        bool create();
@@ -36,7 +36,7 @@ public:
        const char *opt();
        const char *name();
 
-       void addSubmenu(const char * optstr, const char * namestr, MenuFunc funcptr = nullptr, MenuPreCreate precreateptr = nullptr, MenuGetString getstringptr = nullptr);
+       void addSubmenu(const char * optstr, const char * namestr, MenuGetString getstringptr = nullptr, MenuFunc funcptr = nullptr, MenuPreCreate precreateptr = nullptr);
        void addSubmenu(MenuItem item);
        void addSubmenus(MenuItem *items, int count);
 
@@ -44,9 +44,9 @@ private:
        const char * strOpt;
        const char * strName;
 
+       MenuGetString ptrGetString;
        MenuFunc ptrFunc;
        MenuPreCreate ptrPrecreate;
-       MenuGetString ptrGetString;
 };
 
 #endif //__GENSETTINGMENU_H_
index 0140dc4..a5925f5 100644 (file)
@@ -28,7 +28,6 @@ public:
        virtual MENURUN_ERROR getstring(std::string &resultstr, int argc, char *argv[]);
 
        virtual MENURUN_ERROR run(SettingCommand &settingcmds, std::string &title, int argc, char *argv[]);
-
        virtual MENURUN_ERROR get(std::string &resultstr, SettingCommand &settingcmds, int argc, char *argv[]);
 
        virtual bool isImplemented() { return false; }
index e82557a..b8117d4 100644 (file)
@@ -19,7 +19,6 @@ string getString(string& commandstr, GenSettingMenu *rootmenu, int argc, char *a
        MENURUN_ERROR result = rootmenu->get(resultstr, settingcommands, argc, argv);
        if (result != MENURUN_ERR_NONE) {
                resultstr = "[Failed]";
-//             SettingMenu::printError(result);
        }
        return resultstr;
 }
@@ -34,7 +33,7 @@ bool isSeperator(char ch)
 
 MENURUN_ERROR getReportString(string &reportStr, const string &templateStr, int argc, char *argv[])
 {
-       GenSettingMenu mainMenu("setting-cmd", "Main Menu", nullptr, MainMemu_Precreate);
+       GenSettingMenu mainMenu("setting-cmd", "Main Menu", nullptr, nullptr, MainMemu_Precreate);
 
        if (!mainMenu.create()) {
                eprintf("Main Menu failed to create\n");
@@ -102,7 +101,7 @@ MENURUN_ERROR menufunc_report(int argc, char *argv[], GenSettingMenu *menu)
        fseek(pfile, 0, SEEK_END);
        int fsize = ftell(pfile);
        fseek(pfile, 0, SEEK_SET);
-       
+
        eprintf("fsize : %d\n", fsize);
 
        int numread = fread(reportTemplate, sizeof(char), fsize, pfile);
index 83d3177..2778ed0 100644 (file)
 
 using namespace std;
 
-#if 0
-void setMainMenu_mobile(SettingMenu &mainmenu)
-{
-       mainmenu.destroy();
-
-       mainmenu.subMenus.push_back(new MMWiFi);
-       mainmenu.subMenus.push_back(new MMBluetooth);
-       mainmenu.subMenus.push_back(new MMAirplaneMode);
-       mainmenu.subMenus.push_back(new MMMobileNetwork);
-       mainmenu.subMenus.push_back(new MMSound);
-       mainmenu.subMenus.push_back(new MMDisplay);
-       mainmenu.subMenus.push_back(new MMApps);
-       mainmenu.subMenus.push_back(new MMWallpaper);
-       mainmenu.subMenus.push_back(new MMLockScreen);
-       mainmenu.subMenus.push_back(new MMAccessibility);
-       mainmenu.subMenus.push_back(new MMPrivacySecurity);
-       mainmenu.subMenus.push_back(new MMAccounts);
-       mainmenu.subMenus.push_back(new MMBackupReset);
-       mainmenu.subMenus.push_back(new MMLanguageInput);
-       mainmenu.subMenus.push_back(new MMBattery);
-       mainmenu.subMenus.push_back(new MMData);
-       mainmenu.subMenus.push_back(new MMStorage);
-       mainmenu.subMenus.push_back(new MMDateTime);
-       mainmenu.subMenus.push_back(new MMDataUsage);
-       mainmenu.subMenus.push_back(new MMAboutDevice);
-}
-#else
 
 MenuItem mobile_mainmenulist[] = {
-       {"wf", "Wi-Fi", nullptr, precreate_wf},
-       {"bt", "BlueTooth", nullptr, precreate_bt},
-       {"am", "Airplane Mode", nullptr, precreate_am},
-       {"te", "Tethering", nullptr, nullptr},
-       {"mn", "Mobile networks", nullptr, nullptr},
-       {"so", "Sound", nullptr, precreate_so},
-       {"di", "Display", nullptr, precreate_di},
-       {"ap", "Apps", nullptr, precreate_apps},
-       {"wp", "Wallpaper", nullptr, nullptr},
-       {"ls", "LockScreen", nullptr, nullptr},
-       {"ac", "Accessibility", nullptr, nullptr},
-       {"ps", "Privacy and security", nullptr, nullptr},
-       {"ac", "Accounts", nullptr, nullptr},
-       {"br", "Backup and reset", nullptr, nullptr},
-       {"li", "Language and input", nullptr, nullptr},
-       {"ba", "Battery", menufunc_ba, nullptr},
-       {"da", "Data", nullptr, nullptr},
-       {"st", "Storage", nullptr, precreate_st},
-       {"dt", "Date and time", nullptr, precreate_dt},
-       {"du", "Data Usage", nullptr, precreate_du},
-       {"ad", "About device", nullptr, precreate_ad}
+       {"wf", "Wi-Fi", nullptr, nullptr, precreate_wf},
+       {"bt", "BlueTooth", nullptr, nullptr, precreate_bt},
+       {"am", "Airplane Mode", nullptr, nullptr, precreate_am},
+       {"te", "Tethering", nullptr, nullptr, nullptr},
+       {"mn", "Mobile networks", nullptr, nullptr, nullptr},
+       {"so", "Sound", nullptr, nullptr, precreate_so},
+       {"di", "Display", nullptr, nullptr, precreate_di},
+       {"ap", "Apps", nullptr, nullptr, precreate_apps},
+       {"wp", "Wallpaper", nullptr, nullptr, nullptr},
+       {"ls", "LockScreen", nullptr, nullptr, nullptr},
+       {"ac", "Accessibility", nullptr, nullptr, nullptr},
+       {"ps", "Privacy and security", nullptr, nullptr, nullptr},
+       {"ac", "Accounts", nullptr, nullptr, nullptr},
+       {"br", "Backup and reset", nullptr, nullptr, nullptr},
+       {"li", "Language and input", nullptr, nullptr, nullptr},
+       {"ba", "Battery", nullptr, menufunc_ba, nullptr},
+       {"da", "Data", nullptr, nullptr, nullptr},
+       {"st", "Storage", nullptr, nullptr, precreate_st},
+       {"dt", "Date and time", nullptr, nullptr, precreate_dt},
+       {"du", "Data Usage", nullptr, nullptr, precreate_du},
+       {"ad", "About device", nullptr, nullptr, precreate_ad}
 };
 
-#endif
-
-
-#if 0
-void setMainMenu_wearable(SettingMenu &mainmenu)
-{
-       mainmenu.destroy();
-
-       mainmenu.subMenus.push_back(new MWWiFi);
-       mainmenu.subMenus.push_back(new MWBluetooth);
-       mainmenu.subMenus.push_back(new MWAirplaneMode);
-       mainmenu.subMenus.push_back(new MWMobileNetwork);
-       mainmenu.subMenus.push_back(new MWSound);
-       mainmenu.subMenus.push_back(new MWDisplay);
-       mainmenu.subMenus.push_back(new MWAccessibility);
-       mainmenu.subMenus.push_back(new MWAccounts);
-       mainmenu.subMenus.push_back(new MWLanguageInput);
-       mainmenu.subMenus.push_back(new MWDataUsage);
-       mainmenu.subMenus.push_back(new MWAboutDevice);
-
-       mainmenu.create();
-}
-#else
-
 MenuItem wearable_main_menulist[] = {
-       {"wf", "Wi-Fi", nullptr, precreate_wf},
-       {"bt", "BlueTooth", nullptr, precreate_bt},
-       {"am", "Airplane Mode", nullptr, precreate_am},
-       {"mn", "Mobile networks", nullptr, nullptr},
-       {"so", "Sound", nullptr, precreate_so},
-       {"di", "Display", nullptr, precreate_di},
-       {"ac", "Accessibility", nullptr, nullptr},
-       {"ac", "Accounts", nullptr, nullptr},
-       {"li", "Language and input", nullptr, nullptr},
-       {"du", "Data Usage", nullptr, precreate_du},
-       {"ad", "About device", nullptr, precreate_ad},
+       {"wf", "Wi-Fi", nullptr, nullptr, precreate_wf},
+       {"bt", "BlueTooth", nullptr, nullptr, precreate_bt},
+       {"am", "Airplane Mode", nullptr, nullptr, precreate_am},
+       {"mn", "Mobile networks", nullptr, nullptr, nullptr},
+       {"so", "Sound", nullptr, nullptr, precreate_so},
+       {"di", "Display", nullptr, nullptr, precreate_di},
+       {"ac", "Accessibility", nullptr, nullptr, nullptr},
+       {"ac", "Accounts", nullptr, nullptr, nullptr},
+       {"li", "Language and input", nullptr, nullptr, nullptr},
+       {"du", "Data Usage", nullptr, nullptr, precreate_du},
+       {"ad", "About device", nullptr, nullptr, precreate_ad},
 };
-#endif
 
 void MainMemu_Precreate(GenSettingMenu *menu)
 {
@@ -133,9 +81,9 @@ void MainMemu_Precreate(GenSettingMenu *menu)
 
 int main(int argc, char *argv[])
 {
-       GenSettingMenu mainMenu("setting-cmd", "Main Menu", nullptr, MainMemu_Precreate);
+       GenSettingMenu mainMenu("setting-cmd", "Main Menu", nullptr, nullptr, MainMemu_Precreate);
 
-       mainMenu.addSubmenu("report", "Report", menufunc_report);
+       mainMenu.addSubmenu("report", "Report", nullptr, menufunc_report);
 
        if (!mainMenu.create()) {
                eprintf("Main Menu failed to create\n");
index 7287040..7c44e8c 100644 (file)
 
 using namespace std;
 
-
 #define SETTING_ABOUT_POWER_SUPPLY_PATH "/sys/class/power_supply"
 #define SETTING_ABOUT_STAT_PATH "/proc/stat"
 #define MAX_COMMON_BUFFER_LEN          1024
 
-
-
 /**
  * @brief get Model Name.
  *             - No ini: Unavailable
@@ -46,29 +43,26 @@ using namespace std;
  * @param szStr the buffer for store the model name
  * @param nSize buffer size
  */
-static void __get_phone_model_name(char *szStr, int nSize)
+static bool __get_phone_model_name(char *szStr, int nSize)
 {
-       if (szStr == nullptr)
+       if (szStr == nullptr) {
                eprintf("szStr parameter is nullptr\n");
+               return false;
+       }
 
        char *value = nullptr;
        int ret = system_info_get_platform_string(
                        "http://tizen.org/system/model_name", &value);
-       SETTINGCMD_TRACE("value : %s", value);
        if (ret != SYSTEM_INFO_ERROR_NONE) {
-               eprintf("fail to call system_info_get_platform_string");
                FREE(value);
+               eprintf("fail to call system_info_get_platform_string");
                snprintf(szStr, nSize, "%s", "Unavailable");
-               return;
+               return false;
        }
 
-       if (value)
-               snprintf(szStr, nSize, "%s", value);
-       else
-               snprintf(szStr, nSize, "%s", "Unavailable");
-
-
+       snprintf(szStr, nSize, "%s", value);
        FREE(value);
+       return true;
 }
 
 /**
@@ -77,24 +71,26 @@ static void __get_phone_model_name(char *szStr, int nSize)
  * @param szStr the buffer for store SW version
  * @param nSize buffer size
  */
-static void __get_sw_version(char *szStr, int nSize)
+static bool __get_sw_version(char *szStr, int nSize)
 {
-       if (szStr == nullptr)
+       if (szStr == nullptr) {
                eprintf("szStr parameter is nullptr\n");
+               return false;
+       }
 
        char *version = nullptr;
        int ret = system_info_get_platform_string(
                        "http://tizen.org/feature/platform.version", &version);
        if (ret != SYSTEM_INFO_ERROR_NONE) {
-               eprintf("fail to call system_info_get_platform_string");
                FREE(version);
+               eprintf("fail to call system_info_get_platform_string");
                snprintf(szStr, nSize, "%s", "Unavailable");
-               return;
+               return false;
        }
 
        snprintf(szStr, nSize, "TIZEN %s", version);
-
        FREE(version);
+       return true;
 }
 
 /**
@@ -113,29 +109,21 @@ static bool __stat_get_cpuinfo(char *szStr, int nSize)
        char *cpu_name = nullptr;
        ret = system_info_get_platform_string(
                "http://tizen.org/system/platform.processor", &cpu_name);
-
-       if (0 != ret) {
+       if (SYSTEM_INFO_ERROR_NONE != ret) {
+               FREE(cpu_name);
+               eprintf("fail to call system_info_get_platform_string");
                snprintf(szStr, nSize, "%s", "Unavailable");
                return false;
        }
 
        snprintf(szStr, nSize, "%s", cpu_name);
-       free(cpu_name);
+       FREE(cpu_name);
        return true;
 }
+
 /*---------------------------------------------------------------------------------------------------------------------------*/
 #define MAX_DISPLAY_STR_LEN_ON_PHONE_INFO      256
 
-static MENURUN_ERROR menufunc_ad_name(int argc, char *argv[], GenSettingMenu *menu)
-{
-       char *name_value = vconf_get_str(VCONFKEY_SETAPPL_DEVICE_NAME_STR);
-       printf("\tName : ");
-       printf("%s\n", name_value);
-       free(name_value);
-
-       return MENURUN_ERR_NONE;
-}
-
 static MENURUN_ERROR menugetstring_ad_name(string &resultstr, int argc, char *argv[], GenSettingMenu *menu)
 {
        char *name_value = vconf_get_str(VCONFKEY_SETAPPL_DEVICE_NAME_STR);
@@ -145,87 +133,39 @@ static MENURUN_ERROR menugetstring_ad_name(string &resultstr, int argc, char *ar
        return MENURUN_ERR_NONE;
 };
 
-static MENURUN_ERROR menufunc_ad_mn(int argc, char *argv[], GenSettingMenu *menu)
-{
-
-       char str[MAX_DISPLAY_STR_LEN_ON_PHONE_INFO];
-       str[0] = '\0';
-       __get_phone_model_name(str, MAX_DISPLAY_STR_LEN_ON_PHONE_INFO);
-       printf("\tModel number : ");
-       printf("%s\n", str);
-
-       return MENURUN_ERR_NONE;
-}
-
 static MENURUN_ERROR menugetstring_ad_mn(string &resultstr, int argc, char *argv[], GenSettingMenu *menu)
 {
        char str[MAX_DISPLAY_STR_LEN_ON_PHONE_INFO];
        str[0] = '\0';
-       __get_phone_model_name(str, MAX_DISPLAY_STR_LEN_ON_PHONE_INFO);
-       resultstr = str;
+       if(!__get_phone_model_name(str, MAX_DISPLAY_STR_LEN_ON_PHONE_INFO))
+               return MENURUN_ERR_NORMAL;
 
+       resultstr = str;
        return MENURUN_ERR_NONE;
 };
 
-static MENURUN_ERROR menufunc_ad_ver(int argc, char *argv[], GenSettingMenu *menu)
-{
-
-       char str[MAX_DISPLAY_STR_LEN_ON_PHONE_INFO];
-       str[0] = '\0';
-       __get_sw_version(str, MAX_DISPLAY_STR_LEN_ON_PHONE_INFO);
-       printf("\tModel number : ");
-       printf("%s\n", str);
-
-       return MENURUN_ERR_NONE;
-}
-
 static MENURUN_ERROR menugetstring_ad_ver(string &resultstr, int argc, char *argv[], GenSettingMenu *menu)
 {
        char str[MAX_DISPLAY_STR_LEN_ON_PHONE_INFO];
        str[0] = '\0';
-       __get_sw_version(str, MAX_DISPLAY_STR_LEN_ON_PHONE_INFO);
-       resultstr = str;
+       if (!__get_sw_version(str, MAX_DISPLAY_STR_LEN_ON_PHONE_INFO))
+               return MENURUN_ERR_NORMAL;
 
+       resultstr = str;
        return MENURUN_ERR_NONE;
 };
 
-static MENURUN_ERROR menufunc_ad_cpu(int argc, char *argv[], GenSettingMenu *menu)
-{
-       char str[MAX_DISPLAY_STR_LEN_ON_PHONE_INFO];
-       str[0] = '\0';
-       __stat_get_cpuinfo(str, MAX_DISPLAY_STR_LEN_ON_PHONE_INFO);
-       printf("\tModel number : ");
-       printf("%s\n", str);
-
-       return MENURUN_ERR_NONE;
-}
-
 static MENURUN_ERROR menugetstring_ad_cpu(string &resultstr, int argc, char *argv[], GenSettingMenu *menu)
 {
        char str[MAX_DISPLAY_STR_LEN_ON_PHONE_INFO];
        str[0] = '\0';
-       __stat_get_cpuinfo(str, MAX_DISPLAY_STR_LEN_ON_PHONE_INFO);
-       resultstr = str;
+       if (!__stat_get_cpuinfo(str, MAX_DISPLAY_STR_LEN_ON_PHONE_INFO))
+               return MENURUN_ERR_NORMAL;
 
+       resultstr = str;
        return MENURUN_ERR_NONE;
 };
 
-static MENURUN_ERROR menufunc_ad_ram(int argc, char *argv[], GenSettingMenu *menu)
-{
-       runtime_memory_info_s mem_info = {0};
-       int ret_value = runtime_info_get_system_memory_info(&mem_info);
-       if (RUNTIME_INFO_ERROR_NONE != ret_value)
-               return MENURUN_ERR_NORMAL;
-
-       char str[MAX_DISPLAY_STR_LEN_ON_PHONE_INFO];
-       float ram_total_gb = mem_info.total / 1000000.0;
-       snprintf(str, sizeof(str), "%0.1fGB", ram_total_gb);
-
-       printf("\tRAM : ");
-       printf("%s\n", str);
-
-       return MENURUN_ERR_NONE;
-}
 static MENURUN_ERROR menugetstring_ad_ram(string &resultstr, int argc, char *argv[], GenSettingMenu *menu)
 {
        runtime_memory_info_s mem_info = { 0 };
@@ -241,27 +181,6 @@ static MENURUN_ERROR menugetstring_ad_ram(string &resultstr, int argc, char *arg
        return MENURUN_ERR_NONE;
 };
 
-static MENURUN_ERROR menufunc_ad_res(int argc, char *argv[], GenSettingMenu *menu)
-{
-       int dim1, dim2;
-       int ret_value = system_info_get_platform_int(
-                                       "http://tizen.org/feature/screen.width", &dim1);
-       if (SYSTEM_INFO_ERROR_NONE != ret_value)
-               return MENURUN_ERR_NORMAL;
-
-       system_info_get_platform_int(
-                       "http://tizen.org/feature/screen.height", &dim2);
-       if (SYSTEM_INFO_ERROR_NONE != ret_value)
-               return MENURUN_ERR_NORMAL;
-
-       char str[MAX_DISPLAY_STR_LEN_ON_PHONE_INFO];
-       snprintf(str, sizeof(str), "%d x %d", dim1, dim2);
-
-       printf("\tResolution : ");
-       printf("%s\n", str);
-
-       return MENURUN_ERR_NONE;
-}
 static MENURUN_ERROR menugetstring_ad_res(string &resultstr, int argc, char *argv[], GenSettingMenu *menu)
 {
        int dim1, dim2;
@@ -270,7 +189,7 @@ static MENURUN_ERROR menugetstring_ad_res(string &resultstr, int argc, char *arg
        if (SYSTEM_INFO_ERROR_NONE != ret_value)
                return MENURUN_ERR_NORMAL;
 
-       system_info_get_platform_int(
+       ret_value = system_info_get_platform_int(
                "http://tizen.org/feature/screen.height", &dim2);
        if (SYSTEM_INFO_ERROR_NONE != ret_value)
                return MENURUN_ERR_NORMAL;
@@ -282,30 +201,28 @@ static MENURUN_ERROR menugetstring_ad_res(string &resultstr, int argc, char *arg
        return MENURUN_ERR_NONE;
 };
 
-
 /* sim_number is started from 1, ex) SIM1, SIM2, SIM3 ... */
 static bool _get_sim_phone_number(int sim_number, char *str, int str_size)
 {
        telephony_handle_list_s tel_h_list;
-       char *phone_num = nullptr;
-       int ret = 0;
        if (telephony_init(&tel_h_list) != TELEPHONY_ERROR_NONE) {
                eprintf("telephony_init failed");
                return false;
        }
 
        if ((int)tel_h_list.count < sim_number-1) {
-               printf("\tThere is no SIM%d card\n", sim_number);
+               eprintf("\tThere is no SIM%d card\n", sim_number);
                if (telephony_deinit(&tel_h_list) != TELEPHONY_ERROR_NONE) {
                        eprintf("telephony_deinit failed");
                        return false;
                }
                return false;
        }
-
-       ret = telephony_sim_get_subscriber_number(tel_h_list.handle[sim_number-1], &phone_num);
+       
+       char *phone_num = nullptr;
+       int ret = telephony_sim_get_subscriber_number(tel_h_list.handle[sim_number-1], &phone_num);
        if (ret != 0 || phone_num == nullptr) {
-               printf("\tThere is no SIM%d card, ret:%d phone_num addr:0x%x\n", sim_number, ret, (unsigned int)phone_num);
+               eprintf("\tThere is no SIM%d card, ret:%d phone_num addr:0x%x\n", sim_number, ret, (unsigned int)phone_num);
                if (telephony_deinit(&tel_h_list) != TELEPHONY_ERROR_NONE) {
                        eprintf("telephony_deinit failed");
                        return false;
@@ -314,7 +231,7 @@ static bool _get_sim_phone_number(int sim_number, char *str, int str_size)
        }
        snprintf(str, str_size, "%s", phone_num);
        free(phone_num);
-       
+
        if (telephony_deinit(&tel_h_list) != TELEPHONY_ERROR_NONE) {
                eprintf("telephony_deinit failed");
                return false;
@@ -323,17 +240,6 @@ static bool _get_sim_phone_number(int sim_number, char *str, int str_size)
        return true;
 }
 
-
-static MENURUN_ERROR menufunc_ad_sim1(int argc, char *argv[], GenSettingMenu *menu)
-{
-       char str[MAX_DISPLAY_STR_LEN_ON_PHONE_INFO];
-       if (!_get_sim_phone_number(1, str, sizeof(str)))
-               return MENURUN_ERR_NORMAL;
-       printf("\tSIM1 Phone Number : ");
-       printf("%s\n", str);
-
-       return MENURUN_ERR_NONE;
-}
 static MENURUN_ERROR menugetstring_ad_sim1(string &resultstr, int argc, char *argv[], GenSettingMenu *menu)
 {
        char str[MAX_DISPLAY_STR_LEN_ON_PHONE_INFO];
@@ -344,16 +250,6 @@ static MENURUN_ERROR menugetstring_ad_sim1(string &resultstr, int argc, char *ar
        return MENURUN_ERR_NONE;
 };
 
-static MENURUN_ERROR menufunc_ad_sim2(int argc, char *argv[], GenSettingMenu *menu)
-{
-       char str[MAX_DISPLAY_STR_LEN_ON_PHONE_INFO];
-       if (!_get_sim_phone_number(2, str, sizeof(str)))
-               return MENURUN_ERR_NORMAL;
-       printf("\tSIM2 Phone Number : ");
-       printf("%s\n", str);
-
-       return MENURUN_ERR_NONE;
-}
 static MENURUN_ERROR menugetstring_ad_sim2(string &resultstr, int argc, char *argv[], GenSettingMenu *menu)
 {
        char str[MAX_DISPLAY_STR_LEN_ON_PHONE_INFO];
@@ -368,8 +264,6 @@ static MENURUN_ERROR menugetstring_ad_sim2(string &resultstr, int argc, char *ar
 static bool _get_sim_ime(int sim_number, char * str, int str_size)
 {
        telephony_handle_list_s tel_h_list;
-       char *ime_buffer = nullptr;
-
        if (telephony_init(&tel_h_list) != TELEPHONY_ERROR_NONE) {
                eprintf("telephony_init failed");
                return false;
@@ -384,6 +278,7 @@ static bool _get_sim_ime(int sim_number, char * str, int str_size)
                return false;
        }
 
+       char *ime_buffer = nullptr;
        int ret = telephony_modem_get_imei(tel_h_list.handle[sim_number-1], &ime_buffer);
        if (ret != 0 || ime_buffer == nullptr) {
                printf("\tThere is no SIM%d card, ret:%d ime_buffer addr:0x%x\n", sim_number, ret, (unsigned int)ime_buffer);
@@ -404,16 +299,6 @@ static bool _get_sim_ime(int sim_number, char * str, int str_size)
        return true;
 }
 
-static MENURUN_ERROR menufunc_ad_ime1(int argc, char *argv[], GenSettingMenu *menu)
-{
-       char str[MAX_DISPLAY_STR_LEN_ON_PHONE_INFO];
-       if (!_get_sim_ime(1, str, sizeof(str)))
-               return MENURUN_ERR_NORMAL;
-       printf("\tIME1 : ");
-       printf("%s\n", str);
-
-       return MENURUN_ERR_NONE;
-}
 static MENURUN_ERROR menugetstring_ad_ime1(string &resultstr, int argc, char *argv[], GenSettingMenu *menu)
 {
        char str[MAX_DISPLAY_STR_LEN_ON_PHONE_INFO];
@@ -424,16 +309,6 @@ static MENURUN_ERROR menugetstring_ad_ime1(string &resultstr, int argc, char *ar
        return MENURUN_ERR_NONE;
 };
 
-static MENURUN_ERROR menufunc_ad_ime2(int argc, char *argv[], GenSettingMenu *menu)
-{
-       char str[MAX_DISPLAY_STR_LEN_ON_PHONE_INFO];
-       if (!_get_sim_ime(2, str, sizeof(str)))
-               return MENURUN_ERR_NORMAL;
-       printf("\tIME2 : ");
-       printf("%s\n", str);
-
-       return MENURUN_ERR_NONE;
-}
 static MENURUN_ERROR menugetstring_ad_ime2(string &resultstr, int argc, char *argv[], GenSettingMenu *menu)
 {
        char str[MAX_DISPLAY_STR_LEN_ON_PHONE_INFO];
@@ -444,34 +319,6 @@ static MENURUN_ERROR menugetstring_ad_ime2(string &resultstr, int argc, char *ar
        return MENURUN_ERR_NONE;
 };
 
-static MENURUN_ERROR menufunc_ad_bta(int argc, char *argv[], GenSettingMenu *menu)
-{
-       int ret = bt_initialize();
-       char *local_address = nullptr;
-       ret = bt_adapter_get_address(&local_address);
-
-       bt_deinitialize();
-
-       if (ret != BT_ERROR_NONE) {
-               char errmsg[200];
-               if (ret == BT_ERROR_NOT_ENABLED)
-                       snprintf(errmsg, sizeof(errmsg), "Bluetooth is disabled!\n");
-               else
-                       snprintf(errmsg, sizeof(errmsg), "Bluetooth is unavailable!\n");
-               eprintf("Fail to get BT address, error code: %s\n", errmsg);
-               free(local_address);
-               return MENURUN_ERR_NORMAL;
-       }
-       
-       char str[MAX_DISPLAY_STR_LEN_ON_PHONE_INFO];
-       snprintf(str, sizeof(str), "%s", local_address);
-       free(local_address);
-       printf("\tBluetooth address : ");
-       printf("%s\n", str);
-
-       return MENURUN_ERR_NONE;
-}
-
 static MENURUN_ERROR menugetstring_ad_bta(string &resultstr, int argc, char *argv[], GenSettingMenu *menu)
 {
        int ret = bt_initialize();
@@ -499,39 +346,6 @@ static MENURUN_ERROR menugetstring_ad_bta(string &resultstr, int argc, char *arg
        return MENURUN_ERR_NONE;
 };
 
-static MENURUN_ERROR menufunc_ad_wfa(int argc, char *argv[], GenSettingMenu *menu)
-{
-       wifi_manager_h wifi_handle = nullptr;
-       char *mac_addr = nullptr;
-
-       /* get wifi handle */
-       int ret = wifi_manager_initialize(&wifi_handle);
-       if (ret != WIFI_MANAGER_ERROR_NONE)
-               return MENURUN_ERR_NORMAL;
-
-       /* get wifi mac address */
-       ret = wifi_manager_get_mac_address(wifi_handle, &mac_addr);
-       if (ret != WIFI_MANAGER_ERROR_NONE && mac_addr) {
-               char errmsg[200];
-               snprintf(errmsg, sizeof(errmsg), "WIFI is unavailable!\n");
-               eprintf("Fail to get WIFI address, error code: %s\n", errmsg);
-               return MENURUN_ERR_NORMAL;
-       }
-
-       char str[MAX_DISPLAY_STR_LEN_ON_PHONE_INFO];
-       snprintf(str, sizeof(str), "%s", mac_addr);
-       free(mac_addr);
-
-       /* release wifi handle */
-       ret = wifi_manager_deinitialize(wifi_handle);
-       if (ret != WIFI_MANAGER_ERROR_NONE)
-               return MENURUN_ERR_NORMAL;
-
-       printf("\tWIFI address : ");
-       printf("%s\n", str);
-
-       return MENURUN_ERR_NONE;
-}
 static MENURUN_ERROR menugetstring_ad_wfa(string &resultstr, int argc, char *argv[], GenSettingMenu *menu)
 {
        wifi_manager_h wifi_handle = nullptr;
@@ -565,45 +379,6 @@ static MENURUN_ERROR menugetstring_ad_wfa(string &resultstr, int argc, char *arg
        return MENURUN_ERR_NONE;
 };
 
-static MENURUN_ERROR menufunc_ad_sto(int argc, char *argv[], GenSettingMenu *menu)
-{
-       long long int total = 0;
-       long long int avail = 0;
-       struct statvfs internal;
-       struct statvfs external;
-
-       if (storage_get_internal_memory_size(&internal) < 0) {
-               eprintf("Fail to get internal memory size\n");
-               return MENURUN_ERR_NORMAL;
-       }
-       total = (long long int)internal.f_frsize
-                       * (long long int)internal.f_blocks;
-       avail = (long long int)internal.f_bsize
-                       * (long long int)internal.f_bavail;
-
-       if (storage_get_external_memory_size(&external) < 0) {
-               eprintf("Fail to get external memory size\n");
-       } else {
-               total += (long long int)external.f_frsize
-                               * (long long int)external.f_blocks;
-               avail += (long long int)external.f_bsize
-                               * (long long int)external.f_bavail;
-       }
-
-       /* check if values are appropriate*/
-       if (total < 0 || avail < 0 || total < avail) {
-               eprintf("Fail to get internal memory size\n");
-               return MENURUN_ERR_NORMAL;
-       }
-
-       char str[MAX_DISPLAY_STR_LEN_ON_PHONE_INFO];
-       snprintf(str, sizeof(str), "%0.1fGB available (Total %0.1fGB)",
-               (double)(avail / 1000000000.0), (double)(total / 1000000000.0));
-       printf("\tStorage Size : ");
-       printf("%s\n", str);
-
-       return MENURUN_ERR_NONE;
-}
 static MENURUN_ERROR menugetstring_ad_sto(string &resultstr, int argc, char *argv[], GenSettingMenu *menu)
 {
        long long int total = 0;
@@ -644,39 +419,9 @@ static MENURUN_ERROR menugetstring_ad_sto(string &resultstr, int argc, char *arg
        return MENURUN_ERR_NONE;
 };
 
-static MENURUN_ERROR menufunc_ad_batt(int argc, char *argv[], GenSettingMenu *menu)
-{
-       char file[MAX_DISPLAY_STR_LEN_ON_PHONE_INFO] = { 0, };
-       snprintf(file, MAX_DISPLAY_STR_LEN_ON_PHONE_INFO, "%s/%s/%s",
-               SETTING_ABOUT_POWER_SUPPLY_PATH, "battery", "capacity");
-
-       int fd = open(file, O_RDONLY);
-       if (fd == -1) {
-               eprintf("open file fail\n");
-               return MENURUN_ERR_NORMAL;
-       }
-
-       char buf[MAX_DISPLAY_STR_LEN_ON_PHONE_INFO] = { 0, };
-       int r = read(fd, buf, MAX_DISPLAY_STR_LEN_ON_PHONE_INFO);
-       if (r < 0 || r >= MAX_DISPLAY_STR_LEN_ON_PHONE_INFO) {
-               eprintf("read file fail\n");
-               return MENURUN_ERR_NORMAL;
-       }
-       close(fd);
-
-       buf[r] = '\0';
-       int val = atoi(buf);
-
-       char str[MAX_DISPLAY_STR_LEN_ON_PHONE_INFO];
-       snprintf(str, sizeof(str), "%d%%", val);
-       printf("\tBattery percentage : ");
-       printf("%s\n", str);
-
-       return MENURUN_ERR_NONE;
-}
 static MENURUN_ERROR menugetstring_ad_batt(string &resultstr, int argc, char *argv[], GenSettingMenu *menu)
 {
-       char file[MAX_DISPLAY_STR_LEN_ON_PHONE_INFO] = { 0, };
+       char file[MAX_DISPLAY_STR_LEN_ON_PHONE_INFO] = { '\0', };
        snprintf(file, MAX_DISPLAY_STR_LEN_ON_PHONE_INFO, "%s/%s/%s",
                SETTING_ABOUT_POWER_SUPPLY_PATH, "battery", "capacity");
 
@@ -686,19 +431,21 @@ static MENURUN_ERROR menugetstring_ad_batt(string &resultstr, int argc, char *ar
                return MENURUN_ERR_NORMAL;
        }
 
-       char buf[MAX_DISPLAY_STR_LEN_ON_PHONE_INFO] = { 0, };
+       char buf[MAX_DISPLAY_STR_LEN_ON_PHONE_INFO] = { '\0', };
        int r = read(fd, buf, MAX_DISPLAY_STR_LEN_ON_PHONE_INFO);
        if (r < 0 || r >= MAX_DISPLAY_STR_LEN_ON_PHONE_INFO) {
                eprintf("read file fail\n");
+               close(fd);
                return MENURUN_ERR_NORMAL;
        }
+       buf[r] = '\0';
        close(fd);
 
-       buf[r] = '\0';
+       // remove escape char except for number
        int val = atoi(buf);
-
        char str[MAX_DISPLAY_STR_LEN_ON_PHONE_INFO];
-       snprintf(str, sizeof(str), "%d%%", val);
+       snprintf(str, sizeof(str), "%d %%", val);
+
        resultstr = str;
 
        return MENURUN_ERR_NONE;
@@ -722,14 +469,14 @@ static int __stat_get_cpu_usage(float *usr_pct, float *sys_pct)
                cpu_num = 1;
 
        gettimeofday(&cur_tv, nullptr);
-       
+
        FILE *fp = fopen(SETTING_ABOUT_STAT_PATH, "r");
        if (fp == nullptr) {
                eprintf("fp == nullptr\n");
                return -ENOENT;
        }
 
-       char cpu_info[MAX_COMMON_BUFFER_LEN] = { 0, };
+       char cpu_info[MAX_COMMON_BUFFER_LEN] = {'\0', };
        char *cpu_info_p = fgets(cpu_info, MAX_COMMON_BUFFER_LEN, fp);
 
        if (cpu_info_p == nullptr) {
@@ -801,23 +548,7 @@ static int __stat_get_cpu_usage(float *usr_pct, float *sys_pct)
 
        return 0;
 }
-static MENURUN_ERROR menufunc_ad_cpuu(int argc, char *argv[], GenSettingMenu *menu)
-{
-       printf("\tWhen you want to stop checking CPU usage,\n");
-       printf("\tPress Ctrl+C\n\n");
-
-       while(1) {
-               float usr = 0.0, sys = 0.0;
-               int result = __stat_get_cpu_usage(&usr, &sys);
-               if (result != 0)
-                       continue;
-               printf("\tCPU usage : ");
-               printf("%.0f%%\n", usr + sys);
-               sleep(1);
-       }
 
-       return MENURUN_ERR_NONE;
-}
 static MENURUN_ERROR menugetstring_ad_cpuu(string &resultstr, int argc, char *argv[], GenSettingMenu *menu)
 {
        const int repeatcout = 4;
@@ -841,21 +572,21 @@ static MENURUN_ERROR menugetstring_ad_cpuu(string &resultstr, int argc, char *ar
 }
 
 MenuItem ad_menulist[] ={
-       {"name", "Name", menufunc_ad_name, nullptr, menugetstring_ad_name},
-       {"mn", "Model number", menufunc_ad_mn, nullptr, menugetstring_ad_mn},
-       {"ver", "Tizen version", menufunc_ad_ver, nullptr, menugetstring_ad_ver},
-       {"cpu", "CPU", menufunc_ad_cpu, nullptr, menugetstring_ad_cpu},
-       {"ram", "RAM", menufunc_ad_ram, nullptr, menugetstring_ad_ram},
-       {"res", "Resolution", menufunc_ad_res, nullptr, menugetstring_ad_res},
-       {"sim1", "SIM1 Phone Number", menufunc_ad_sim1, nullptr, menugetstring_ad_sim1},
-       {"sim2", "SIM2 Phone Number", menufunc_ad_sim2, nullptr, menugetstring_ad_sim2},
-       {"ime1", "IME1", menufunc_ad_ime1, nullptr, menugetstring_ad_ime1},
-       {"ime2", "IME2", menufunc_ad_ime2, nullptr, menugetstring_ad_ime2},
-       {"bta", "Bluetooth address", menufunc_ad_bta, nullptr, menugetstring_ad_bta},
-       {"wfa", "Wi-Fi MAC Address", menufunc_ad_wfa, nullptr, menugetstring_ad_wfa},
-       {"sto", "Storage", menufunc_ad_sto, nullptr, menugetstring_ad_sto},
-       {"batt", "Battery percentage", menufunc_ad_batt, nullptr, menugetstring_ad_batt},
-       {"cpuu", "CPU usage", menufunc_ad_cpuu, nullptr, menugetstring_ad_cpuu}
+       {"name", "Name", menugetstring_ad_name, nullptr, nullptr},
+       {"mn", "Model number", menugetstring_ad_mn, nullptr, nullptr},
+       {"ver", "Tizen version", menugetstring_ad_ver, nullptr, nullptr},
+       {"cpu", "CPU", menugetstring_ad_cpu, nullptr, nullptr},
+       {"ram", "RAM", menugetstring_ad_ram, nullptr, nullptr},
+       {"res", "Resolution", menugetstring_ad_res, nullptr, nullptr},
+       {"sim1", "SIM1 Phone Number", menugetstring_ad_sim1, nullptr, nullptr},
+       {"sim2", "SIM2 Phone Number", menugetstring_ad_sim2, nullptr, nullptr},
+       {"ime1", "IME1", menugetstring_ad_ime1, nullptr, nullptr},
+       {"ime2", "IME2", menugetstring_ad_ime2, nullptr, nullptr},
+       {"bta", "Bluetooth address", menugetstring_ad_bta, nullptr, nullptr},
+       {"wfa", "Wi-Fi MAC Address", menugetstring_ad_wfa, nullptr, nullptr},
+       {"sto", "Storage", menugetstring_ad_sto, nullptr, nullptr},
+       {"batt", "Battery percentage", menugetstring_ad_batt, nullptr, nullptr},
+       {"cpuu", "CPU usage", menugetstring_ad_cpuu, nullptr, nullptr}
 };
 
 void precreate_ad(GenSettingMenu *menu)
index 4f31714..135040b 100644 (file)
 
 using namespace std;
 
-static MENURUN_ERROR menufunc_am_st(int argc, char *argv[], GenSettingMenu *menu)
-{
-    bool ret_state = false;
-
-       int ret = system_settings_get_value_bool(SYSTEM_SETTINGS_KEY_NETWORK_FLIGHT_MODE, &ret_state);
-       if (ret != 0) {
-               eprintf("\tFlight mode get fail\n");
-               return MENURUN_ERR_NORMAL;
-    }
-
-       printf("\tFlight mode : %s\n", ret_state? "On" : "Off");
-    return MENURUN_ERR_NONE;
-}
 static MENURUN_ERROR menugetstring_am_st(string &resultstr, int argc, char *argv[], GenSettingMenu *menu)
 {
        bool ret_state = false;
 
        int ret = system_settings_get_value_bool(SYSTEM_SETTINGS_KEY_NETWORK_FLIGHT_MODE, &ret_state);
-       if (ret != 0) {
+       if (ret != SYSTEM_SETTINGS_ERROR_NONE) {
                eprintf("\tFlight mode get fail\n");
                return MENURUN_ERR_NORMAL;
        }
@@ -38,7 +25,7 @@ static MENURUN_ERROR menugetstring_am_st(string &resultstr, int argc, char *argv
 }
 
 MenuItem am_menulist[] ={
-       {"st", "State", menufunc_am_st, nullptr, menugetstring_am_st}
+       {"st", "State", menugetstring_am_st, nullptr, nullptr}
 };
 
 void precreate_am(GenSettingMenu *menu)
index 381940f..03726a1 100644 (file)
@@ -194,22 +194,22 @@ static MENURUN_ERROR menufunc_apps_lall(int argc, char *argv[], GenSettingMenu *
 
 
 MenuItem apps_menulist[] ={
-       {"lall", "List Apps - All", menufunc_apps_lall},
-       {"ldn", "List Apps - Downloads", nullptr},
-       {"lrun", "List Apps - Running", nullptr},
-       {"loff", "List Apps - Turned off", nullptr},
-       {"ainfo", "Application Info", nullptr},
-       {"astor", "App - Storage", nullptr},
-       {"afc", "App - Force close", nullptr},
-       {"aoff", "App - Turn off", nullptr},
-       {"astop", "App - Stop", nullptr},
-       {"aclc", "App - Clear cache", nullptr},
-       {"lhome", "List Homes", nullptr},
-       {"home", "Home", nullptr},
-       {"ldef", "List Defaults", nullptr},
-       {"cldef", "Clear Defaults", nullptr},
-       {"setdef", "Set Defaults", nullptr},
-       {"usetdef", "Unset Defaults", nullptr}
+       {"lall", "List Apps - All", nullptr, menufunc_apps_lall, nullptr},
+       {"ldn", "List Apps - Downloads", nullptr, nullptr, nullptr},
+       {"lrun", "List Apps - Running", nullptr, nullptr, nullptr},
+       {"loff", "List Apps - Turned off", nullptr, nullptr, nullptr},
+       {"ainfo", "Application Info", nullptr, nullptr, nullptr},
+       {"astor", "App - Storage", nullptr, nullptr, nullptr},
+       {"afc", "App - Force close", nullptr, nullptr, nullptr},
+       {"aoff", "App - Turn off", nullptr, nullptr, nullptr},
+       {"astop", "App - Stop", nullptr, nullptr, nullptr},
+       {"aclc", "App - Clear cache", nullptr, nullptr, nullptr},
+       {"lhome", "List Homes", nullptr, nullptr, nullptr},
+       {"home", "Home", nullptr, nullptr, nullptr},
+       {"ldef", "List Defaults", nullptr, nullptr, nullptr},
+       {"cldef", "Clear Defaults", nullptr, nullptr, nullptr},
+       {"setdef", "Set Defaults", nullptr, nullptr, nullptr},
+       {"usetdef", "Unset Defaults", nullptr, nullptr, nullptr}
 };
 
 void precreate_apps(GenSettingMenu *menu)
index 4f4abca..541052c 100644 (file)
@@ -103,7 +103,7 @@ MENURUN_ERROR menufunc_ba(int argc, char *argv[], GenSettingMenu *menu)
                if ((r >= 0) && (r < MAX_DISPLAY_STR_LEN_ON_PHONE_INFO)) {
                        buf[r] = '\0';
                        val = atoi(buf);
-                       snprintf(str, sizeof(str), "%d%%", val);
+                       snprintf(str, sizeof(str), "%d %%", val);
                } else {
                        eprintf("read file fail\n");
                        snprintf(str, sizeof(str), "%s unavailable!\n", SETTING_ABOUT_POWER_SUPPLY_PATH);
index dc34c08..21c7c2c 100644 (file)
 
 using namespace std;
 
-static MENURUN_ERROR menufunc_bt_st(int argc, char *argv[], GenSettingMenu *menu)
-{
-       if (bt_initialize() != BT_ERROR_NONE) {
-               eprintf("bt_initialize() failed\n");
-               return MENURUN_ERR_NORMAL;
-       }
-       bt_adapter_state_e bt_state;
-       if (bt_adapter_get_state(&bt_state) != BT_ERROR_NONE) {
-               eprintf("bt_adapter_get_state() failed\n");
-               bt_deinitialize();
-               return MENURUN_ERR_NORMAL;
-       }
-       bt_deinitialize();
-
-       printf("\tState : %s\n", (bt_state == BT_ADAPTER_ENABLED) ? "On" : "Off");
-    return MENURUN_ERR_NONE;
-}
 static MENURUN_ERROR menugetstring_bt_st(string &resultstr, int argc, char *argv[], GenSettingMenu *menu)
 {
        if (bt_initialize() != BT_ERROR_NONE) {
@@ -47,7 +30,7 @@ static MENURUN_ERROR menugetstring_bt_st(string &resultstr, int argc, char *argv
 }
 
 MenuItem bt_menulist[] ={
-       {"st", "State", menufunc_bt_st, nullptr, menugetstring_bt_st}
+       {"st", "State", menugetstring_bt_st, nullptr, nullptr}
 };
 
 void precreate_bt(GenSettingMenu *menu)
index 1fd8dae..87d7c57 100644 (file)
@@ -61,7 +61,7 @@ typedef enum {
        RESTRICTIONS_ERROR = 1
 } restrictions_result;
 
-static char *number_postfixes[] = {
+static const char *number_postfixes[] = {
        "th", "st", "nd", "rd"
 };
 
@@ -1123,10 +1123,10 @@ static MENURUN_ERROR menufunc_du_wi(int argc, char *argv[], GenSettingMenu *menu
 }
 
 MenuItem du_menulist[] ={
-       {"smwl", "Set mobile data usage limit, warning level", menufunc_du_smwl, nullptr},
-       {"gmwl", "Get mobile data usage limit, warning level", menufunc_du_gmwl, nullptr},
-       {"mo", "Mobile data usage", menufunc_du_mo, nullptr},
-       {"wi", "Wifi data usage", menufunc_du_wi, nullptr},
+       {"smwl", "Set mobile data usage limit, warning level", nullptr, menufunc_du_smwl, nullptr},
+       {"gmwl", "Get mobile data usage limit, warning level", nullptr, menufunc_du_gmwl, nullptr},
+       {"mo", "Mobile data usage", nullptr, menufunc_du_mo, nullptr},
+       {"wi", "Wifi data usage", nullptr, menufunc_du_wi, nullptr},
 };
 
 void precreate_du(GenSettingMenu *menu)
index e34af95..b4155df 100644 (file)
 
 using namespace std;
 
-static MENURUN_ERROR menufunc_dt_au(int argc, char *argv[], GenSettingMenu *menu)
-{
-       int auto_update = 0;
-
-       int ret = vconf_get_bool(VCONFKEY_SETAPPL_STATE_AUTOMATIC_TIME_UPDATE_BOOL, &auto_update);
-       if (ret != 0) {
-               eprintf("\tFail to get Auto update!\n");
-               return MENURUN_ERR_NORMAL;
-       }
-
-       printf("\tAuto update : ");
-       printf("%s\n", auto_update? "On" : "Off");
-
-       return MENURUN_ERR_NONE;
-}
 static MENURUN_ERROR menugetstring_dt_au(string &resultstr, int argc, char *argv[], GenSettingMenu *menu)
 {
        int auto_update = 0;
@@ -43,11 +28,11 @@ static MENURUN_ERROR menugetstring_dt_au(string &resultstr, int argc, char *argv
 
 
 MenuItem dt_menulist[] ={
-       {"au", "Auto update", menufunc_dt_au, nullptr, menugetstring_dt_au},
-       {"date", "Date", nullptr, nullptr},
-       {"time", "Time", nullptr, nullptr},
-       {"tz", "Time zone", nullptr, nullptr},
-       {"24h", "24 hour", nullptr, nullptr}
+       {"au", "Auto update", menugetstring_dt_au, nullptr, nullptr},
+       {"date", "Date", nullptr, nullptr, nullptr},
+       {"time", "Time", nullptr, nullptr, nullptr},
+       {"tz", "Time zone", nullptr, nullptr, nullptr},
+       {"24h", "24 hour", nullptr, nullptr, nullptr}
 };
 
 void precreate_dt(GenSettingMenu *menu)
index b032af6..020700f 100644 (file)
 
 using namespace std;
 
-static MENURUN_ERROR menufunc_di_ab(int argc, char *argv[], GenSettingMenu *menu)
-{
-       int auto_bright = 0;
-    // TODO : I can't find BRIGHTNESS_AUTOMATIC value at device_display apis
-       int ret = vconf_get_int(VCONFKEY_SETAPPL_BRIGHTNESS_AUTOMATIC_INT, &auto_bright);
-    if( ret != 0 ) {
-        eprintf("\tAuto Bright get fail!\n");
-               return MENURUN_ERR_NORMAL;
-    }
-
-       printf("\tAuto Bright : %s\n", auto_bright? "On" : "Off");
-       return MENURUN_ERR_NONE;
-}
 static MENURUN_ERROR menugetstring_di_ab(string &resultstr, int argc, char *argv[], GenSettingMenu *menu)
 {
        int auto_bright = 0;
@@ -37,19 +24,6 @@ static MENURUN_ERROR menugetstring_di_ab(string &resultstr, int argc, char *argv
        return MENURUN_ERR_NONE;
 }
 
-static MENURUN_ERROR menufunc_di_br(int argc, char *argv[], GenSettingMenu *menu)
-{
-       int brightness = 0;
-
-       int ret = vconf_get_int(VCONFKEY_SETAPPL_LCD_BRIGHTNESS, &brightness);
-    if(ret != 0) {
-        eprintf("\tbrightness get fail!\n");
-               return MENURUN_ERR_NORMAL;
-    }
-
-       printf("\tBrightness : %d\n", brightness);
-       return MENURUN_ERR_NONE;
-}
 static MENURUN_ERROR menugetstring_di_br(string &resultstr, int argc, char *argv[], GenSettingMenu *menu)
 {
        int brightness = 0;
@@ -66,19 +40,6 @@ static MENURUN_ERROR menugetstring_di_br(string &resultstr, int argc, char *argv
        return MENURUN_ERR_NONE;
 }
 
-static MENURUN_ERROR menufunc_di_fs(int argc, char *argv[], GenSettingMenu *menu)
-{
-    int fontsize = 0;
-
-    int ret = system_settings_get_value_int(SYSTEM_SETTINGS_KEY_FONT_SIZE, &fontsize);
-    if( ret != 0) {
-        eprintf("\tFont get fail!\n");
-               return MENURUN_ERR_NORMAL;
-    }
-
-       printf("\tFont Size: %d\n", fontsize );
-    return MENURUN_ERR_NONE;
-}
 static MENURUN_ERROR menugetstring_di_fs(string &resultstr, int argc, char *argv[], GenSettingMenu *menu)
 {
        int fontsize = 0;
@@ -95,25 +56,12 @@ static MENURUN_ERROR menugetstring_di_fs(string &resultstr, int argc, char *argv
        return MENURUN_ERR_NONE;
 }
 
-static MENURUN_ERROR menufunc_di_to(int argc, char *argv[], GenSettingMenu *menu)
-{
-       int timeout = 0;
-       vconf_get_int(VCONFKEY_SETAPPL_LCD_TIMEOUT_NORMAL, &timeout);
-
-       printf("\tScreen timeout : ");
-       if (timeout)
-               printf("Always\n");
-       else
-               printf("%d sec\n", timeout);
-
-       return MENURUN_ERR_NONE;
-}
 static MENURUN_ERROR menugetstring_di_to(string &resultstr, int argc, char *argv[], GenSettingMenu *menu)
 {
        int timeout = 0;
        vconf_get_int(VCONFKEY_SETAPPL_LCD_TIMEOUT_NORMAL, &timeout);
 
-       if (timeout) {
+       if (timeout == 0) {
                resultstr = "Always";
        }
        else {
@@ -126,10 +74,10 @@ static MENURUN_ERROR menugetstring_di_to(string &resultstr, int argc, char *argv
 }
 
 MenuItem di_menulist[] ={
-       {"ab", "Auto Bright", menufunc_di_ab, nullptr, menugetstring_di_ab},
-       {"br", "Brightness", menufunc_di_br, nullptr, menugetstring_di_br},
-       {"fs", "Font Size", menufunc_di_fs, nullptr, menugetstring_di_fs},
-       {"to", "Screen timeout", menufunc_di_to, nullptr, menugetstring_di_to}
+       {"ab", "Auto Bright", menugetstring_di_ab, nullptr, nullptr},
+       {"br", "Brightness", menugetstring_di_br, nullptr, nullptr},
+       {"fs", "Font Size", menugetstring_di_fs, nullptr, nullptr},
+       {"to", "Screen timeout", menugetstring_di_to, nullptr, nullptr}
 };
 
 void precreate_di(GenSettingMenu *menu)
index af6410f..2f37aa2 100644 (file)
@@ -71,24 +71,6 @@ static void _sound_manager_deinit(sound_stream_info_t *info)
 
 ///////////////////////////////////////////////////////////////////////
 
-static MENURUN_ERROR menufunc_so_rvol(int argc, char *argv[], GenSettingMenu *menu)
-{
-       sound_stream_info_t sound_stream_info;
-       if (!_sound_manager_init(&sound_stream_info)) {
-               eprintf("_sound_manager_init() failed!");
-               return MENURUN_ERR_NORMAL;
-       }
-       int vol = 0;
-       if (sound_manager_get_volume(SOUND_TYPE_RINGTONE, &vol) != SOUND_MANAGER_ERROR_NONE) {
-               _sound_manager_deinit(&sound_stream_info);
-               return MENURUN_ERR_NORMAL;
-       }
-       _sound_manager_deinit(&sound_stream_info);
-
-       printf("\tRingtone Volume : ");
-       printf("%d\n", vol);
-       return MENURUN_ERR_NONE;
-}
 static MENURUN_ERROR menugetstring_so_rvol(string &resultstr, int argc, char *argv[], GenSettingMenu *menu)
 {
        sound_stream_info_t sound_stream_info;
@@ -109,24 +91,6 @@ static MENURUN_ERROR menugetstring_so_rvol(string &resultstr, int argc, char *ar
        return MENURUN_ERR_NONE;
 }
 
-static MENURUN_ERROR menufunc_so_nvol(int argc, char *argv[], GenSettingMenu *menu)
-{
-       sound_stream_info_t sound_stream_info;
-       if (!_sound_manager_init(&sound_stream_info)) {
-               eprintf("_sound_manager_init() failed!");
-               return MENURUN_ERR_NORMAL;
-       }
-       int vol = 0;
-       if (sound_manager_get_volume(SOUND_TYPE_NOTIFICATION, &vol) != SOUND_MANAGER_ERROR_NONE) {
-               _sound_manager_deinit(&sound_stream_info);
-               return MENURUN_ERR_NORMAL;
-       }
-       _sound_manager_deinit(&sound_stream_info);
-
-       printf("\tNotification Volume : ");
-       printf("%d\n", vol);
-       return MENURUN_ERR_NONE;
-}
 static MENURUN_ERROR menugetstring_so_nvol(string &resultstr, int argc, char *argv[], GenSettingMenu *menu)
 {
        sound_stream_info_t sound_stream_info;
@@ -147,24 +111,6 @@ static MENURUN_ERROR menugetstring_so_nvol(string &resultstr, int argc, char *ar
        return MENURUN_ERR_NONE;
 }
 
-static MENURUN_ERROR menufunc_so_mvol(int argc, char *argv[], GenSettingMenu *menu)
-{
-       sound_stream_info_t sound_stream_info;
-       if (!_sound_manager_init(&sound_stream_info)) {
-               eprintf("_sound_manager_init() failed!");
-               return MENURUN_ERR_NORMAL;
-       }
-       int vol = 0;
-       if (sound_manager_get_volume(SOUND_TYPE_MEDIA, &vol) != SOUND_MANAGER_ERROR_NONE) {
-               _sound_manager_deinit(&sound_stream_info);
-               return MENURUN_ERR_NORMAL;
-       }
-       _sound_manager_deinit(&sound_stream_info);
-
-       printf("\tMedia Volume : ");
-       printf("%d\n", vol);
-       return MENURUN_ERR_NONE;
-}
 static MENURUN_ERROR menugetstring_so_mvol(string &resultstr, int argc, char *argv[], GenSettingMenu *menu)
 {
        sound_stream_info_t sound_stream_info;
@@ -185,25 +131,6 @@ static MENURUN_ERROR menugetstring_so_mvol(string &resultstr, int argc, char *ar
        return MENURUN_ERR_NONE;
 }
 
-static MENURUN_ERROR menufunc_so_svol(int argc, char *argv[], GenSettingMenu *menu)
-{
-       sound_stream_info_t sound_stream_info;
-       if (!_sound_manager_init(&sound_stream_info)) {
-               eprintf("_sound_manager_init() failed!");
-               return MENURUN_ERR_NORMAL;
-       }
-
-       int vol = 0;
-       if (sound_manager_get_volume(SOUND_TYPE_SYSTEM, &vol) != SOUND_MANAGER_ERROR_NONE) {
-               _sound_manager_deinit(&sound_stream_info);
-               return MENURUN_ERR_NORMAL;
-       }
-       _sound_manager_deinit(&sound_stream_info);
-
-       printf("\tSystem Volume : ");
-       printf("%d\n", vol);
-       return MENURUN_ERR_NONE;
-}
 static MENURUN_ERROR menugetstring_so_svol(string &resultstr, int argc, char *argv[], GenSettingMenu *menu)
 {
        sound_stream_info_t sound_stream_info;
@@ -229,10 +156,10 @@ static MENURUN_ERROR menugetstring_so_svol(string &resultstr, int argc, char *ar
 
 
 MenuItem so_menulist[] ={
-       {"rvol", "Ringtone Volume", menufunc_so_rvol, nullptr, menugetstring_so_rvol},
-       {"nvol", "Notification Volume", menufunc_so_nvol, nullptr, menugetstring_so_nvol},
-       {"mvol", "Media Volume", menufunc_so_mvol, nullptr, menugetstring_so_mvol},
-       {"svol", "System Volume", menufunc_so_svol, nullptr, menugetstring_so_svol}
+       {"rvol", "Ringtone Volume", menugetstring_so_rvol, nullptr, nullptr},
+       {"nvol", "Notification Volume", menugetstring_so_nvol, nullptr, nullptr},
+       {"mvol", "Media Volume", menugetstring_so_mvol, nullptr, nullptr},
+       {"svol", "System Volume", menugetstring_so_svol, nullptr, nullptr}
 };
 
 void precreate_so(GenSettingMenu *menu)
index 56a334d..6b0cd58 100644 (file)
@@ -54,12 +54,11 @@ static char *usb_otg_path = 0;
 } while (0)
 #endif
 
-void storage_get_external_storage_status(char *path, double *total,
+static void storage_get_external_storage_status(const char *path, double *total,
                double *avail)
 {
        struct statvfs s;
 
-
        if (!storage_get_external_memory_size(&s)) {
                *total = (double)s.f_frsize * s.f_blocks;
                *avail = (double)s.f_bfree * s.f_frsize;
@@ -505,7 +504,7 @@ static MENURUN_ERROR menufunc_st_sd(int argc, char *argv[], GenSettingMenu *menu
                return MENURUN_ERR_NORMAL;
        }
 
-       char *MMC_path = _TZ_SYS_MEDIA"/sdcard";
+       const char *MMC_path = _TZ_SYS_MEDIA"/sdcard";
 
        storage_get_external_storage_status(MMC_path, &total, &avail);
        storage_size_to_str(total, total_str, sizeof(total_str));
@@ -708,19 +707,19 @@ static MENURUN_ERROR menufunc_st_ds(int argc, char *argv[], GenSettingMenu *menu
 }
 
 MenuItem st_menulist[] ={
-       {"me", "Device memory storage info", menufunc_st_me},
-       {"sd", "SD card storage info", menufunc_st_sd},
-       {"ds", "Get default storage shared content", menufunc_st_ds},
-       {"da", "Get default storage app installation", menufunc_st_da},
-       {"ids", "Internal memory for default storage shared content", menufunc_st_ids},
-       {"sds", "SD Card for default storage shared content", menufunc_st_sds},
-       {"ida", "Internal memory for Default storage app installation", menufunc_st_ida},
-       {"sda", "SD Card for Default storage app installation", menufunc_st_sda},
-       {"mss", "Get Mount SD card status", menufunc_st_mss},
-       {"ms", "Mount SD card", menufunc_st_ms},
-       {"us", "Unmount SD card", menufunc_st_us},
-       {"fs", "Format SD card", menufunc_st_fs},
-       {"fsp", "Format SD card as device storage", nullptr}
+       {"me", "Device memory storage info", nullptr, menufunc_st_me, nullptr},
+       {"sd", "SD card storage info", nullptr, menufunc_st_sd, nullptr},
+       {"ds", "Get default storage shared content", nullptr, menufunc_st_ds, nullptr},
+       {"da", "Get default storage app installation", nullptr, menufunc_st_da, nullptr},
+       {"ids", "Internal memory for default storage shared content", nullptr, menufunc_st_ids, nullptr},
+       {"sds", "SD Card for default storage shared content", nullptr, menufunc_st_sds, nullptr},
+       {"ida", "Internal memory for Default storage app installation", nullptr, menufunc_st_ida, nullptr},
+       {"sda", "SD Card for Default storage app installation", nullptr, menufunc_st_sda, nullptr},
+       {"mss", "Get Mount SD card status", nullptr, menufunc_st_mss, nullptr},
+       {"ms", "Mount SD card", nullptr, menufunc_st_ms, nullptr},
+       {"us", "Unmount SD card", nullptr, menufunc_st_us, nullptr},
+       {"fs", "Format SD card", nullptr, menufunc_st_fs, nullptr},
+       {"fsp", "Format SD card as device storage", nullptr, nullptr, nullptr}
 };
 
 void precreate_st(GenSettingMenu *menu)
index babab67..f2e6b85 100644 (file)
 
 using namespace std;
 
-static MENURUN_ERROR menufunc_wf_st(int argc, char *argv[], GenSettingMenu *menu)
-{
-       int state = 0;
-       int ret = vconf_get_int(VCONFKEY_WIFI_STATE, &state);
-       if (ret != 0) {
-               eprintf("\twifi state get fail!\n");
-               return MENURUN_ERR_NORMAL;
-       }
-
-       printf("\tState : %s\n", state ? "On" : "Off");
-       return MENURUN_ERR_NONE;
-}
 static MENURUN_ERROR menugetstring_wf_st(string &resultstr, int argc, char *argv[], GenSettingMenu *menu)
 {
        int state = 0;
@@ -35,18 +23,6 @@ static MENURUN_ERROR menugetstring_wf_st(string &resultstr, int argc, char *argv
        return MENURUN_ERR_NONE;
 }
 
-static MENURUN_ERROR menufunc_wf_nf(int argc, char *argv[], GenSettingMenu *menu)
-{
-       bool state = false;
-       int ret = system_settings_get_value_bool(SYSTEM_SETTINGS_KEY_NETWORK_WIFI_NOTIFICATION, &state);
-       if (ret != 0) {
-               eprintf("\twifi notification get fail!\n");
-               return MENURUN_ERR_NORMAL;
-       }
-
-       printf("\tNodification : %s\n", state ? "On" : "Off");
-       return MENURUN_ERR_NONE;
-}
 static MENURUN_ERROR menugetstring_wf_nf(string &resultstr, int argc, char *argv[], GenSettingMenu *menu)
 {
        bool state = false;
@@ -56,13 +32,13 @@ static MENURUN_ERROR menugetstring_wf_nf(string &resultstr, int argc, char *argv
                return MENURUN_ERR_NORMAL;
        }
 
-       printf("\tNodification : %s\n", state ? "On" : "Off");
+       resultstr = state ? "On" : "Off";
        return MENURUN_ERR_NONE;
 }
 
 MenuItem wf_menulist[] = {
-       {"st", "State", menufunc_wf_st, nullptr, menugetstring_wf_st},
-       {"nf", "Notification", menufunc_wf_nf, nullptr, menugetstring_wf_nf}
+       {"st", "State", menugetstring_wf_st, nullptr, nullptr},
+       {"nf", "Notification", menugetstring_wf_nf, nullptr, nullptr}
 };
 
 
index 83c68da..1592280 100644 (file)
@@ -3,4 +3,4 @@ This is sample report of [Connections Status]
    * Airplane Mode : @am/st
    * Bluetooth Status : @bt/st
    * Whi-fi Status : @wf/st
-   * Whi-fi Notification: @wf/st
+   * Whi-fi Notification: @wf/nf