resource/examples: Removing /W4 warnings 27/20727/8
authorPawel Winogrodzki <pawelwi@microsoft.com>
Mon, 12 Jun 2017 22:17:36 +0000 (15:17 -0700)
committerDan Mihai <Daniel.Mihai@microsoft.com>
Fri, 16 Jun 2017 22:34:57 +0000 (22:34 +0000)
Removing "unreachable code" warnings due to infinite waits.

Bug: https://jira.iotivity.org/browse/IOT-1583

Change-Id: I8d057494e39540f35f1ec447217d70eff556d1c0
Signed-off-by: Pawel Winogrodzki <pawelwi@microsoft.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/20727
Reviewed-by: Phil Coval <philippe.coval@osg.samsung.com>
Tested-by: jenkins-iotivity <jenkins@iotivity.org>
Reviewed-by: Dan Mihai <Daniel.Mihai@microsoft.com>
resource/examples/devicediscoveryclient.cpp
resource/examples/devicediscoveryserver.cpp
resource/examples/mediaserver.cpp

index 03e36d0..b657aa7 100644 (file)
@@ -179,14 +179,9 @@ int main(int argc, char* argv[]) {
             std::cout << "failed." << std::endl;
         }
 
-        // A condition variable will free the mutex it is given, then do a non-
-        // intensive block until 'notify' is called on it.  In this case, since we
-        // don't ever call cv.notify, this should be a non-processor intensive version
-        // of while(true);
-        std::mutex blocker;
-        std::condition_variable cv;
-        std::unique_lock<std::mutex> lock(blocker);
-        cv.wait(lock, []{return false;});
+        std::cout << "Waiting. Press \"Enter\" to quit." << std::endl;
+        // Ignoring all input except for EOL.
+        std::cin.ignore(std::numeric_limits<std::streamsize>::max(), '\n');
 
         OC_VERIFY(OCPlatform::stop() == OC_STACK_OK);
 
index ce216be..dcfcb41 100644 (file)
@@ -186,14 +186,9 @@ int main()
 
     DeletePlatformInfo();
 
-    // A condition variable will free the mutex it is given, then do a non-
-    // intensive block until 'notify' is called on it.  In this case, since we
-    // don't ever call cv.notify, this should be a non-processor intensive version
-    // of while(true);
-    std::mutex blocker;
-    std::condition_variable cv;
-    std::unique_lock<std::mutex> lock(blocker);
-    cv.wait(lock, []{return false;});
+    std::cout << "Waiting. Press \"Enter\" to quit." << std::endl;
+    // Ignoring all input except for EOL.
+    std::cin.ignore(std::numeric_limits<std::streamsize>::max(), '\n');
 
     // Perform platform clean up.
     OC_VERIFY(OCPlatform::stop() == OC_STACK_OK);
index da8ad44..b3d70cf 100644 (file)
@@ -676,15 +676,12 @@ int main(int argc, char* argv[])
         myMedia.createResource();
         cout << "Created resource." << std::endl;
 
-        // A condition variable will free the mutex it is given, then do a non-
-        // intensive block until 'notify' is called on it.  In this case, since we
-        // don't ever call cv.notify, this should be a non-processor intensive version
-        // of while(true);
-        std::mutex blocker;
-        std::condition_variable cv;
-        std::unique_lock<std::mutex> lock(blocker);
-        cout <<"Waiting" << std::endl;
-        cv.wait(lock, []{return false;});
+        cout <<"Waiting. Press \"Enter\" to quit." << std::endl;
+#ifdef _MSC_VER
+#undef max
+#endif
+        // Ignoring all input except for EOL.
+        cin.ignore(numeric_limits<streamsize>::max(), '\n');
 
         // Perform platform clean up.
         OC_VERIFY(OCPlatform::stop() == OC_STACK_OK);