| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
| |
With proper checks for boundary conditions, we no longer need to protect
against reaching the terminating null character.
Reviewed-by: Robin Burchell
|
|
|
|
|
|
|
|
|
|
| |
Checking of boundary condition needs to happen whenever there is a
change. Checking anywhere else is useless.
This fixes an issue where we would overrun the placeholder if there was
a 'Z' next to the placeholder string.
Reviewed-by: Robin Burchell
|
|
|
|
|
|
|
|
|
| |
This removes knowledge about the structure of the filename template from
the randomization process and is a pre-requisite to adding Unicode
support to the code.
Task-number: QTBUG-4796
Reviewed-by: Robin Burchell
|
|
|
|
|
|
|
|
|
| |
The duplication was there to allow qmake and other tools to compile
without a full build of QtCore. Cleaning up this code justifies losing
the benefit of PID-tagged temporary files in those tools.
Reviewed-by: Olivier Goffart
Reviewed-by: Robin Burchell
|
|
|
|
|
|
|
| |
There should be no semantic differences, otherwise.
Reviewed-by: Harald Fernengel
Reviewed-by: Robin Burchell
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
On Posix platforms, when attempting to atomically create file and obtain
descriptor, we only continue if we get an EEXIST error. If the path
points to a non-existing directory, we should get ENOENT and will fail
as well.
On Windows, the check was already ignored. We would detect the file does
not exist, attempt to obtain a handle upon return to
QTemporaryFileEngine::open and fail there without running "for a *very*
long time".
Checking if the directory exists beforehand is not only unnecessary it
also constitutes a race condition. Trusting the result of this check is
dangerous and doubly useless.
Reviewed-by: Olivier Goffart
Reviewed-by: Robin Burchell
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
There are exactly 2 uses of the _gettemp function: one for windows
platforms, one for all others. Windows requests a file name, others
request a valid file descriptor for that file name.
Despite the 2 distinct use cases, the code was full of #ifdefs to keep
never used code paths compiling.
With this change the qt_mkstemp wrapper is eliminated, together with
dead code inside if (doopen) {} blocks on Windows platforms. The more
proper Q_OS_WIN macro is preferred to Q_WS_WIN.
_gettemp is also changed to return the valid file descriptor on
non-Windows platforms.
Reviewed-by: Olivier Goffart
Reviewed-by: Robin Burchell
|
|
|
|
|
|
|
|
| |
The option to create a directory is never used, so let's drop it. There
are bigger issues with this code.
Reviewed-by: Olivier Goffart
Reviewed-by: Robin Burchell
|
| |
|
|
|
|
|
|
|
|
| |
... to also verify template prefix and corner cases with runs of less
than 6 Xs.
Reviewed-by: Olivier Goffart
Reviewed-by: Robin Burchell
|
|
|
|
|
|
| |
Task-number: QTBUG-4796
Reviewed-by: Olivier Goffart
Reviewed-by: Robin Burchell
|
|
|
|
|
|
| |
Reviewed-by: Olivier Goffart
Reviewed-by: Harald Fernengel
Reviewed-by: Robin Burchell
|
|
|
|
|
| |
because the qobject_cast function do not have the right to access the
staticMetaObject.
|
|
|
|
| |
Reviewed-by: Trust me
|
|
|
|
| |
Reviewed-by: Trust me
|
|
|
|
| |
RevBy: Liang Qi
|
|
|
|
|
|
|
| |
changes include whitespace, parenting, recovering erroneously deleted
code etc.
Reviewed-by: Markus Goetz
|
|
|
|
|
| |
Merge-Request: 2505
Reviewed-by: Peter Hartmann
|
|
|
|
|
|
|
|
|
|
| |
Faster disk cache for mobile devices. Reduce file-system touching
operations and work around FAT performance issues. Features new
on-disk layout. Cached objects are now saved in multiple
subdirectories and filenames are shorter in length.
Merge-Request: 2505
Reviewed-by: Peter Hartmann
|
|
|
|
|
|
|
|
|
| |
Some of the text can start from subpixel positions (start from
100.5 for instance), we need to make sure they stick to pixel grid
in this case.
Reviewed-by: Eskil
(cherry picked from commit bf03b38d2b32a0cfca181398597884e4252466f4)
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
stringbuilder.cpp(160) : error C2666: 'operator +' : 2 overloads have similar conversions
stringbuilder.h(416): could be 'QStringBuilder<A,B> operator +<QByteArray,char[5]>(const A &,const B (&))'
with
[
A=QByteArray,
B=char [5]
]
or 'built-in C++ operator+(__int64, char [5])'
while trying to match the argument list '(QByteArray, char [5])'
The reason is the cast to QNoImplicitBoolCast (which is int)
|
| |
|
|
|
|
|
| |
Reviewed-by: Denis Dzyubenko
Task-number: QTBUG-17099
|
| |
|
|
|
|
| |
Reviewed-by: Denis
|
|
|
|
|
|
|
|
|
| |
This breaks source compatibility if one made its own QConcatenable
as it nows require a Prefered type.
And also sometimes if QT_USE_FAST_OPERATOR_PLUS was used, and
the result of an addition between two QByteArray is used directly
Reviewed-by: Denis
|
|
|
|
|
|
|
|
|
| |
Merge-request: 1162
Reviewed-by: João Abecasis <joao.abecasis@nokia.com>
[ Symbols for QZipReader were previously exported with
959ab5d84d67ec84c2342b083e362ca75613e8cb. There's no good reason not to
do the same for QZipWriter. Classes remain private API. -- João ]
|
|
|
|
|
|
|
|
|
| |
In this case we can't use line width to align the text, we have to
use the rectangle width given in QPainter::drawText for alignment
instead.
Reviewed-by: Eskil
(cherry picked from commit b92a31da01ee50fed4b10282e79f812167faf659)
|
|
|
|
|
|
|
|
|
|
| |
Resets currentTestObject if an exception is caught while executing a
test suite. This is required or else subsequent test executions will
trigger an assert.
Task-number: QTBUG-3869
Merge-request: 1163
Reviewed-by: João Abecasis <joao.abecasis@nokia.com>
|
|
|
|
|
|
|
|
| |
on some systems (i.e. on SPARC), the order of IPC-related includes is
meaningful; sys/types.h, then sys/ipc.h, then sys/sem.h, and so on
Merge-request: 2586
Reviewed-by: João Abecasis <joao.abecasis@nokia.com>
|
|
|
|
|
|
|
|
|
|
|
| |
according to X/OPEN we have to define semun ourselves but on some
systems sem.h will have it, so let's use a prefixed one (qt_semun).
this allows to avoid the code duplication and makes things a bit easier
to undesrtand
Merge-request: 2586
Reviewed-by: João Abecasis <joao.abecasis@nokia.com>
|
|
|
|
|
|
|
| |
Make the get() function inline
and simplify the hash computation
Reviewed-by: brad
|
| |
|
|
|
|
|
| |
Reviewed-by: brad
Reviewed-by: thiago
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The virtual QObject::qt_metacall will recurse to the whole object hierarchy
to find from which class a function should be called.
But it is possible to know, at connection time, from which exact
QMetaObject a function belongs, and the relative offset into it.
So we make the slot calls from the qt_static_metacall function.
So activation of signals is faster.
- We must not call a slot from a class that has been destroyed.
To avoid this, there is a check on the methodOffset. If it is
smaller, that means we might be called (indirectly) from
the destructor. We fallback to the virtual call to qt_metacall
that does the right thing.
- The signature of the static method is void (*) (QObject*,MetaCall,int,void**)
It returns void, so the compiler is allowed to do tail recusive
optimization. Having the QObject* as first parameter make it ready
on the stack for the call to the member function.
- The new static method has to be a member function in order to
be able to access the private slots.
Reviewed-by: brad
|
|
|
|
|
|
|
| |
Avoid the overhead of QObjectConnectionListVector::at(), when we
know anyway which list to take.
Reviewed-by: brad
|
|
|
|
| |
Testing that slot from destroyed class are not called
|
|
|
|
|
|
|
| |
QThreadData::current is too slow, it needs to access the TLS
The currentThreadId is much faster.
Reviewed-by: Brad
|
|
|
|
|
| |
Merge-request: 1011
Reviewed-by: João Abecasis <joao.abecasis@nokia.com>
|
|
|
|
|
| |
Merge-request: 1011
Reviewed-by: João Abecasis <joao.abecasis@nokia.com>
|
|
|
|
|
| |
Merge-request: 1011
Reviewed-by: João Abecasis <joao.abecasis@nokia.com>
|
|
|
|
|
|
|
|
|
|
| |
If a compositing manager is active, setting
the window pixmap background to None for ARGB windows
makes little sense.
In this case the background should be transparent
Merge-request: 1135
Reviewed-by: Denis Dzyubenko <denis.dzyubenko@nokia.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Use XFixes extension to track changes to
the compositing manager in such a way that
QX11Info::isCompositingManagerRunning()
returns a valid result even if compositing has
changed after the creation of the QApplication
Many thanks to Fredrikh Hoglund for suggesting the
correct approach
Task-number: QTBUG-6044
Merge-request: 1133
Reviewed-by: Denis Dzyubenko <denis.dzyubenko@nokia.com>
|
|
|
|
|
|
|
| |
This fix the qnetworkreply::ioPostToHttpsUploadProgress() auto test.
Before the readbuffer where always limited to 1k for ssl sockets.
Reviewed-by: Markus Goetz
|
|\ |
|
| |\
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
master-integration
* 'master' of scm.dev.nokia.troll.no:qt/oslo-staging-1:
Unix configure: Add support for --sysroot
Add a 'capabilities' flag to the marshaller and demarshaller
Add a set of connection capabilities to QDBusConnection
Move the main D-Bus session and system connections to the main thread
Add a way to obtain the DBusConnection* pointer from a QDBusConnection
Make the QIconvCodec on Unix not enforce the Latin1 codec.
|
| | |
| | |
| | |
| | | |
Reviewed-By: axis
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The marshaller and demarshaller need to know the features negotiated
with the peer so as to determine what's permitted. This is especially
important to the marshaller: the libdbus-1 API may "throw a fit" if we
try to pass a type that isn't allowed or the server may disconnect us.
The use of the capabilities in the demarshaller are for symmetry and
for us to toggle the feature in unit tests.
Task-number: QTBUG-17478
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The capabilities are negotiated with the D-Bus peer or bus at
connection time and may include extra features not available when
D-Bus 1.0 was released.
Currently (as of D-Bus 1.4), the only additional feature is Unix file
descriptor passing.
Proposed features are "maybe types" and single-precision floating
point. They may be added to D-Bus 1.6.
Task-number: QTBUG-17478
|
| | |
| | |
| | |
| | |
| | |
| | | |
Some applications experience weird behaviour that sometimes it works and
sometimes it doesn't, due to D-Bus being first used in a thread. So
instead do everything in the main thread for the two main connections.
|