summaryrefslogtreecommitdiffstats
path: root/src/assistant/qhelpgenerator/helpgenerator.cpp
Commit message (Collapse)AuthorAgeFilesLines
* Change license for tools filesLucie Gérard2024-04-091-1/+1
| | | | | | | | | | | | According to QUIP-18 [1], all tools files should be LicenseRef-Qt-Commercial OR GPL-3.0-only WITH Qt-GPL-exception-1.0 [1]: https://contribute.qt-project.org/quips/18 Pick-to: 6.7 6.7.0 Task-number: QTBUG-121787 Change-Id: I7b69118185d4237ec096dc76e78ee84c2ccd5eb2 Reviewed-by: Kai Köhne <kai.koehne@qt.io>
* QtHelp: Bit more cleanup in cpp filesJarek Kobus2024-02-131-1/+1
| | | | | | | | | | | | Inline some short methods in private API. Use more {} for default constructed values. Fix indentations. Add some TODOs. Task-number: QTBUG-122025 Change-Id: I29062709f6d302a8768ead1c20eda3af5c256c74 Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org> Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io>
* Port from qAsConst() to std::as_const()Marc Mutz2022-10-061-9/+9
| | | | | | | | | | | | | | | We've been requiring C++17 since Qt 6.0, and our qAsConst use finally starts to bother us (QTBUG-99313), so time to port away from it now. Since qAsConst has exactly the same semantics as std::as_const (down to rvalue treatment, constexpr'ness and noexcept'ness), there's really nothing more to it than a global search-and-replace. Task-number: QTBUG-99313 Change-Id: I234704ba429750ddee958a82f6c941d041da0653 Reviewed-by: Jarek Kobus <jaroslaw.kobus@qt.io> Reviewed-by: Ivan Solovev <ivan.solovev@qt.io>
* Port from container::count() and length() to size()Marc Mutz2022-10-061-5/+5
| | | | | | | | | | | | | | | | | | | | | | This is a semantic patch using ClangTidyTransformator as in qtbase/df9d882d41b741fef7c5beeddb0abe9d904443d8: auto QtContainerClass = anyOf( expr(hasType(cxxRecordDecl(isSameOrDerivedFrom(hasAnyName(classes))))).bind(o), expr(hasType(namedDecl(hasAnyName(<classes>)))).bind(o)); makeRule(cxxMemberCallExpr(on(QtContainerClass), callee(cxxMethodDecl(hasAnyName({"count", "length"), parameterCountIs(0))))), changeTo(cat(access(o, cat("size"), "()"))), cat("use 'size()' instead of 'count()/length()'")) a.k.a qt-port-to-std-compatible-api with config Scope: 'Container', with the extended set of container classes recognized. Change-Id: I95f6410e57a6a92b1cf91bbedfbe3d517cab6b44 Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org> Reviewed-by: Kai Koehne <kai.koehne@qt.io> Reviewed-by: Friedemann Kleint <Friedemann.Kleint@qt.io>
* Ensure FileAttributeSetTable is filled ordered so we get the same qch file ↵Albert Astals Cid2022-06-191-1/+3
| | | | | | | | | | each time This is part of making all of Qt reproducible Change-Id: I1a4120cab6844887fb4b48edb238b8f1b55f0eb9 Reviewed-by: Kai Koehne <kai.koehne@qt.io> Reviewed-by: Friedemann Kleint <Friedemann.Kleint@qt.io>
* Use SPDX license identifiersLucie Gérard2022-05-191-38/+2
| | | | | | | | | | | Replace the current license disclaimer in files by a SPDX-License-Identifier. Files that have to be modified by hand are modified. License files are organized under LICENSES directory. Task-number: QTBUG-67283 Change-Id: I5335388c0472b0ee554234fc7eca60769e504660 Reviewed-by: Jörg Bornemann <joerg.bornemann@qt.io>
* tools: do not rely on transitive includesFabian Kosmale2022-03-181-0/+1
| | | | | Change-Id: Ib78f984827988a39f56c4423eae40c67797549ea Reviewed-by: Jörg Bornemann <joerg.bornemann@qt.io>
* Fix qttools build after include cleanup in qtbaseVolker Hilsheimer2022-03-181-0/+1
| | | | | | | | QMap and QHash need to be explicitly included. Pick-to: 6.3 Change-Id: I43de3cdf0911c5ed5d86a2107c15cc32d94da30a Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io>
* qhelpgenerator: Fix infinite loop issue in checkLinksVenugopal Shivashankar2022-03-161-1/+1
| | | | | | | | | | | | | | | | | | | | | The while loop that looks for the next match, depends on the offset to move to the next match. Even before this offset is updated, the loop is continued if the first capture group of the `linkPattern` regex contains the sequence "://", making the loop to iterate over the same match. Getting all the matches at once and iterating over them is probably a better alternative. Amends 85803440b28aadd3f4fe3e99a52e52378671b9e0 Pick-to: 6.2 6.3 Fixes: QTBUG-101070 Change-Id: I0acd4428244398ccfbb88e042229a928c234b951 Reviewed-by: Topi Reiniö <topi.reinio@qt.io> Reviewed-by: Jarek Kobus <jaroslaw.kobus@qt.io> Reviewed-by: Kai Koehne <kai.koehne@qt.io>
* Update QStringConverter usageMårten Nordheim2020-10-121-5/+5
| | | | | | | Following fa8d021fa6fcb040fb702b6ffd2deee52a3b748a in qtbase Change-Id: I46495c1f0cde4e304e414b05cd66d14b00afd6b1 Reviewed-by: Lars Knoll <lars.knoll@qt.io>
* Remove QTextCodec dependency from assistantLars Knoll2020-05-181-9/+10
| | | | | | | | Use QStringConverter instead and remove some code where QStringConverter provides equivalent functionality. Change-Id: I88318411d90de979f943428fb27fc3e044d9f3ea Reviewed-by: Liang Qi <liang.qi@qt.io>
* Port assistant to use QRegularExpressionLars Knoll2020-05-091-5/+7
| | | | | | Change-Id: Ie512c5f80d90bb44848d7cc0959ede67d7428b7a Reviewed-by: Alex Blasche <alexander.blasche@qt.io> Reviewed-by: Samuel Gaist <samuel.gaist@idiap.ch>
* Fix deprecation warnings about deprecated container conversionsFriedemann Kleint2019-05-231-5/+5
| | | | | | | | | Fix warnings introduced by qtbase/92f984273262531f909ede17a324f546fe502b5c. Change-Id: Iaca85ad36591f7208f63305b885e7ff59c014a72 Reviewed-by: Christian Ehrlicher <ch.ehrlicher@gmx.de> Reviewed-by: Lars Knoll <lars.knoll@qt.io>
* QHelpGenerator: add missing QRegExp includesSamuel Gaist2018-12-061-0/+1
| | | | | | | | Following qtbase QRegExp include cleanup, this patch fixes the missing includes. Change-Id: Ida96fc67ead6723ec738442b1e9cffd1206ea9e8 Reviewed-by: Kai Koehne <kai.koehne@qt.io>
* Fix division by zeroJesus Fernandez2018-08-201-3/+3
| | | | | | | | | | | CID 191124 (#1 of 1): Division or modulo by float zero (DIVIDE_BY_ZERO) 3. divide_by_zero: In expression 60. / (double)numberOfFiles, division by expression numberOfFiles which may be zero has undefined behavior. CID 191125 (#1 of 1): Division or modulo by float zero (DIVIDE_BY_ZERO) 3. divide_by_zero: In expression 27. / (double)numberOfIndices, division by expression numberOfIndices which may be zero has undefined behavior. Change-Id: If4f4a6795c1dfcae904ef7279ab25a1f855a6387 Reviewed-by: Karsten Heimrich <karsten.heimrich@qt.io>
* Remove the QHelpDataInterface base classJarek Kobus2018-07-261-9/+9
| | | | | Change-Id: I249d871bc437ad42b8b6ecbf95b19ba5520fdf74 Reviewed-by: Kai Koehne <kai.koehne@qt.io>
* Unexport HelpProjectData and move it to helpgenerator app treeJarek Kobus2018-07-261-0/+879
Change-Id: I5af687d5f7dd3fc886a33453b5613df062a8f5a2 Reviewed-by: Kai Koehne <kai.koehne@qt.io>