Remove AbstractMetaFunction::invalid()

Change AbstractMetaBuilderPrivate::traverseFunction() to consistently
return 0 if something does not fit. Previously, functions with
half-parsed arguments with the invalid flag set were returned. This
caused strange side effects since the flag was not checked in
all places.

The only relevant information is whether some constructors were
rejected. In that case, no default constructors or default copy
constructors should be generated. This is now determined
by checking the code model function item; the attribute
HasRejectConstructor is introduced for this.

This fixes:

- Make it possible to reject the QTextStreamManipulator
  constructor taking a function pointer without having
  a default constructor generated:

  typedef void (QTextStream::*QTSMFI)(int);
  QTextStreamManipulator(QTSMFI m, int a) Q_DECL_NOTHROW

- Implement QtCharts whose class QAbstractSeries has a rejected
  constructor.

Change-Id: I6310574ba677dac20699f257340d2c2a55674353
Reviewed-by: Christian Tismer <tismer@stackless.com>

1 files changed, 2 insertions, 0 deletions

diff --git a/sources/shiboken2/generator/shiboken2/shibokengenerator.cpp b/sources/shiboken2/generator/shiboken2/shibokengenerator.cpp
index 4768ddc88..fb9946be1 100644
--- a/sources/shiboken2/generator/shiboken2/shibokengenerator.cpp
+++ b/sources/shiboken2/generator/shiboken2/shibokengenerator.cpp
@@ -1004,6 +1004,8 @@ bool ShibokenGenerator::isValueTypeWithCopyConstructorOnly(const AbstractMetaCla
 {
     if (!metaClass || !metaClass->typeEntry()->isValue())
         return false;
+    if ((metaClass->attributes() & AbstractMetaAttributes::HasRejectedConstructor) != 0)
+        return false;
     AbstractMetaFunctionList ctors = metaClass->queryFunctions(AbstractMetaClass::Constructors);
     if (ctors.count() != 1)
         return false;