This is a fail test... could it be that the missing return is actually part of the behavior being checked by this test?

Cheers,

Jon

On 8/23/14, 12:36 AM, Eric Fiselier wrote:
Author: ericwf
Date: Sat Aug 23 01:36:11 2014
New Revision: 216317

URL: http://llvm.org/viewvc/llvm-project?rev=216317&view=rev
Log:
Add return statement to a test class's assignment operator. Defect found by 
Coverity Scan.

Modified:
     
libcxx/trunk/test/utilities/memory/unique.ptr/unique.ptr.runtime/unique.ptr.runtime.ctor/default01.fail.cpp

Modified: 
libcxx/trunk/test/utilities/memory/unique.ptr/unique.ptr.runtime/unique.ptr.runtime.ctor/default01.fail.cpp
URL: 
http://llvm.org/viewvc/llvm-project/libcxx/trunk/test/utilities/memory/unique.ptr/unique.ptr.runtime/unique.ptr.runtime.ctor/default01.fail.cpp?rev=216317&r1=216316&r2=216317&view=diff
==============================================================================
--- 
libcxx/trunk/test/utilities/memory/unique.ptr/unique.ptr.runtime/unique.ptr.runtime.ctor/default01.fail.cpp
 (original)
+++ 
libcxx/trunk/test/utilities/memory/unique.ptr/unique.ptr.runtime/unique.ptr.runtime.ctor/default01.fail.cpp
 Sat Aug 23 01:36:11 2014
@@ -25,7 +25,7 @@ class Deleter
  public:

      Deleter(Deleter&) {}
-    Deleter& operator=(Deleter&) {}
+    Deleter& operator=(Deleter&) { return *this; }

      void operator()(void*) const {}
  };


_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits


--
Jon Roelofs
[email protected]
CodeSourcery / Mentor Embedded
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to