| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This commit changes the following in relation to string use
- The implicit convert from 'QCString' to 'const char *' is removed
- Strings parameters use 'const QCString &' as much as possible in favor
over 'const char *'
- 'if (s)' where s is a QCString has been replaced by 'if(!s.isEmpty())'
- data() now always returns a valid C-string and not a 0-pointer.
- when passing a string 's' to printf and related functions 'qPrint(s)' is
used instead of 's.data()'
- for empty string arguments 'QCString()' is used instead of '0'
- The copy() operation has been removed
- Where possible 'qstrcmp(a,b)==0' has been replaces by 'a==b' and
'qstrcmp(a,b)<0' has been replaced by 'a<b'
- Parameters of string type that were default initialized with '= 0' are
no initialized with '= QCString()'
|
| | |
|
| | |
|
| |\
| |
| |
| | |
https://github.com/albert-github/doxygen into albert-github-feature/bug_md_pg_cnt
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
All examples together are a bit big so they are all in the attached file together with the warnings before and after this patch.
When looking at the output we see that a number of lines are off by 1 ore more due to the insertion of extra lines in the code or due to the fact that there is a reference to the start of the documentation of the page and not the line of the page command.
- commentscan.l
in case of a rule contaiinge {DOCNL} this can be `=n` or `\ilinebr` in the later case this should not be replaced by `\n` as this results in an increase of the line count later on.
in case of a page like command also register the line of the command as "topline".
- entry.h, entry.cpp
storage space for the "topline" registering the line of the page like commands.
- doxygen.cpp
setting and using the "topline"
- markdown.cpp, markdown.h
don't add a `\n` as this increases the line count but use the `\ilinebr`
to get correct warnings see to it that when having empty lines at the top of the page and a page is added that the empty lines appear after the page command.
- index.cpp
using the "topline" instead of the "docLine" to get the correct warning
- pagedef.cpp, pagedef.h
set and retrieve the "topline" for page like commands.
- util.cpp, util.h
setting and using the "topline"
in `addRelatedPage` use the known file name and line for the warning regarding the section label
- cite.cpp, context.cpp, reflist.cpp
changed to have good function calls.
|
| |/
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When having an example like:
```
/// \file
/** \xrefitem my_errors "err2" "ERR1" ERROR 101*/
#define MY_ERR_CANNOT_OPEN_FILE 101
/** \xrefitem my_errors "err2" "ERR2" ERROR 102*/
#define MY_ERR_CANNOT_CLOSE_FILE 102
/** \xrefitem my_errors "err2" "ERR1a" ERROR 101a*/
#define MY_ERR_CANNOT_OPEN_FILE3 103
/** \xrefitem my_errors "err2" "ERR2a" ERROR 102a*/
#define MY_ERR_CANNOT_CLOSE_FILE4 104
```
the sorting should be done on the part `MY_ERR_CANNOT_...` but no sorting is done (since 1.8.18) as:
- call to `qstricmp` contains twice the same value
- a boolean value should be the result of the compare but in fact an integer value was returned
This problem is most likely a regression due to
```
Commit: aca13723a9373a1080ca7f108e7be0905b9ae793 [aca1372]
Date: Thursday, February 27, 2020 10:38:22 PM
Restructure the way RefLists are handled
```
|
| |\ |
|
| | | |
|
| |/
|
|
|
| |
In case a xref item has no reference the name is empty and thus the item should not be displayed.
As it is possible that none of the items have a reference the page should only be added when there are item available to display
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| |
|
|
|
|
|
|
|
|
| |
The example code with the `xrefitem` in the documentation gives a warning:
```
my_errors:1: warning: multiple use of page label 'my_errors', (other occurrence: .../aa.h, line: 4)
```
it is possible (and explicitly stated) to have an `\page` with the same name giving some extra information.
The warning regarding the multiple page is removed in case of a xrefitem page.
|
| | |
|
| |
|
|
|
| |
In Objective-C the separator for classes / protocols / functions is a single colon `:` resulting in recognizing it as an non existing emoji.
Note that this can only happen with generated names, in e.g. a page name an emoji can be used.
|
| |\
| |
| | |
issue_6589 anchor after test command in a namespace produces duplicate tests
|
| | |
| |
| |
| | |
Not only should be checked whether the item to be inserted is the main item but also if it is not one of the extra items
|
| |/
|
|
|
| |
In case there are multiple `\xreflist` commands with the same key only the first item has a correct linking possibility, the subsequent items don't have it.
Resulting in warnings in e.g. docbook and LaTeX output
|
| |
|
|
|
|
| |
correctly
Besides backslashes in the title (issue #5901) the backslashes in the argument list need to be escaped.
|
| | |
|
| |\
| |
| |
| | |
https://github.com/albert-github/doxygen into albert-github-feature/bug_759063
|
| | |
| |
| |
| |
| |
| |
| | |
illegal command warning
Illegal command is in case cause by the fact that the namespace-name and class-name are "concatenated" by means of a '\' in the tite of the, internal, xreflist.
This patch fixes this by doubling the backslash.
|
| | | |
|
| |/
|
|
|
| |
Problems are cause by the annotation in the arguments (i.e. @myAnnotation) and the fact that the function prototype is used as title for the todo list.
Escaping the @ (by means of @@) solves this problem.
|
| |
|
|
|
|
|
| |
functions
The globalScope as shown in the output has no meaning and is confusing (and is also lacking the starting <).
This problem is a regression from: Bug 740218 - Full scope needed when making link inside cross-referenced section [with test case]
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
