do not return boolean from QBitArray::fill()

the return value is always true

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

drop support for custom QEasingCurve type

use one of the animation classes instead if such is required

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

search for programs in the standard executables locations in addition to PATH

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

look for the program before exec-ing from QProcess

stat-ing multiple times is much more optimal than exec-ing multiple times,
bonus points for using QStandardPaths::findExecutable() to look for the
program (which should check for it in ~/.local/bin too)

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

version bump

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

remove redundant QTextControl::event() reimplementation

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

use correct QString constructor for expected data in tst_QJsonDocument::read()

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

always use QImage for QTextDocument resources

now that QPixmap always uses QImage (internally) it's better to use QImage
directly

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

check the time even if the serial is the same in QKeyMapper::translateKeyEvent()

apparently X11 sends key events with same serial even seconds after (if the
key is the same), also now only key presses will be considered
auto-repeated because X11 sends (fake/simulated) key release events

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

move the static variable outside the conditional code in QKeyMapper::translateKeyEvent()

fun fact: it is very easy to break auto-repeat code by reparenting QWidget
from a QPushButton event (KMediaWidget does it)

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

correct memcmp() return value checks in QTextConverter::toUnicode()

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

return if ubrk_safeClone() fails in QTextBoundaryFinder assignment operator

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

call ubrk_following() and ubrk_preceding() with the current QTextBoundaryFinder position

I remember ubrk_following() and ubrk_preceding() causing a crash if the
position is not within the text bounds so leaving the -1 checks just in
case

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

check for self-assignment in QTextBoundaryFinder assignment operator

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

handle strings as UTF-8 encoded from QJsonDocument

the default QString codec is US-ASCII

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

chop BOMs from QTextConverter::toUnicode()

fun fact: anything that is checking for BOMs is either not passing them
to QTextCodec/QTextConverter (e.g. Kate) when converting or was just using
QTextCodec::IgnoreHeader flag (QXmlStreamWriter used to)

okular xps generator now works btw (xml files in .xps files seem to be
UTF-16 encoded with BOM)

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

QPngHandler::write() optimization

why allocate when it can be done without?

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

inherit std::bitset<T> as private base class of QTimersSet

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

correct indendations in qslider source file

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

remove unused QSizeGripPrivate member

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

remove unused definitions in qsizegrip source file

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

correct break scope in QDockWidget::event()

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

QApplication and QWidgetAnimator friendship has been broken

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

partial build fix for the case when QT_NO_COLORDIALOG is defined

qtconfig tool requires QColorDialog, that's why it is broken

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

QT_NO_FILEDIALOG config option is no longer broken

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

QT_NO_FONTDIALOG config option is no longer broken

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

correct indentation in QWindowsStyle::drawPrimitive()

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

remove redundant check in QEasingCurve::setType()

since QEasingCurve instances are no longer shared and no detaching is done
the check that was optimization to avoid detaching is no longer worth it

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

correct repo directory name in crossdeb.sh script

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

QFontDatabase review

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

QEasingCurve review

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

move the QEasingCurve::Custom case to the switch in QEasingCurve::valueForProgress()

side note: only plasma and KWin set custom QEasingCurve function, both are
not used

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

translations update

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

remove resource system leftovers

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

correct debug message in QTextStreamPrivate::flushWriteBuffer()

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

write BOM only from QTextStream, on demand

ICU finally does not generate BOMs (since idk when), I guess someone took
note. chopping BOMs from the result may have to be done only if the input
data had BOM (for compat) but let's see if that is even needed

Kate encoding tests pass now btw

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

consistent QTextConverter::ConversionFlag enums

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

remove most of the QTextStream manipulators and the ambiguous void pointer operator

most of them are unused since QTextStream is used mostly for streaming to
QFile, besides that the streaming of void pointer could result in streaming
in std::endl function pointer address or some other non-sense

I will not gut-out QTextStream yet (not in this release cycle) because I
know for a fact such change will break things that use the declarative
component (such as plasma and its applets due to nan/inf compat code in
QTextStream)

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

test QTextConverter::hasFailure() in QTextCodec test

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

call UCNV_FROM_U_CALLBACK_SUBSTITUTE() and UCNV_TO_U_CALLBACK_SUBSTITUTE() with NULL as context

what a surprise it is to learn that both function replace characters only
in some cases (when context is NULL or some internally set context)

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

someone's typo

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

copy the other object position from QTextBoundaryFinder copy-constructor

as done by the assignment operator

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

QTextBoundaryFinder documentation review

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

remove unused HB_MIN(), HB_MAX() and HB_FIXED_CONSTANT() macros defined in qharfbuz_p header

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

QEasingCurve::valueForProgress() optimization

doing qreal (either float or double) math on demand is much more optimal,
also reducing the switches to one is a bonus

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

QEasingCurve::InCurve, QEasingCurve::OutCurve, QEasingCurve::SineCurve and QEasingCurve::CosineCurve are deffinetly not internal

used by plasma system-monitor applet for example

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

QEasingCurve documentation review

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

call FatalSignalHandler constructor

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

QOpacityEffect review

by starting the elapsed timer just before the fade timer the opacity
transition will be smoother at the start of the effect

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

