1berry/qtbase
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Examples: fix a couple of nodiscard warnings from QFile::open

Browse Source 
In one case, the code simply checked for isOpen afterwards; refactor
it to use QFile::open's result. In another case, a file was opened
from the resource system, so add a check.

Pick-to: 6.9 6.8
Change-Id: I5f4c22dd5ce678f15c9c1609c4228d50a2b32a1d
Reviewed-by: Thiago Macieira <thiago.macieira@intel.com>
This commit is contained in:
Giuseppe D'Angelo 2025-03-25 11:58:36 +01:00
parent 323f408be6
commit eeead68995
2 changed files with 13 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
examples/corelib/serialization/convert/main.cpp
View File

@ -23,12 +23,14 @@ static const Converter *prepareConverter(QString format, Converter::Direction di
: QIODevice::ReadOnly;
const char *dirn = out ? "output" : "input";
if (stream->fileName().isEmpty())
stream->open(out ? stdout : stdin, mode);
else
stream->open(mode);
bool isOpen;
if (!stream->isOpen()) {
if (stream->fileName().isEmpty())
isOpen = stream->open(out ? stdout : stdin, mode);
else
isOpen = stream->open(mode);
if (!isOpen) {
qFatal("Could not open \"%s\" for %s: %s",
qPrintable(stream->fileName()), dirn, qPrintable(stream->errorString()));
} else if (format == "auto"_L1) {

7
examples/widgets/draganddrop/draggabletext/dragwidget.cpp
View File

@ -20,7 +20,12 @@ DragWidget::DragWidget(QWidget *parent)
: QWidget(parent)
{
QFile dictionaryFile(QStringLiteral(":/dictionary/words.txt"));
dictionaryFile.open(QIODevice::ReadOnly);
if (!dictionaryFile.open(QIODevice::ReadOnly)) {
// This would be a build problem, as the dictionary is in the
// resource system.
qFatal("Could not open the dictionary file: %s",
qPrintable(dictionaryFile.errorString()));
}
QTextStream inputStream(&dictionaryFile);
int x = 5;