diff options
author | Giuseppe D'Angelo <[email protected]> | 2025-03-25 11:58:36 +0100 |
---|---|---|
committer | Giuseppe D'Angelo <[email protected]> | 2025-04-24 10:44:03 +0100 |
commit | eeead68995653ab38e43e9f1494feaaa679dfd07 (patch) | |
tree | d27faff34eee926996cabab0d2fafee17c00745d | |
parent | 323f408be6bcf5bfa5e2d447be94381556e52a62 (diff) |
Examples: fix a couple of nodiscard warnings from QFile::open
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 <[email protected]>
-rw-r--r-- | examples/corelib/serialization/convert/main.cpp | 8 | ||||
-rw-r--r-- | examples/widgets/draganddrop/draggabletext/dragwidget.cpp | 7 |
2 files changed, 11 insertions, 4 deletions
diff --git a/examples/corelib/serialization/convert/main.cpp b/examples/corelib/serialization/convert/main.cpp index d3021fadca2..aae20eb4a9a 100644 --- a/examples/corelib/serialization/convert/main.cpp +++ b/examples/corelib/serialization/convert/main.cpp @@ -23,12 +23,14 @@ static const Converter *prepareConverter(QString format, Converter::Direction di : QIODevice::ReadOnly; const char *dirn = out ? "output" : "input"; + bool isOpen; + if (stream->fileName().isEmpty()) - stream->open(out ? stdout : stdin, mode); + isOpen = stream->open(out ? stdout : stdin, mode); else - stream->open(mode); + isOpen = stream->open(mode); - if (!stream->isOpen()) { + if (!isOpen) { qFatal("Could not open \"%s\" for %s: %s", qPrintable(stream->fileName()), dirn, qPrintable(stream->errorString())); } else if (format == "auto"_L1) { diff --git a/examples/widgets/draganddrop/draggabletext/dragwidget.cpp b/examples/widgets/draganddrop/draggabletext/dragwidget.cpp index d56f91e9001..4142a564356 100644 --- a/examples/widgets/draganddrop/draggabletext/dragwidget.cpp +++ b/examples/widgets/draganddrop/draggabletext/dragwidget.cpp @@ -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; |