| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
|
| |
Clarified that the format used in QImage::fromData() is the image format,
not the pixel format.
|
|
|
|
|
|
| |
These structures are only used in versions after 0.922.
Reviewed-by: Donald <qt-info@nokia.com>
|
|
|
|
|
|
|
|
|
| |
throwing them.
Autotest included.
Task: 258081
Reviewed-by: Trustme
|
|
|
|
| |
to respective APIs.
|
| |
|
|
|
|
| |
This closes task 235801.
|
|
|
|
| |
Reviewed-by: Gunnar Sletta
|
|
|
|
| |
too "advanced".
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Some compilers don't obey the same rules of "top-of-object" values for
casting a pointer from a given class to void *. In any case, that can
only work for polymorphic types (with a virtual table).
So don't track the pointers by their pointer value, but instead by the
d-pointer of the QSharedPointer object.
The same cases that were caught before should still be caught. We
still won't catch the creating a second QSharedPointer for the same
object if the pointer values are different, though (when cast to
void*).
Reviewed-by: Bradley T. Hughes
|
|
|
|
|
|
|
|
|
| |
In de05f9a40e41deb79daf5c4935b2645d70d7f322 I removed the fcntl that
set FD_CLOEXEC because it was supposed to be set by
qt_safe_pipe. Turns out that this code didn't call the wrapper
function...
Reviewed-by: ossi
|
|
|
|
|
|
| |
I don't know if this was only the debugging symbols or if the compiler
was really wrong. But while debugging, Self was
ExternalRefCountWithCustomDeleter, which is wrong at this point.
|
|
|
|
|
|
|
|
|
| |
I added this test because I thought that the compiler would find the
forward-declarations due to the "one definition" rule. In hindsight,
it's not a good idea.
Sun CC warns about this, gcc doesn't. With Sun CC, the code leaks,
with gcc it doesn't.
|
|
|
|
| |
But I'm told 5.10 does.
|
|
|
|
| |
Reviewed-by: Trustme
|
|
|
|
| |
Only for obsolete links from non-obsolete things.
|
|
|
|
|
|
|
|
| |
The AdpReader checked for a version attribute == 3.2.0.
This has been replaced by a check for >= 3.2.0
Task-number: 258551
Reviewed-by: kh
|
|
|
|
|
|
|
| |
Exclude more files from the import that are not needed for the Qt build,
and include jsc for qtscript debugging.
Reviewed-by: Trust me
|
|
|
|
|
|
|
|
|
| |
When you have an icon set we do not show the text label,
but the previous code would still use the text for the size
hint calculation.
Task-number: 218836
Reviewed-by: ogoffart
|
|
|
|
|
|
|
|
|
| |
After discussions with product management, it was considered best that
this example be removed until it can be improved. There are some bugs
an irregularities that should be cleaned up, and the theme should be
made less violent.
Reviewed-by: Volker Hilsheimer
|
| |
|
|
|
|
|
|
| |
RevBy: Frans Englich
Details: QHash::value creates an internal compiler error in this case
on Metrowerks, using find instead
|
| |
|
|
|
|
|
|
|
| |
Moved common parts of QHelpSearchIndexReader{Default,Clucene} into
a new common base class QHelpSearchIndexReader.
Reviewed-by: kh
|
|
|
|
|
| |
Solaris seems to define glXReleaseTexImageEXT rather than let it be
resolved as a function pointer.
|
|
|
|
|
|
|
|
|
|
|
| |
Checkboxes would animate when first shown if they were checked.
This is unintentional and looks a bit odd in wizard for instance.
To fix this we simply check if the old state was set. Note that
this is safe because we will at least require the enabled state
flag to be set.
Task-number:253075
Reviewed-by: ogoffart
|
|
|
|
| |
Reviewed-by: Trustme
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
QtConcurrent had the following code:
template <typename T> class ThreadEngineStarterBase
{ ...
protected:
ThreadEngine<T> *threadEngine;
};
template <typename T>
class ThreadEngineStarter : public ThreadEngineStarterBase<T>
{
public:
ThreadEngineStarter(ThreadEngine<T> *threadEngine)
:ThreadEngineStarterBase<T>(threadEngine) {}
[...]
};
The Sun CC compiler simply didn't parse the parameter declaration in
the constructor. Instead of complaining, it silently ignored the
problem. Which meant that the constructor simply used the
uninitialised member variable from the base class to call the parent
constructor, which ended up initialised with itself.
You'd think that it's just because the parameter has the same name as
a member variable. But it appears to be a compiler bug altogether. If
you change the name, then you start getting compile errors.
This change is a workaround that worked.
Reviewed-By: Bradley T. Hughes
|
|
|
|
|
|
|
|
|
|
| |
At least the RogueWave STL found in Sun Studio 12 (latest version
available) isn't standards-compliant enough for QtConcurrent's needs.
Note that WebKit also requires STL support, so this means QtWebKit
will not compile with RW STL on Solaris.
Reviewed-By: Bradley T. Hughes
|
|
|
|
|
|
|
|
|
|
|
|
| |
I had added the version check when we only had CC 5.5 and 5.6,
expecting that 5.7 would have the support. And if it didn't, then
someone would notice the compile error in QtConcurrent, bumping the
version number here.
Except that QtConcurrent was never enabled with Sun CC. Which meant
that we never got to test TTP support.
Reviewed-By: Bradley T. Hughes
|
|
|
|
|
|
|
| |
See ba191b0a26b966ad1fb596a905307399922bc44a for a similar commit done
to QStringMatcher.
Reviewed-By: Bradley T. Hughes
|
|
|
|
|
| |
As pointed out on IRC, setTransform is used most frequently in code and in
an ideal world would be the only such function.
|
| |
|
|
|
|
| |
Reviewed-by: TrustMe
|
| |
|
|
|
|
|
|
|
|
| |
qmake now uses a few of the files from the 'codecs' directory so add
that directory to the VPATH directory such that building qmake using
this .pro file is possible.
Reviewed-by: TrustMe
|
| |
|
|
|
|
|
|
|
|
| |
The header file should be looked up relative to the directory of the
compilation unit and not qmake's current directory or the like. Simplified
the logic slightly to achieve that.
Reviewed-by: João Abecasis <joao@abecasis.name>
|
|
|
|
| |
Need the right include for the mach_*() functions.
|
|\ |
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The arrow was reported to be on the wrong side of the control.
Technically the arrow part seems to cover the whole rect on
Vista and Gtk+ but due to compatibility it is probably safer to keep
the old rects for now.
Task-number: 252857
Reviewed-by: ogoffart
|
| |
| |
| |
| |
| |
| | |
Mac OS X only change
Reviewed-by: Thiago
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Merge the monotonic clock source detection in
qeventdispatcher_unix.cpp with that in qcore_unix.cpp. As discussed
with Thiago, we're also removing older compat code at the same time
(spinning on select() when we think it woke up early).
Reviewed-by: Thiago
|
| |
| |
| |
| |
| |
| | |
QPainter.
Reviewed-by: Simon Hausmann
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
QTransform and respective APIs should be used. Still some changes required
- Some references to QMatrix left in documentation
- Qt code uses QMatrix APIs (ie translationX)
Reviewed-by: Samuel
|
| | |
|
| | |
|
| |
| |
| |
| |
| |
| |
| | |
The comment was speaking about upload progress, however the code
is about download progress.
Reviewed-by: TrustMe
|
| | |
|
| |
| |
| |
| |
| |
| | |
Task: 258573
Reviewed-By: TrustMe
|