coap-http-proxy: Declare strict void function 01/23701/5
authorPhilippe Coval <philippe.coval@osg.samsung.com>
Tue, 14 Nov 2017 11:04:34 +0000 (12:04 +0100)
committerMats Wichmann <mats@linux.com>
Wed, 31 Jan 2018 16:06:08 +0000 (16:06 +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: I175b1790afcd01ea614da707bd51881ed4556a8d
Signed-off-by: Philippe Coval <philippe.coval@osg.samsung.com>
service/coap-http-proxy/include/CoapHttpHandler.h
service/coap-http-proxy/include/CoapHttpParser.h
service/coap-http-proxy/samples/proxy_client.c
service/coap-http-proxy/samples/proxy_main.c
service/coap-http-proxy/src/CoapHttpHandler.c
service/coap-http-proxy/src/CoapHttpParser.c
service/coap-http-proxy/unittests/CoAPHttpUnitTest.cpp

index 9f3fa07..18bf743 100644 (file)
@@ -44,13 +44,13 @@ OCStackResult CHPInitialize(bool secure);
  * Terminate the CoAP HTTP Proxy.
  * @return  ::OC_STACK_OK or Appropriate error code.
  */
-OCStackResult CHPTerminate();
+OCStackResult CHPTerminate(void);
 
 /**
  * API to check if CoAP-HTTP Proxy is initialized.
  * @return  true if initialized else false.
  */
-bool CHPIsInitialized();
+bool CHPIsInitialized(void);
 
 #ifdef __cplusplus
 }
index 5e5067c..66687a2 100644 (file)
@@ -133,12 +133,12 @@ typedef void (*CHPResponseCallback)(const HttpResponse_t *response, void *contex
 /**
  * Function to initialize Parser and HTTP stack.
  */
-OCStackResult CHPParserInitialize();
+OCStackResult CHPParserInitialize(void);
 
 /**
  * Function to terminate parser and HTTP stack.
  */
-OCStackResult CHPParserTerminate();
+OCStackResult CHPParserTerminate(void);
 
 /**
  * Function to initiate TCP session and post HTTP request. If the method returns
index 352bb04..4af2fef 100644 (file)
@@ -69,7 +69,7 @@ void handleSigInt(int signum)
     }
 }
 
-OCPayload* putPayload()
+OCPayload* putPayload(void)
 {
     OCRepPayload* payload = OCRepPayloadCreate();
     if (!payload)
@@ -86,7 +86,7 @@ OCPayload* putPayload()
     return (OCPayload*)payload;
 }
 
-static void PrintUsage()
+static void PrintUsage(void)
 {
     OIC_LOG(INFO, TAG, "Usage : proxy_client -t <1..5>");
     OIC_LOG(INFO, TAG, "-t 1  :  Discover Proxy");
@@ -187,7 +187,7 @@ OCStackApplicationResult getReqCB(void* ctx, OCDoHandle handle,
     return OC_STACK_DELETE_TRANSACTION;
 }
 
-int InitProxyRequest()
+int InitProxyRequest(void)
 {
     OIC_LOG(INFO, TAG, "InitProxyRequest");
     OCHeaderOption option;
@@ -322,7 +322,7 @@ OCStackApplicationResult discoveryReqCB(void* ctx, OCDoHandle handle,
     return OC_STACK_KEEP_TRANSACTION;
 }
 
-int InitDiscovery()
+int InitDiscovery(void)
 {
     OCStackResult ret;
     OCCallbackData cbData;
index 7c680f5..fd4b195 100644 (file)
@@ -34,7 +34,7 @@ static char CRED_FILE[] = "oic_svr_db_server.dat";
 
 void handleSigInt(int signum);
 
-static void PrintUsage()
+static void PrintUsage(void)
 {
     printf("Usage : proxy_main -s <0|1>\n");
     printf("-s 0 : Launch proxy in unsecure mode.\n");
index 22f649b..e6a8eb6 100644 (file)
@@ -64,7 +64,7 @@ OCStackResult CHPHandleOCFRequest(const OCEntityHandlerRequest* requestInfo,
 OCEntityHandlerResult CHPEntityHandler(OCEntityHandlerFlag flag,
                                        OCEntityHandlerRequest* entityHandlerRequest,
                                        void* callbackParam);
-bool CHPIsInitialized()
+bool CHPIsInitialized(void)
 {
     return g_isCHProxyInitialized;
 }
@@ -118,7 +118,7 @@ OCStackResult CHPInitialize(bool secure)
     return OC_STACK_OK;
 }
 
-OCStackResult CHPTerminate()
+OCStackResult CHPTerminate(void)
 {
     OIC_LOG_V(DEBUG, TAG, "%s IN", __func__);
 
@@ -184,7 +184,7 @@ static void CHPGetProxyURI(OCHeaderOption* options, uint8_t *numOptions, char* u
 }
 
 // TODO: Will be moved to OCProxyPayload
-static OCRepPayload* CHPGetDiscoveryPayload()
+static OCRepPayload* CHPGetDiscoveryPayload(void)
 {
     OCRepPayload* payload = OCRepPayloadCreate();
     if (!payload)
index 2d8862a..9eb469c 100644 (file)
@@ -97,8 +97,8 @@ static int g_refreshFds[2];
  */
 static pthread_t g_multiHandleThread;
 
-static void CHPParserLockMutex();
-static void CHPParserUnlockMutex();
+static void CHPParserLockMutex(void);
+static void CHPParserUnlockMutex(void);
 
 static void CHPParserResetHeaderOptions(u_arraylist_t** headerOptions)
 {
@@ -350,7 +350,7 @@ OCStackResult CHPParserInitializePipe(int fds[2])
     return OC_STACK_OK;
 }
 
-static OCStackResult CHPParserInitializeMutex()
+static OCStackResult CHPParserInitializeMutex(void)
 {
     // create the mutex with the attributes set
     int ret = pthread_mutex_init(&g_multiHandleMutex, PTHREAD_MUTEX_DEFAULT);
@@ -362,7 +362,7 @@ static OCStackResult CHPParserInitializeMutex()
     return OC_STACK_OK;
 }
 
-static OCStackResult CHPParserTerminateMutex()
+static OCStackResult CHPParserTerminateMutex(void)
 {
     int ret = pthread_mutex_destroy(&g_multiHandleMutex);
     if (0 != ret)
@@ -373,7 +373,7 @@ static OCStackResult CHPParserTerminateMutex()
     return OC_STACK_OK;
 }
 
-static void CHPParserLockMutex()
+static void CHPParserLockMutex(void)
 {
     int ret = pthread_mutex_lock(&g_multiHandleMutex);
     if(ret != 0)
@@ -382,7 +382,7 @@ static void CHPParserLockMutex()
     }
 }
 
