summaryrefslogtreecommitdiffstats
path: root/tests
Commit message (Collapse)AuthorAgeFilesLines
* fix silly typoKent Hansen2009-06-031-1/+13
| | | | | | | | | | Yeesh. The function worked for the common case of the argument being a plain script object (obviously, otherwise this would have been discovered sooner), but it would never pick the less expensive path when replacing the QObject pointer of an existing proxy. And if you passed in a QVariant proxy (now who would ever do something like that...?), it would assert.
* don't require use of SIGNAL macro in calls to addTransition()Kent Hansen2009-06-031-0/+28
| | | | | Just as with the QSignalTransition::signal property, this makes it possible to use the function from language bindings (e.g. QtScript).
* Merge branch '4.5'Denis Dzyubenko2009-06-021-1/+1
|\
| * Correct invalid font.Norwegian Rock Cat2009-06-021-1/+1
| | | | | | | | QFont() returns the app font, not an invalid font.
* | Setting a focus on a widget hierarchy which contains both visible andDenis Dzyubenko2009-06-021-0/+35
| | | | | | | | | | | | | | | | | | | | | | invisible widgets could cause a crash. Also, when there is a widget hierarchy A->B->C and A and C are marked as visible but B is hidden, when the user gives focus explicitely to C, we should remember that and respect it when B becomes visible. Task-number: 254563 Reviewed-by: Thierry Bastian
* | Merge branch '4.5' of scm.dev.nokia.troll.no:qt/qtSimon Hausmann2009-06-024-7/+125
|\ \ | |/ | | | | | | | | | | | | Conflicts: src/opengl/gl2paintengineex/qpaintengineex_opengl2.cpp tests/auto/qgraphicswidget/tst_qgraphicswidget.cpp tests/auto/selftests/expected_skip.txt tests/auto/selftests/tst_selftests.cpp
| * Fixed failure of tst_Selftests::checkXML.Rohan McGovern2009-06-013-4/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Cherry-pick of 548da9a5434d615456a7a6efda3380b7138c6000 and some earlier changes predating public repo. Note that this test is not failing if using Qt 4.5 only. However, it fails if using Qt 4.5 selftests against Qt master testlib. We want to be able to use master testlib to run 4.5 testcases, and this change does no harm when using Qt 4.5 testlib, so backport it to 4.5 for convenience. Original change description follows: A few tests use printf, which means they interfere with the XML test logging. Blacklist them for the XML test. Note that these tests happened to pass under the old test logger implementation. That was because the test logger always printed XML tags on a single line, and the printf calls contained no special XML characters. The test logs generated were technically valid XML but contained extraneous text.
| * Fix a ASSERT/Crash when adding two times the same QAction to a QGW.Alexis Menard2009-05-281-0/+16
| | | | | | | | | | | | | | | | | | We were adding two times in the QActionPrivate list the entry for the current QGraphicsWidget if the action was existing before. Task-number:KDE Reviewed-by:bnilsen BT:yes
| * Fix wrong sorting when using the QFileSystemModel with QTreeViewAlexis Menard2009-05-281-3/+62
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | An optimization was made to the sorting of QFileDialog to sort only the current root (meaning what the user see). This avoided slowness when the model was big with lots of leafs. The problem here is for the treeview, the root is always the same, we expands only nodes. In that case, a recursive sorting is needed to ensure that all expanded nodes are correctly sorted (and filtered). This will be slower that's why i use an hidden flag in the d pointer to deactivate the recursive sort for the QFileDialog. Task-number:254701 Reviewed-by:olivier BT:yes
| * Fixed a problem with streaming QIcons containing multiple pixmaps.Trond Kjernåsen2009-05-281-0/+43
| | | | | | | | | | | | | | | | | | If pixmaps were added through QIcon::addFile() with different sizes than the sizes of the pixmaps themselves, streaming the icon in didn't work properly. Task-number: 254374 Reviewed-by: Kim
| * Minor cleanup.Trond Kjernåsen2009-05-281-4/+4
| | | | | | | | Reviewed-by: Kim
* | Fixed bug in QClipData::fixup().Samuel Rødal2009-06-021-0/+24
| | | | | | | | | | | | | | The bounding rect computed in fixup() is one pixel too wide, causing potential memory corruption by painting outside device boundaries. Reviewed-by: Trond
* | Add some performance tests for QMatrix4x4Rhys Weatherley2009-06-023-0/+268
| |
* | Unit tests for 2D QMatrix4x4::translate() and scale()Rhys Weatherley2009-06-021-0/+40
| |
* | Optimize QMatrix4x4::mapRect() for translation and scale operationsRhys Weatherley2009-06-021-0/+114
| |
* | Merge branch 'review/master'Marius Bugge Monsen2009-05-291-1/+1
|\ \
| * \ Merge branch 'master' of git://gitorious.org/qt/qtDavid Faure2009-05-2953-77/+10734
| |\ \
| * | | typo fixDavid Faure2009-05-291-1/+1
| | | |
* | | | extend QStringBuilder benchmark to handle QT_NO_CAST_FROM_ASCII defined andhjk2009-05-292-56/+122
| |/ / |/| | | | | | | | undefined.
* | | QStringBuilder benchmark: make testing of the drop-in replacement ↵hjk2009-05-282-17/+21
| | | | | | | | | | | | | | | | | | | | | operator+() easier. This adds a few explicit QString(...) casts around operator+() based expressions to make them acceptable for QCOMPARE.
* | | Introduce a new class QStringBuilder to speed up the creation ofhjk2009-05-282-0/+316
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | QString objects from smaller chunks. The QStringBuilder class: QStringBuilder uses expression templates (using the '%' operator) to postpone any actual concatenation until it is assigned to an actual QString. At that time it knows the exact sizes of all chunks, can compute the required space, allocates once a QString of appriopriate size and then copies over the chunk data one-by-one. In addition, QLatin1Literal is a drop-in replacement for QLatin1String (which we can't change for compatibility reasons) that knows its size, therefore saving a few cycles when computing the size of the resulting string. Some further saved cycles stem from inlining and reduced reference counting logic (the QString created from a QStringBuilder has typically ref count equal to 1, while QString::append() needs an extra test) Minor changes to the existing QString class: - Introduce QString constructor to create an uninitialized QString of a given size. This particular constructor is used by QStringBuilder class. - Introduce a QT_USE_FAST_CONCATENATION macro to disable the existing overloads of operator+() and helps finding the places where they are used in code. - Introduce QT_USE_FAST_OPERATOR_PLUS. This also disables the existing overloads of operator+() and creates a new templated operator+() with identical implementation of operator%(). This allows code that is compilable QT_CAST_{TO,FROM}_ASCII to use QStringBuilder almost transparently. The only case that is not covered is creating objects like QUrl that are implicitly constructible from a QString from a QStringBuilder result. This needs to be converted explicitly to a QString first, e.g. by using QUrl url(QString(QLatin1String("http://") + hostName)); Reviewed-by: MariusSO
* | | Merge branch '4.5' of git@scm.dev.nokia.troll.no:qt/qtSimon Hausmann2009-05-277-5/+154
|\ \ \ | | |/ | |/| | | | | | | Conflicts: tests/auto/qtreeview/tst_qtreeview.cpp
| * | Fixed: Setting a border using stylesheet for QComboBox adds an unwated frame.Olivier Goffart2009-05-271-3/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This was already fix. But there was still a frame if there was a stylesheet on the applicaiton. The reason is that the frame's constructor call the style for some hints. And later the combobox will query the style again for the frame hint, before the view get polished. The problem is that the StyleSheetStyle will fill the css cache with wrong information on the first call. This is not visible if there is no stylesheet as in the constructor, the widget style is still the default application stylesheet if there is no global applicaiton stylesheet. Task-number: 254589 Reviewed-by: jbache BT:
| * | Fixes for merge-request #407Leonardo Sobral Cunha2009-05-271-33/+37
| | | | | | | | | | | | Removed unused line and added task number to autotest.
| * | Dont show children when parent is not visible in QGraphicsItemdiaulas2009-05-271-0/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | When setVisible() is called on a QGraphicsItem, if the parent of that item was hidden, the child shouldn't be actually shown. Task-number: 197802 Reviewed-by: leo Reviewed-by: alexis
| * | QFileDialog selection bug when calling it multiple times.Alexis Menard2009-05-261-0/+33
| | | | | | | | | | | | | | | | | | | | | | | | The problem was that we don't clear the selection model if the previous selection was valid. Task-number:251341 Reviewed-by:jasplin
| * | Test for QDate::fromDate with string format.Olivier Goffart2009-05-261-0/+26
| | | | | | | | | | | | | | | Note that two digit years are always in the year 19xx This is excepted for compatibility reason as discussed with Peter
| * | Fixed: QSortFilterProxyModel setDynamicSortFilter doesn't works when setting ↵Olivier Goffart2009-05-262-1/+38
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | the model initially This was caused by two different bug: - In the QSortFilterProxyModel, we need to re-sort when setting the source model change the sorting column (happen when setting a model initially) - In the treeview, we need to activate the sorting even if there is no column yet (because the initial model is empty Task-number: 254234 Reviewed-by: Thierry BT:
| * | Fixed qmake writing targets into the root of the current drive whenRohan McGovern2009-05-261-0/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | DESTDIR=./ and using Windows, nmake and shadow builds. qmake would canonicalize the DESTDIR of "./" to "". Then it would check if the original DESTDIR ended with "/", and if so, append it to the new DESTDIR, resulting in a DESTDIR of "/" - the root of the current drive. Don't do that. This bug doesn't occur with in-source builds because qmake detects that the source and build directories are the same directory and replaces the DESTDIR of "./" with "" before it reaches the buggy code. Autotest: included Reviewed-by: Lincoln Ramsay
* | | Add comparation of images with indexed colorBenjamin Poulain2009-05-271-0/+28
| | | | | | | | | | | | | | | | | | | | | | | | Previously, two images with indexed colors were not equal if their color tables were not the same. The image are not compared by the value of the pixels Reviewed-by: Samuel
* | | Avoid deep copies of pixmaps when using cache in QGraphicsView.Alexis Menard2009-05-273-0/+103
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When we have already a pixmap of an item in the cache, we should remove the entry from the cache otherwise when we will repaint/modify the copy of the pixmap (the copy of the cache) then we will trigger a deep copy ; this is because both entries in the cache and our copy share the same data and if you modify one of them a copy is triggered. I have added a benchmark as well to test that. Reviewed-by:bnilsen Reviewed-by:andreas
* | | Add a flag that ensure that a connection is made only oneOlivier Goffart2009-05-271-4/+77
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It is often desirable, when doing connection, to ensure that the same connection is only made once. This can be done with the Qt::UniqueConnection 'flag' Also documented the order the slot are called Reviewed-by: Brad
* | | We first remove the pixmap from the cache and then release the keyAlexis Menard2009-05-261-0/+8
| | | | | | | | | | | | | | | | | | | | | If we don't remove the pixmap from QCache first then the key is invalid and the removal failed Reviewed-by: sroedal
* | | Fixed a bug in animation when you set a start value that doesn't haveThierry Bastian2009-05-261-19/+58
| | | | | | | | | | | | | | | | | | | | | | | | | | | the same type as the property For example, if you set a start value of 0 (integer) for a qreal animation, it would not work. Reviewed-by: janarve
* | | Fix the previous fix.Jan-Arve Sæther2009-05-261-177/+5
| | | | | | | | | | | | | | | The previous fix fails on msvc-2005, since QString::setNum() does not handle rounding of numbers such as 0.015625 consistently.
* | | Merge branch '4.5'Thiago Macieira2009-05-225-1/+159
|\ \ \ | |/ / | | | | | | | | | Conflicts: src/corelib/kernel/qsharedmemory_unix.cpp
| * | Pressing enter in a QPlainTextEdit embedded on a itemview should insert a ↵Olivier Goffart2009-05-221-0/+78
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | newline Do the same special case as for QTextEdit (yes, this is a pitty that we have special cases like that Reviewed-by: Thierry Task-number: 252532
| * | Unconditionally open temporary files in ReadWrite modeJoão Abecasis2009-05-221-0/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Although QTemporaryFile hides QFile::open(OpenMode), this function is still available when accessing instance methods through the base class. Unconditionally setting ReadWrite allows the temporary file to be re-opened with different flags. Task-number: 248223 Reviewed-by: Thiago
| * | QTemporaryFile would forget fileName while file was "closed"João Abecasis2009-05-221-0/+23
| | | | | | | | | | | | | | | | | | | | | Note: this showed even if the file descriptor was kept open. Reviewed-by: Peter Hartmann Reviewed-by: Thiago
| * | Fix auto testJoão Abecasis2009-05-221-1/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | When copying a resource file to the native file system the (read-only) permissions also get copied. On Windows platforms, this was preventing a test file from being deleted. Reviewed-by: Peter Hartmann Reviewed-by: Thiago
| * | QFile::copy: close source file when using fallback mechanismJoão Abecasis2009-05-223-1/+36
| | | | | | | | | | | | | | | | | | | | | Also added check in test case for rename fallback. Task-number: 165920 Reviewed-by: Thiago
* | | add various testsKent Hansen2009-05-221-18/+86
| | | | | | | | | | | | Trying to get coverage up to 100%
* | | test what happens when changing to an invalid signal while state is activeKent Hansen2009-05-221-0/+6
| | |
* | | add tests for multiple signal/event transitions from same sourceKent Hansen2009-05-221-0/+68
| | |
* | | make sure signal transition is correctly unregistered/re-registeredKent Hansen2009-05-221-8/+27
| | | | | | | | | | | | | | | | | | When the signal or senderObject is changed while the state is active, the transition needs to be unregistered _before_ either property is changed, only _then_ can it be re-registered.
* | | make sure event transition is correctly unregistered/re-registeredKent Hansen2009-05-221-0/+13
| | | | | | | | | | | | | | | | | | When the eventType or eventObject is changed while the state is active, the transition needs to be unregistered _before_ either property is changed, only _then_ can it be re-registered.
* | | add qstatemachine autotest to auto.proKent Hansen2009-05-221-0/+1
| | |
* | | add some edge case tests for QStateMachine::start() & stop()Kent Hansen2009-05-221-0/+8
| | |
* | | Fix QEasingCurve autotestsLeonardo Sobral Cunha2009-05-221-15/+193
| | | | | | | | | | | | | | | | | | | | | | | | Easing curve autotests was converting the easing(qreal) to int before, but this is a very rough comparison and was failing due to different conversions to int, so now we do a qFuzzyCompare. Reviewed-by: Thierry
* | | Merge branch '4.5'Thiago Macieira2009-05-223-1/+600
|\ \ \ | |/ /