* Replace Q_DECL_OVERRIDE with override
* Init all POD members
* Update includes to common style and remove unneded ones

Task-number: QTBUG-60652
Change-Id: Ibb5c0347bd14cfc0386f39059498edd384a7200c
Reviewed-by: Denis Shienkov <denis.shienkov@gmail.com>

Conflicts:
	.qmake.conf

Change-Id: Ie7fa76a9fb0013106e5a21b587585c8ccd7339d5

Change-Id: Ib57b52598e2f452985e9fffd1459eb18587c99e3
Reviewed-by: Joerg Bornemann <joerg.bornemann@qt.io>

Change-Id: I36eef79e45e4dbc030b738d4453d7727d3418b48

Change-Id: I86804f418157aaa5fba0614bd68abb1d504aaa9c

Binary compatibility files updated for Qt5.9.0

Change-Id: Ib1009be62ab89954ea7221de88d8c0a748c8c017
Reviewed-by: Sergio Ahumada <sahumada@texla.cl>

Change-Id: I4b203538574e67c33c45a295b79bb13f6dcd244c
Reviewed-by: Alex Trotsenko <alex1973tr@gmail.com>
Reviewed-by: Denis Shienkov <denis.shienkov@gmail.com>

Conflicts:
	.qmake.conf
	src/serialport/qserialport_win.cpp

Change-Id: Icc832e218a1135cb82e780ccaa63abd990289eaf

Change-Id: Ie6c5829bb5816b31a337976228dd218767c7c87d

... as there are plans to remove it from the QtCore.

Change-Id: I79a2250b7b392896fa1c7307488a74dd44d62736
Reviewed-by: Joerg Bornemann <joerg.bornemann@qt.io>
Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@qt.io>

Listing important changes and bug-fixes.

Change-Id: If72171427d5cf3de39f9404e15dc3b394d78a265
Reviewed-by: Alex Blasche <alexander.blasche@qt.io>

Change-Id: Ifc7f1e60966489b0352ef326a7fa5d141eee8d8f

'bytesToRead' takes the result of signed 64-bit computation which could
be negative. So, its type should be qint64.

Change-Id: Ib51129143fcb70475ad383df417ae4282c858e41
Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@qt.io>

Read notifications should be enabled all the time while the device is
opened unless the buffer size limit is reached. To enable the read
notifier again, the user needs to read the data or enlarge the buffer
with a setReadBufferSize() call.

According to that scenario, it's very likely that the device will
toggle the state of notifier twice (on->off->on) in one notification
cycle. This patch prevents this unnecessary toggling by:

- unconditionally setting the notifier's state in setReadBufferSize()/
  readData();
- deferring the notification disabling in readNotification() on Unix
  and in completeAsyncRead() on Windows to the next notification
  cycle.

Change-Id: I97fc041bb705c034a7e73d1437e64b9b34dc2c18
Reviewed-by: Denis Shienkov <denis.shienkov@gmail.com>

This reduces the number of syscalls and thus the CPU load when dealing
with fast ports. And 32KiB seems still small enough to be no concern on
modern hardware even if it's excessive for a given port.

Change-Id: I4749e4255627d3abf2393323216e01ac6436413b
Reviewed-by: Alex Trotsenko <alex1973tr@gmail.com>
Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@qt.io>
Reviewed-by: Denis Shienkov <denis.shienkov@gmail.com>

This patch replicates commit 225ef1ba


for Windows.

Change-Id: Ic4e3444183289e7ce1a487500633fb5ad95590b9
Reviewed-by: Denis Shienkov <denis.shienkov@gmail.com>

If the user calls setReadBufferSize() with a value lesser than the
current size of the read buffer, then we get UB in readNotification():

  readFromPort(nullptr, some_negative_value);

Change-Id: Iddf5838c84c8615670285e9883c48f29faacc07e
Reviewed-by: Denis Shienkov <denis.shienkov@gmail.com>

We should not add the size of the internal write buffer here because
QIODevice::bytesToWrite() already checks it. Also, take into account
the size of 'writeChunkBuffer' on Windows.

Change-Id: I87dfcb4dd1cd9d2bbb6f82a084d79fb4db9323d8
Reviewed-by: Thiago Macieira <thiago.macieira@intel.com>

We do not need to use the ifdef guards for the CBR_ constants,
because constants are standard and always exists in native API.

