easy-setup: Declare strict void function 67/23867/4
authorPhilippe Coval <philippe.coval@osg.samsung.com>
Mon, 13 Nov 2017 15:44:16 +0000 (16:44 +0100)
committerMats Wichmann <mats@linux.com>
Thu, 25 Jan 2018 16:50:42 +0000 (16:50 +0000)
Declare void functions as strict prototype

If built along -Wstrict-prototypes it will raise and error,
this is the case on TizenRT.

More patches to come to enable this flag by default.

Bug: https://jira.iotivity.org/browse/IOT-2539
Change-Id: Idc9e1ac5f318e4324bff6401e38097a6ea9cacda
Signed-off-by: Philippe Coval <philippe.coval@osg.samsung.com>
service/easy-setup/enrollee/inc/easysetup.h
service/easy-setup/enrollee/src/easysetup.c
service/easy-setup/enrollee/src/resourcehandler.c
service/easy-setup/enrollee/src/resourcehandler.h
service/easy-setup/sampleapp/enrollee/linux/enrolleewifi.c
service/easy-setup/sampleapp/enrollee/tizen-sdb/EnrolleeSample/enrolleewifi.cpp
service/easy-setup/sampleapp/mediator/linux/richsdk_sample/mediator.cpp
service/easy-setup/sampleapp/mediator/linux/richsdk_sample/submediator.cpp

index b87d15c..ae340ef 100644 (file)
@@ -104,7 +104,7 @@ ESResult ESSetErrorCode(ESErrorCode esErrCode);
  *
  * @return ::ES_OK on success, some other value upon failure.
  */
-ESResult ESTerminateEnrollee();
+ESResult ESTerminateEnrollee(void);
 
 /**
  * This function is to set two function pointer to handle user-specific properties in in-comming
index 0e63140..789ae5b 100644 (file)
@@ -268,7 +268,7 @@ ESResult ESSetErrorCode(ESErrorCode esErrCode)
 #pragma GCC diagnostic pop
 #endif
 
-ESResult ESTerminateEnrollee()
+ESResult ESTerminateEnrollee(void)
 {
     OIC_LOG(INFO, ES_ENROLLEE_TAG, "ESTerminateEnrollee IN");
 
index dd9495d..9840d2f 100644 (file)
@@ -1526,7 +1526,7 @@ OCStackResult CreateEasySetupResources(bool isSecured, ESResourceMask resourceMa
     return res;
 }
 
-OCStackResult DeleteEasySetupResources()
+OCStackResult DeleteEasySetupResources(void)
 {
     OCStackResult res = OC_STACK_ERROR;
     if (g_ESWiFiConfResource.handle != NULL)
index 4ac7fff..24728f8 100644 (file)
@@ -96,7 +96,7 @@ extern CoapCloudConfResource g_ESCoapCloudConfResource;
 extern DevConfResource g_ESDevConfResource;
 
 OCStackResult CreateEasySetupResources(bool isSecured, ESResourceMask resourceMask);
-OCStackResult DeleteEasySetupResources();
+OCStackResult DeleteEasySetupResources(void);
 
 OCStackResult SetDeviceProperty(ESDeviceProperty *deviceProperty);
 OCStackResult SetEnrolleeState(ESEnrolleeState esState);
index 20fc972..70e8c95 100644 (file)
@@ -55,7 +55,7 @@ static bool gIsSecured = false;
 
 UserProperties g_userProperties;
 
-void PrintMenu()
+void PrintMenu(void)
 {
     printf("============\n");
     printf("S: Enabled Security\n");
@@ -159,7 +159,7 @@ FILE* server_fopen(const char *path, const char *mode)
     }
 }
 
-void EnableSecurity()
+void EnableSecurity(void)
 {
     printf("Inside EnableSecurity API..\n");
 
@@ -170,7 +170,7 @@ void EnableSecurity()
     OCRegisterPersistentStorageHandler(&ps);
 }
 
-void StartEasySetup()
+void StartEasySetup(void)
 {
     printf("StartEasySetup IN\n");
 
@@ -196,7 +196,7 @@ void StartEasySetup()
     printf("StartEasySetup OUT\n");
 }
 
-void SetDeviceInfo()
+void SetDeviceInfo(void)
 {
     printf("SetDeviceInfo IN\n");
 
@@ -226,7 +226,7 @@ void SetDeviceInfo()
     printf("SetDeviceInfo OUT\n");
 }
 
-void StopEasySetup()
+void StopEasySetup(void)
 {
     printf("StopEasySetup IN\n");
 
@@ -252,12 +252,12 @@ void StopEasySetup()
     printf("StopEasySetup OUT\n");
 }
 
-void SetCallbackForUserdata()
+void SetCallbackForUserdata(void)
 {
     ESSetCallbackForUserdata(&ReadUserdataCb, &WriteUserdataCb);
 }
 
-int main()
+int main(void)
 {
     printf("#########################\n");
     printf("EasySetup Enrollee SAMPLE\n");
index 5f1ec2d..8dc9437 100644 (file)
@@ -62,7 +62,7 @@ static bool gWiFiCBflag = false;
 static char gSSID[OIC_STRING_MAX_VALUE];
 static char gPasswd[OIC_STRING_MAX_VALUE];
 
-void PrintMenu()
+void PrintMenu(void)
 {
     cout << "========================" << endl;
     cout << "A: Enabled Security" << endl;
@@ -180,7 +180,7 @@ FILE* server_fopen(const char *path, const char *mode)
     }
 }
 
-void EnableSecurity()
+void EnableSecurity(void)
 {
     cout << "Inside EnableSecurity API.." << endl;
 
@@ -191,7 +191,7 @@ void EnableSecurity()
     OCRegisterPersistentStorageHandler(&ps);
 }
 
-void StartEasySetup()
+void StartEasySetup(void)
 {
     cout << "StartEasySetup IN" << endl;
 
@@ -242,7 +242,7 @@ void StartEasySetup()
     cout << "StartEasySetup OUT" << endl;
 }
 
-void SetDeviceInfo()
+void SetDeviceInfo(void)
 {
     cout << "SetDeviceInfo IN" << endl;
 
@@ -268,7 +268,7 @@ void SetDeviceInfo()
     cout << "SetDeviceInfo OUT" << endl;
 }
 
-void StopEasySetup()
+void StopEasySetup(void)
 {
     cout << "StopEasySetup IN" << endl;
     if (ESTerminateEnrollee() == ES_ERROR)
@@ -395,7 +395,7 @@ gboolean mainThread(GIOChannel *source, GIOCondition condition, gpointer data)
     return TRUE;
 }
 
-int main()
+int main(void)
 {
     cout << "#########################" << endl;
     cout << "EasySetup Enrollee SAMPLE" << endl;
index a187555..82b28c1 100644 (file)
@@ -559,7 +559,7 @@ static FILE *client_open(const char *path, const char *mode)
     }
 }
 
-int main()
+int main(void)
 {
     OCPersistentStorage ps {client_open, fread, fwrite, fclose, unlink };
 
index 7c0edf5..1e78f64 100644 (file)
@@ -557,7 +557,7 @@ static FILE *client_open(const char *path, const char *mode)
     }
 }
 
-int main()
+int main(void)
 {
     OCPersistentStorage ps {client_open, fread, fwrite, fclose, unlink };