| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
| |
| |
| |
| |
| |
| |
| |
| | |
Allow tests to remove full paths from strings to circumvent hard-coded
paths in output.
Change-Id: I4270f201649e9e5145b1ce75d2d372605a95579d
Reviewed-by: Topi Reiniö <topi.reinio@qt.io>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The platform no longer has a 'd' suffix for debug binaries
(see qtbase/4d289edb14aeb4976218cc306bfc514e37521b82).
Add a check.
Fixes: QTBUG-80763
Task-number: QTBUG-78445
Change-Id: I8d4617e2111167a546fa172cbca78a656d233ee5
Reviewed-by: Simon Hausmann <simon.hausmann@qt.io>
Reviewed-by: Oliver Wolff <oliver.wolff@qt.io>
|
| |
| |
| |
| |
| |
| |
| | |
Simple cleanup to improve readability.
Change-Id: If9ee2947ad76e64d230584a3cb36e51457f643c3
Reviewed-by: Topi Reiniö <topi.reinio@qt.io>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Replace the use of QVector in most of QDoc.
Also, remove one redundant C-style cast to int for result from method
call that returns an int. As this happened in a macro, the result is
removing a whole bunch of nagging from code inspection.
Fixes: QTBUG-80669
Change-Id: Ib1aed95e01eaddd1e1213a145e815a0c4753ac67
Reviewed-by: Mårten Nordheim <marten.nordheim@qt.io>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This patch introduces an extra parameter to testAndCompare(),
that allows comparing expected vs actual in the provided output directory.
This is necessary for further refactoring of the test,
so each test point will use its own directory for its expected output.
This will help to avoid name collisions.
Change-Id: I6c539ac7e2ec9dae06d8f03ef30f1c218de50ac9
Reviewed-by: Levon Sargsyan <levon.sargsyan@qt.io>
Reviewed-by: Paul Wicking <paul.wicking@qt.io>
|
| |
| |
| |
| |
| |
| |
| | |
Pre-increment instead of post-increment for efficiency and readability.
Change-Id: I10b33360dd36cd1d0f4338441c1ad4724551e80e
Reviewed-by: Topi Reiniö <topi.reinio@qt.io>
|
| |
| |
| |
| |
| |
| |
| | |
Drop casting ints to int.
Change-Id: I571f3e245bfc97a7c0c7bea5bedbf0ed5ab04c60
Reviewed-by: Topi Reiniö <topi.reinio@qt.io>
|
| |
| |
| |
| |
| |
| |
| | |
Fix minor whitespace issues accodring to style.
Change-Id: Ie56e815f23f25ccb1b2ffb03780b9e4c6b9d25eb
Reviewed-by: Topi Reiniö <topi.reinio@qt.io>
|
| |
| |
| |
| |
| |
| |
| | |
Delete commented out code.
Change-Id: Iaa08fed365d5a4d6a2986908761ce42342089730
Reviewed-by: Topi Reiniö <topi.reinio@qt.io>
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
There's no change in output by not sorting the container, so it seems to
be entirely unnecessary.
Also const qualify the container that is no longer sorted.
Change-Id: I8d5186a70623b8db6397eb88b9b0a9b53465a34a
Reviewed-by: Topi Reiniö <topi.reinio@qt.io>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
- Use ranged-based fors where applicable.
- Use auto keyword for iterators.
- Move a few variable declarations to where they're to be used.
- Update docs where applicable.
Fixes: QTBUG-80536
Change-Id: I859440b96428dec4ef108b01d391479d3f8dbd83
Reviewed-by: Topi Reiniö <topi.reinio@qt.io>
|
| |
| |
| |
| |
| | |
Change-Id: Ifbd739510abd5bec4cc2b4fbac9b6667d9ae9281
Reviewed-by: Topi Reiniö <topi.reinio@qt.io>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The filename attribute section in the generated index/webxml
has been taking redudant directory name under Windows platform.
This patch fixes the issue by removing filename deduction code
and replaces it with fileName() from QFileInfo class accordingly.
Change-Id: If0ef2e88c94e837099fd5e669d2b7e96e7cd4280
Fixes: QTBUG-80641
Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org>
Reviewed-by: Topi Reiniö <topi.reinio@qt.io>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This patch removes structuraly and logically dead code,
reported via coverity scan and reported to appropriate
categories of "dead code".
Coverity-Id: 11018
Coverity-Id: 226101
Coverity-Id: 226127
Change-Id: I1930299c57ff10875c5b7e2df51895d9dee42fac
Reviewed-by: Paul Wicking <paul.wicking@qt.io>
Reviewed-by: Levon Sargsyan <levon.sargsyan@qt.io>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Fix the previous test that never generated any QML type documentation,
and add new test types, documented in .cpp, .qdoc, and inline in .qml.
Use all available \qml-specific QDoc commands.
Fixes: QTBUG-80154
Change-Id: I60d9762836b986ca534c664517274517b73c085d
Reviewed-by: Levon Sargsyan <levon.sargsyan@qt.io>
Reviewed-by: Paul Wicking <paul.wicking@qt.io>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This patch fixes potential null pointer derefencing issues all
over qdoc project that has been found via coverity scan and has been
assigned to "Dereference after null check" category.
Fix include coverity issues with REVERSE_NULL as well as FOWARD_NULL
cases.
Change-Id: Icb6f413ea7bbfae76f8e68139baa31b15575a49a
Coverity-Id: 163245
Coverity-Id: 11133
Coverity-Id: 185270
Coverity-Id: 226086
Reviewed-by: Topi Reiniö <topi.reinio@qt.io>
Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Change substitutes Node header dependency on Generator with forward-declaration.
This doesn't break any functionality since the header of Generator
references only Node pointers.
Aim of such change is aligned with the strategy of breaking down any
unnecessary dependencies in the code to make qdoc unit-testable.
Change-Id: Ia2ef2f2fb62a1435bffeda1f1b389a3079ac12b0
Reviewed-by: Paul Wicking <paul.wicking@qt.io>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This patch adds a regression test to cover QTBUG-80259
by adding new test data to the existing end-2-end qdoc test suite.
The test is covering class name sorting functionality
during documentation generation.
Change-Id: I6421981596f4d72e8b0ce8351e85a8e5b3856b45
Fixes: QTBUG-80456
Reviewed-by: Levon Sargsyan <levon.sargsyan@qt.io>
Reviewed-by: Paul Wicking <paul.wicking@qt.io>
|
| |
| |
| |
| |
| |
| |
| |
| | |
Improve readability by pre-incrementing instead of post-incrementing.
This should also improve performance slightly.
Change-Id: Icec66867dfdb04a65e3fe3229d21e2cda98eebb9
Reviewed-by: Volker Hilsheimer <volker.hilsheimer@qt.io>
|
| |
| |
| |
| |
| |
| | |
Change-Id: I8e58dcecdc392c6fc26840f18d4b0e216941de27
Reviewed-by: André Hartmann <aha_1980@gmx.de>
Reviewed-by: Edward Welbourne <edward.welbourne@qt.io>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The comparison function used for sorting compares fullTitle() strings
for PageNode items. When the class hierarchy was refactored, this
had the unintended effect where this comparison was applied to lists
of C++ and QML types as well - the full title is empty for these,
so we ended up comparing the physical location of the type declarations
instead.
Fix this by preferring the fullName() over fullTitle() when comparing,
as the former is what the visible list item names are generated from.
Make the comparison function easier to read and maintain by introducing
a preprocessor macro.
Fixes: QTBUG-80259
Change-Id: I3f35aa80f1a90a6817f836146976d73b857df820
Reviewed-by: Martin Smith <martin.smith@qt.io>
Reviewed-by: Paul Wicking <paul.wicking@qt.io>
|
| |
| |
| |
| |
| |
| |
| |
| | |
HTML, WebXML, and the future DocBook output
Fixes: QTBUG-80100
Change-Id: Idf373e3b1d76ab808597dcabd8649ecb5e2df0f7
Reviewed-by: Topi Reiniö <topi.reinio@qt.io>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
We already know the full URL during construction of the node,
so use it.
This prevents a lot of (error-prone) processing when generating
links to \externalpage nodes.
Change-Id: I0209ae491b40908d063f3beb10b29aaffa00d237
Reviewed-by: Paul Wicking <paul.wicking@qt.io>
|
| |
| |
| |
| |
| |
| |
| |
| | |
Also, remove an argument they were muting to instead
return a new value.
Change-Id: Ibe6c73d36988ed76884c5d5bbdd656bfb5221b3e
Reviewed-by: Paul Wicking <paul.wicking@qt.io>
|
| |
| |
| |
| |
| | |
Change-Id: I41841706c17fee870ca91e679d63a7a00d876519
Reviewed-by: Paul Wicking <paul.wicking@qt.io>
|
|/
|
|
|
|
|
|
| |
Detect msvcrt.dll without version number
Change-Id: Ifafdc79c1b6497e1f6695b9c69d9d9fdd6cc2d94
Reviewed-by: Yuhang Zhao <2546789017@qq.com>
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@qt.io>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Commit 2b6c90317 allowed QDoc to load all available index files if
'depends = *' configuration was used.
This could still fail when output from multiple modules is directed
to the same location (single-dir output mode). QDoc correctly adds
the output directory as a place to look for index files, but because
that output directory is also added as a dependency, it was removed
from the list of sub-dirs to look into once that dependency was
processed. This meant that all the subsequent dependencies could
not be located.
Also, there's a fundamental difference in how we must resolve
the dependencies; without .nosubdirs, QDoc takes the subdirectories
of -indexdir entries and treats them as dependencies. With
.nosubdirs, we need to look for actual .index files in the
-outputdir and load all of them.
Finally, move the logging line for 'depends = *' further down,
so that we can print out the number of resolved dependencies.
Task-number: QTBUG-80051
Change-Id: I35ff5c22e8e1f7e2b0c7de798531ad1c72813e51
Reviewed-by: Martin Smith <martin.smith@qt.io>
|
|
|
|
|
| |
Change-Id: I026206b27518da416773eb1a240e2a5b0ee4449c
Reviewed-by: Paul Wicking <paul.wicking@qt.io>
|
|
|
|
|
| |
Change-Id: I1b5d0d96c8955852369ecc8717b020335f05c010
Reviewed-by: Paul Wicking <paul.wicking@qt.io>
|
|
|
|
|
| |
Change-Id: I044702411c8aba3286c98d42962a4d19720e33fb
Reviewed-by: Paul Wicking <paul.wicking@qt.io>
|
|
|
|
|
| |
Change-Id: Ida9040a17e05b193974d39d72a9998c0a63ed892
Reviewed-by: Paul Wicking <paul.wicking@qt.io>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
QDoc recognized '*' as a valid input for the 'depends' qdocconf
variable, with the intent to load all available index files. This did
not work as we cleared the resolved dependencies by re-fetching
the depends string from Config after resolving the dependencies.
This fix is a backport of another commit that is targeted for the
dev branch, but this useful already for 5.14.
Fixes: QTBUG-80051
Change-Id: I1d752208078452f4b16b1d855cdb67b8cbabf1f4
Reviewed-by: Martin Smith <martin.smith@qt.io>
Reviewed-by: Paul Wicking <paul.wicking@qt.io>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
QDoc's parsing of property types is slightly naive, in that it
cannot parse a type that consists of multiple words. These are
not common, but we do have a few instances of
Q_PROPERTY(enum EnumName ...)
in the Qt source. This commit allows QDoc to parse above properties
by ignoring the string 'enum'.
Fixes: QTBUG-80027
Change-Id: I3b01d463b081042dd27299c2121fafa3c99b98ce
Reviewed-by: Paul Wicking <paul.wicking@qt.io>
|
|
|
|
|
| |
Change-Id: I81de861e9daa00249c47ddbe259f43f47d7c6615
Reviewed-by: Paul Wicking <paul.wicking@qt.io>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- QDoc Manual: UIComponents example project has been moved to be a part
of test case - remove references to it from the manual.
- Add missing image.
- Document the namespace qdesigner_internal as \internal.
Fixes: QTBUG-79833
Change-Id: Ic0984cef4091e794de5e502d04a897b1f532d9d8
Reviewed-by: Paul Wicking <paul.wicking@qt.io>
|
|
|
|
|
|
|
|
|
|
|
| |
The inline and auto links were handled differently compared to
the members list for a group page. This change reuses the same
code for generating any kind of link.
This change amends 8f5ebc02f6653e
Change-Id: Ic9d2bf00a95c718d6074ec2b0b4705324cccf053
Reviewed-by: Paul Wicking <paul.wicking@qt.io>
|
|
|
|
|
|
|
|
| |
Each <row> should have <item>s representing the columns
that contain <para>.
Change-Id: I264f490a3809e9f0776dc3aa2a3c25b783c0fd6b
Reviewed-by: Paul Wicking <paul.wicking@qt.io>
|
|\ |
|
| |\
| | |
| | |
| | | |
Change-Id: Id9060f7bd9103d8f9e29f71d397f3615ead183dc
|
| | |\
| | | |
| | | |
| | | | |
Change-Id: Ic8cb8f2dd90f8fb809034ef07b341f3f7cb71ab2
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Change-Id: Ibe11b7fa1130754ab0a6b1c2379d7e80909dc757
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@qt.io>
Reviewed-by: Eskil Abrahamsen Blomfeldt <eskil.abrahamsen-blomfeldt@qt.io>
|
| | | |
| | | |
| | | |
| | | | |
Change-Id: I58ce340a94bf70b44704e7af2b48952858c22f21
|
| | | |\
| | | | |
| | | | |
| | | | | |
Change-Id: I5355c7537c1f4dc5bbf80e52e3c0c28282fd4446
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
This update ensures that if a class node returns true for isDontDocument(),
it is not included in the map used for generating the all classes list. This
also applies to QML types.
Task-number: QTBUG-78940
Change-Id: I020cb0dd2f16187d5f6c75b400778b1518d7e05a
Reviewed-by: Paul Wicking <paul.wicking@qt.io>
Reviewed-by: Martin Smith <martin.smith@qt.io>
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
QTypeInfo is a class that is generated by a Qt macro. It is not
documented and is not supposed to be documented, but clang sees
it as being in the public API. This update ensures that the
class is marked internal prior to generating the index file.
Ironically, this was one of the classes that motivated the
addition of the \dontdocument command, and it was the test for
isDontDocument() that caused this bug. We might have to visit
the \dontdocument command again.
Task-number: QTBUG-79088
Change-Id: I11307a2236e2ebcdcc205952056ddaca88fc0bd3
Reviewed-by: Paul Wicking <paul.wicking@qt.io>
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Change-Id: I6b9a66f15308ecbbdbeaf2dc66de3875b16dd03f
Reviewed-by: Paul Wicking <paul.wicking@qt.io>
Reviewed-by: Edward Welbourne <edward.welbourne@qt.io>
|
|/ / / /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Check whether the content is already is patched to avoid
unnecessarily touching files.
Fixes: QTBUG-78732
Change-Id: I061927e8908793b78211b4634c3dc772c12c7915
Reviewed-by: André de la Rocha <andre.rocha@qt.io>
Reviewed-by: Kai Koehne <kai.koehne@qt.io>
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
State that the resulting files are not intended to be used
without editing.
Change-Id: I5ae342907a9394427ef9d3471ff2670421da8cc0
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@qt.io>
Reviewed-by: Kavindra Palaraja <kpalaraja@luxoft.com>
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Hint which color will be picked when pressing 'c'.
Change-Id: If48392b288732b30d88bff18b3a3bbf5074947ed
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@qt.io>
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Add an option to copy the color via QColor::name(),
bound to the 'c' key.
Change-Id: Ib33538e5ea5523e5ec5881d75e0b17ab675078e3
Reviewed-by: Alessandro Portale <alessandro.portale@qt.io>
|