summaryrefslogtreecommitdiffstats
path: root/src/corelib/statemachine/qstate.cpp
diff options
context:
space:
mode:
authorKent Hansen <khansen@trolltech.com>2009-08-03 08:35:37 (GMT)
committerKent Hansen <khansen@trolltech.com>2009-08-03 08:40:30 (GMT)
commitacd156649b1083aeeb400fe8fdbbf4e55ada4bed (patch)
treee74b2486cacc4b318b5c0533360ab03e1923a11d /src/corelib/statemachine/qstate.cpp
parent45005c68c6effa41108ef55cd10a84f5977f79a0 (diff)
downloadQt-acd156649b1083aeeb400fe8fdbbf4e55ada4bed.zip
Qt-acd156649b1083aeeb400fe8fdbbf4e55ada4bed.tar.gz
Qt-acd156649b1083aeeb400fe8fdbbf4e55ada4bed.tar.bz2
remove constructors that are not useful
The constructors that take a list of target states produce hard-to-read code, and they're rarely useful in practice since 99% of transitions take a single target state; so it's better to enforce that setTarget{State,States}() be used instead.
Diffstat (limited to 'src/corelib/statemachine/qstate.cpp')
-rw-r--r--src/corelib/statemachine/qstate.cpp6
1 files changed, 4 insertions, 2 deletions
diff --git a/src/corelib/statemachine/qstate.cpp b/src/corelib/statemachine/qstate.cpp
index 2042288..09d0be0 100644
--- a/src/corelib/statemachine/qstate.cpp
+++ b/src/corelib/statemachine/qstate.cpp
@@ -339,7 +339,8 @@ QSignalTransition *QState::addTransition(QObject *sender, const char *signal,
return 0;
}
}
- QSignalTransition *trans = new QSignalTransition(sender, signal, QList<QAbstractState*>() << target);
+ QSignalTransition *trans = new QSignalTransition(sender, signal);
+ trans->setTargetState(target);
addTransition(trans);
return trans;
}
@@ -351,7 +352,8 @@ class UnconditionalTransition : public QAbstractTransition
{
public:
UnconditionalTransition(QAbstractState *target)
- : QAbstractTransition(QList<QAbstractState*>() << target) {}
+ : QAbstractTransition()
+ { setTargetState(target); }
protected:
void onTransition(QEvent *) {}
bool eventTest(QEvent *) { return true; }