Change-Id: I75ae9ffc17b132786fe5e72791b3f7c82c72c76e
Reviewed-by: Anton Kudryavtsev <antkudr@mail.ru>
Reviewed-by: Denis Shienkov <denis.shienkov@gmail.com>

... otherwise, this notifier's instance will be active even
if the serial port is not open.

Change-Id: Ieebb894a048f3ed89ac5d1c259e21bc790f5d387
Reviewed-by: Anton Kudryavtsev <antkudr@mail.ru>
Reviewed-by: Denis Shienkov <denis.shienkov@gmail.com>

The event mask variable is used only once in process of initialization,
so we do not need to keep this variable as the class member.

Change-Id: I1f97a312cc92345d435dee821fefb8c2de33665e
Reviewed-by: Anton Kudryavtsev <antkudr@mail.ru>
Reviewed-by: Denis Shienkov <denis.shienkov@gmail.com>

Change-Id: Ia9b0851ea7259bc4bc1aa709992b5eb215144a77
Reviewed-by: Anton Kudryavtsev <antkudr@mail.ru>
Reviewed-by: Denis Shienkov <denis.shienkov@gmail.com>

Change-Id: I0dc20fa836888d3caa678fd8f5b620625e00ec92
Reviewed-by: Anton Kudryavtsev <antkudr@mail.ru>
Reviewed-by: Denis Shienkov <denis.shienkov@gmail.com>

This module is not yet supported on INTEGRITY and
therefore we'll exclude it from the CI build.

Change-Id: Ia08354f0680865f0a2e08a3a3e3f59a88e98bd4e
Reviewed-by: Simon Hausmann <simon.hausmann@qt.io>

... as this function is implemented in cpp file.

Change-Id: Ia5a0c262ebdb4b98b4575c85ab45fdc93742731f
Reviewed-by: Leena Miettinen <riitta-leena.miettinen@qt.io>

Change-Id: Ib336529542ae9b78937ad1573fde71ccfcc4c1ad

The limitations were introduced by the commit 56c001b4

, but were not
documented.

Change-Id: I5e44ab78e2e5cce2477ebc1f83280fa8c22e0ef6
Reviewed-by: Topi Reiniö <topi.reinio@qt.io>

Change-Id: Ie98fd0ed952ab6d863b02b5de1e93a205871b6db

Change-Id: I77f015100f2b9ed5e1a16ebd1ba0b28216282085
Reviewed-by: Denis Shienkov <denis.shienkov@gmail.com>

If it doesn't work on iOS, it won't work on those platforms either.

Change-Id: Ie4109eda9ce83339435fb3e221e4bcbbeadac821
Reviewed-by: Denis Shienkov <denis.shienkov@gmail.com>

Added binary compatibility file for qtserialport

Change-Id: Ic676c4a92f798746e43e11bd61deb67a3ad9031b
Reviewed-by: Denis Shienkov <denis.shienkov@gmail.com>

Change-Id: Iaf9a626f80652a3b60568dc0ed61d958107713cc

Change-Id: I589b584f7929c781c814a2e8da680917ec34012f

Conflicts:
	.qmake.conf

Change-Id: Ie713faf0d6395f11477fceb35c93388756e33fae

... where has been added the '\c' tag where it is required.

Change-Id: Ie346fc147deb535501eb365693a2ff5fe22a98f9
Reviewed-by: Sergey Belyashov <Sergey.Belyashov@gmail.com>
Reviewed-by: Alex Blasche <alexander.blasche@qt.io>

Change-Id: Ia0a76e03ab9484c6664615c67a66c8696eabae06

Change-Id: Ib60b1ce21226d5702ee7c999508a484d9596c9fd
Reviewed-by: Alex Blasche <alexander.blasche@qt.io>

qserialport.cpp: In member function 'void QSerialPort::setSettingsRestoredOnClose(bool)':
qserialport.cpp:627:9: warning: 'void QSerialPort::settingsRestoredOnCloseChanged(bool)' is deprecated [-Wdeprecated-declarations]

Change-Id: I4ed17403cd1692c4f965a69be711f0c5b03a8376
Reviewed-by: Sergey Belyashov <Sergey.Belyashov@gmail.com>
Reviewed-by: Denis Shienkov <denis.shienkov@gmail.com>

This patch corrects the connect statement used to show the settings
dialog.

Change-Id: I6318e18f7a281d8ddb118ef1f1d2d18e86df9b22
Reviewed-by: Alex Blasche <alexander.blasche@qt.io>

Change-Id: Ia83609bd366762621857a6c63bf935a6cd230742