-static void CHPParserUnlockMutex()
+static void CHPParserUnlockMutex(void)
 {
     int ret = pthread_mutex_unlock(&g_multiHandleMutex);
     if(ret != 0)
@@ -391,7 +391,7 @@ static void CHPParserUnlockMutex()
     }
 }
 
-static OCStackResult CHPParserInitializeMultiHandle()
+static OCStackResult CHPParserInitializeMultiHandle(void)
 {
     CHPParserLockMutex();
     if(g_multiHandle)
@@ -413,7 +413,7 @@ static OCStackResult CHPParserInitializeMultiHandle()
     return OC_STACK_OK;
 }
 
-static OCStackResult CHPParserTerminateMultiHandle()
+static OCStackResult CHPParserTerminateMultiHandle(void)
 {
     CHPParserLockMutex();
     if(!g_multiHandle)
@@ -429,7 +429,7 @@ static OCStackResult CHPParserTerminateMultiHandle()
     return OC_STACK_OK;
 }
 
-OCStackResult CHPParserInitialize()
+OCStackResult CHPParserInitialize(void)
 {
     OIC_LOG_V(DEBUG, TAG, "%s IN", __func__);
 
@@ -480,7 +480,7 @@ OCStackResult CHPParserInitialize()
     return OC_STACK_OK;
 }
 
-OCStackResult CHPParserTerminate()
+OCStackResult CHPParserTerminate(void)
 {
     OIC_LOG_V(DEBUG, TAG, "%s IN", __func__);
     g_terminateParser = true;
index 60f94bf..e10a528 100644 (file)
@@ -49,13 +49,13 @@ static OCConnectivityType connType;
 class CoApHttpTest: public TestWithMock
 {
 protected:
-    void SetUp()
+    void SetUp(void)
     {
         TestWithMock::SetUp();
         OCInit(NULL, 0, OC_CLIENT_SERVER);
     }
 
-    void TearDown()
+    void TearDown(void)
     {
         OCStop();
     }