{non-gerrit}/qt-labs/scxml.git - Unnamed repository; edit this file 'description' to name the repository.

	Commit message (Collapse)	Author	Age	Files	Lines
*	Change parsing of "event" attribute of transitions.HEAD master	Marc Schmitzer	2012-01-27	1	-3/+8
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Previously, the event prefixes for a transition where obtained by using QString::split(' ') on the value of the "event" attribute. QString::split() also retains empty strings resulting from the split, which has two consequences: 1) If the event attribute is empty or missing, a QStringList containing a single empty string is returned. This makes event-less transitions work, which are executed immediately (if the condition is met). 2) If the event attribute contains multiple prefixes separated by more than one space character, the returned list contains the prefixes and one or more empty strings. This makes the transition trigger immediately and on any event. Since an extra space is an easy mistake to make and difficult to spot (speaking from experience), this behaviour is rather undesirable. To fix this, this patch changes the parsing of the event attribute to use QString::split(' ', QString::SkipEmptyParts). This fixes (2) but breaks (1). To keep event-less transitions working, QScxmlTransition::eventTest() is changed to match events of type QEvent::None if the transition has no event prefixes. Due to the above change, this is the case when the "event" attribute is ommitted or contains only space characters. NOTE: Strictly speaking, this removes a feature from qscxml. In the previous state, it was possible to create a transition (presumably with a guard condition) that is both executed immediately when its source state is entered and triggered by one or more signals. With this change, a transition can only be either immediate or triggered by (an) event(s), not both. The W3C scxml specification seems to specify this behaviour (see http://www.w3.org/TR/scxml/#SelectingTransitions ).
*	Clear script in curExecContext after applying it to a transition in ↵	Marc Schmitzer	2011-11-28	1	-0/+1
\| \| \| \| \| \| \| \| \| \| \| \|	QScxmlLoader::loadState(). Without this change, the script is also applied to the state containing the transition in line 1502 if the state sources an external scxml file (via the "src") attribute. In that case, the next element processed by the parser is the closing </scxml> if the transition containing the script is the last in the file. Maybe this should better be fixed in the block starting at line 1499, but clearing the script after applying it somewhere seems safe enough. Merge-request: 3 Reviewed-by: No'am Rosenthal <noam.rosenthal@nokia.com>
*	removed fringe features	No'am Rosenthal	2009-11-26	1	-134/+10
\|
*	updated license headers	No'am Rosenthal	2009-11-26	1	-3/+35
\|
*	many fixes and optimizations to SCXML	No'am Rosenthal	2009-11-26	1	-48/+135
\|
*	adjusted for postInternalEvent API change	No'am Rosenthal	2009-10-03	1	-1/+1
\|
*	Changes in QStateMachine (rootState)	No'am Rosenthal	2009-07-22	1	-3/+3
\|
*	Enabled using SCXML with an existing QScriptEngine	No'am Rosenthal	2009-07-12	1	-29/+53
\|
*	Work with the new activated trigger from QAbstractTransition	Noam Rosenthal	2009-06-18	1	-14/+10
\|
*	Bug fixes after testing with VUI	Noam Rosenthal	2009-06-15	1	-88/+274
\|
*	Fix headers for labs	Noam Rosenthal	2009-06-08	1	-15/+2
\|
*	Fix headers for labs	Noam Rosenthal	2009-06-08	1	-30/+12
\|
*	Documentation fixes	Noam Rosenthal	2009-06-08	1	-7/+15
\|
*	some missing files	Noam Rosenthal	2009-06-08	1	-59/+66
\|
*	scxml for 4.6	Noam Rosenthal	2009-06-08	1	-0/+1428