explicitly convert the elapsed time to double

QElapsedTimer::elapsed() returns qint64 btw

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

do not send move and resize events to the widget from qFadeEffect()

the position and size of the widget matter not to the effect

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

drop effects other than fade

QComboBox animation was essentially non-operational (see
5b5fbca2ed070acc4ed37fe20fc1d16532469aef), UI_AnimateToolBox was just
reserved and unused . there is fade effect for QMenu and QToolTip so
the relevant effect is there

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

QOpacityEffect optimization

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

set the global static variables before deleting the effect instances in qScrollEffect() and qFadeEffect()

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

snapshot the widget as QImage from QRollEffect

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

remove declaration of now non-existing QRollEffect::run() method

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

actually animate QComboBox

for some reason the code for it was Q_WS_WIN && !QT_NO_EFFECTS guarded and
I purged Q_WS_WIN code programatically

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

merge QRollEffect::run() into QRollEffect constructor

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

cancel the opacity/fade effect when the widget is destroyed

widgets are supposed to call qFadeEffect(0) to cancel the effect but just
in case

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

fade effect class rewrite to not use QWidget and event filter

since the effect sets a X11 window property the only thing it has to care
about is if the widget pointer is valid and the elapsed time. moving,
resizing the widget, etc. do not matter

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

remove unused graphics effects feature

doubles as performance optimzation

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

trc man page correction

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

apply style from settings only if it is not empty and the style override is empty from QApplicationPrivate::x11_apply_settings()

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

QApplication::setStyle() documentation update

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

optimize QAlphaWidget (fade effect) further by using window opacity

relies solely on the compositor for it and assumes window opacity is not
set by the user, Katana has animation override
(_KDE_NET_WM_SKIP_CLOSE_ANIMATION X11 atom) and the Katie fade effect
should probably have such (or just check if the opacity value is not 1.0
before running the effect) because effects may overlap with window manager
/compositor effects resulting in all sort of bad things

for that reason I am considering dropping the effects that alter the way
windows/widgets show, hide, are moved, etc. altogether and leaving that
to the window manager/compositor. effects such as colorization of pixmap
and grayscaling (used in QIcon) however are unlikely to be removed

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

optimize QAlphaWidget by avoiding QImage to QPixmap conversion

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

use icons from the system in scripttools component

less static data in the library and better looks - can't go wrong with that

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

remove unused scripttools component images

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

remove unused qicon header inclusion in qscriptbreakpointsmodel source file

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

correct indendation in QKeyMapper::translateKeyEvent()

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

fix ODR-violation in md5c and sha2 source files

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

remove qbitmap source file from the unity exclude list

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

simplify QKeyMapper::clearMappings() and do not use the cached locale codec

since it is known what locale the X11 input method uses it is redundant to
verify that via XLocaleOfIM() (unless X11 fails to detect what locale is
set for some strange reason).

also not using the cached codec returned by QTextCodec::codecForLocale()
because the locale may change at any time (either environment variable
change or just plain setlocale() call as was done by KCatalog as gettext
workaround) and QKeyMapper has to reflect such changes

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

QFontDatabase optimizations

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

do not set the internal QApplicationPrivate::styleOverride from QApplicationPrivate::x11_apply_settings()

the style name is either from the settings or the platform plugin, not from
-style argument

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

show preview even of the desktop style

the desktop style is now determined by QT_PLATFORM_PLUGIN and
DESKTOP_SESSION environment variables with fallback to cleanlooks style so
even if it is the fallback style there is something to show but if it is
not the fallback the user will be able to preview the style chosen by
QGuiPlatformPlugin

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

lock QSettings files via flock()

now QSettings will behave much like opening a file via open() and the like,
see the QFile hijack test case

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

add QSidebar test

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

effectively revert c3c697ea5b4e141f08c4607955f085c49667b806

it being static means its destruction will be delayed until exit and it
holds shared data references the meta types of which may be unregistered by
the time it is destroyed

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

use patternForFont() in getFcPattern() and rename getFcPattern()

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

fix regression since 81cc2609442d5ee1e445040d9863e2538904a7f7

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

build fix for the case when QT_NO_TEMPORARYFILE is defined

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

QT_NO_WIZARD config option is no longer broken

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

adjust expected result in QFontDatabase test case

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

query all fonts to get the styles for font family from QFontDatabase::styles()

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

check if qt_x11Data pointer is valid before using its members from QFontDatabase

qt_x11Data pointer is valid only if QApplication instance is created, fixes
possible crashes

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

remove now unused QFontFamily struct

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

compiler warning fix

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

plug QFontDatabase memory leak

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

compiler warning fix

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

QFontDatabase review

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

query fontconfig directly from QFontDatabase

since fontconfig has its own cache mainting internal QFontDatabase cache
is kinda redundant, have to leak check still but font tests pass

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

inline qt_pixelSize() function

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

scale the fontconfig weight value to QFont::Weight in QFontDatabase

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

remove compat QFontDatabase methods

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

de-duplicate QFontDatabase code

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

add case-sensitivity test for QFontDatabase

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

de-duplicate QFontDatabase code

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

const-ify QFontDatabase::styleString() methods

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

test case for converting QFont from and to QString

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>