| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
| |
- In the past in case of a quoted string there was a space at the end of the string (as also indicated in the comment), now this space is gone so the `wasQuoted` would not kick in. A string with an escaped double quote should be / has to be escaped so it should also be included in the `wasQuoted`
- missing one `data()` -> `str()` (should not give a problem due to the guard, but better for consistency).
|
| |
|
|
|
|
| |
The rule also caught a `\n` at the second position.
|
|
|
|
|
| |
- also escape a "," (comma) when writing out string values (doxygen and doxywizard)
- check on empty lists in stringlist for the wizard
|
| |
|
|
|
|
| |
configuration file (part 2)
|
|
|
|
| |
configuration file
|
|
|
|
| |
Should also ignore `\r` as white space
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When having a doxygen configuration file like:
```
QUIET=YES
@INPUT = file
@UNKNOWN =
@UNKNOWN1
@UNKNOWN 1
@UNK # test
```
we get the warnings
```
warning: ignoring unsupported tag 'UNKNOWN' at line 3, file Doxyfile
warning: ignoring unknown tag 'UNKNOWN1' at line 4, file Doxyfile
warning: ignoring unknown tag 'UNKNOWN' at line 5, file Doxyfile
warning: ignoring unknown tag '1' at line 5, file Doxyfile
warning: ignoring unknown tag 'UNK' at line 6, file Doxyfile
# Difference with default Doxyfile 1.9.0 (fa65bb38f81457d00f9c900bb57eb68bea59b1b4)
QUIET = YES
INPUT = file
```
especially the missing of a warning about the `@` in `@INPUT` can be a bit misleading (it might be that the user wanted to use `@INCLUDE` and and specified `@INPUT`
It would be better to have a warning about a not handled character instead of just ignoring it.
|
|\
| |
| | |
Incorrect handling of string with spaces and no quotes
|
| |
| |
| |
| | |
Correcting some (Cygwin) compilation warnings / errors
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
During some tests on the documentation of LLVM the (CMake generated) doxygen configuration file contained the line:
```
DOT_PATH=D:\Program Files (x86)\Graphviz2.38\bin\dot.exe
```
and this was interpreted by doxygen as
```
DOT_PATH = D:\ProgramFiles(x86)\Graphviz2.38\bin\dot.exe
```
without any message other than that later on the `dot` executable could not be found.
It is clear that here the double quotes were missing.
This problem has been solved by giving a warning and setting the value to the default value.
- configimpl.l
- config_doxyw.l
during the checking some debug facilities for the doxywizard were required and this has been implemented:
- config_doxyw.l
- doxywizard.h
- doxywizard.cpp
it was also discovered that in case of none existing int or bool values were specified the wrong "defaults" were taken (it was set to `0` / `false`), now the correct defaults are taken
- inputbool.cpp
- inputint.cpp
|
|/
|
|
|
| |
In principle a configuration file should not know anything about the doxygen commands, but the handling of the escaped `\` and `@` are an exception of this rule (especially in `ALIASES`).
The mentioned escaped characters were not handled properly.
|
|
|
|
| |
ignored
|
|\
| |
| |
| | |
https://github.com/albert-github/doxygen into albert-github-feature/bug_chm_tests
|
| |
| |
| |
| |
| |
| |
| | |
By means of `nmake tests TEST_FLAGS="--xhtml --keep --cfg GENERATE_HTMLHELP=YES --cfg HHC_LOCATION=C:/PROGRA~2/HTMLHE~1/hhc.exe --cfg SEARCHENGINE=NO" it is possible to generate for each doxygen test an index.chm, but there are a number of problems:
- it is only possible to use extension `.html` so this is now automatically done (configimpl.l) otherwise when opening the file an error is displayed.
- it is not [possible to use "empty" hhc / hhk file (i.e. no `<li>` elements), in this case the files are still generated but not used anymore (error like `error: failed to run html help compiler on index.hhp` e.g. for test 1).
- a full qualified / absolute is not allowed for e.g msc images (test 37) the files should anyway be on the "html" directory so just using the filename in the "hhp" file (` t << QFileInfo(imageFiles.at(i)).fileName().data() << endl;`)
|
| | |
|
| |
| |
| |
| | |
Ignore empty strings
|
| | |
|
|/
|
|
|
|
|
|
|
|
|
|
|
|
| |
In case we have an empty setting in the doxygen configuration file, where an enum value is expected like:
```
OUTPUT_LANGUAGE =
```
we get the warning:
```
warning: argument '(null)' for option OUTPUT_LANGUAGE is not a valid enum value
Using the default: English!
```
The default value should immediately have been used.
|
|\ |
|
| |
| |
| |
| | |
very popular)
|
| | |
|
|/
|
|
|
| |
- add doxygen version to rtf, comment, output
- remove duplicate code (getFullVersion)
- more clear name to get doxygen version (getVersion becomes getrDoxygenVersion). Also to overcomecofusion with the version information for files.
|
| |
|
|
|
|
| |
In pull request #7603 the problem regarding `PREDEFINED = A(x,y)`, but the same patch must be applied to the construct `PREDEFINED += A(x,y)`
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
In the pull request "Enable comma as separator in configuration lists enhancement " (#6563) it was made possible to have commas as separators for lists.
In case we have:
```
PREDEFINED = A(x,y)=sin(x),cos(y)
```
and use `doxygen -x` (or usethe define): this results in:
```
PREDEFINED = A(x \
y)=sin(x) \
cos(y)
```
this can be overcome by means of:
```
PREDEFINED = "A(x,y)=sin(x),cos(y)"
```
But for a lot of existing packages this poses a problem.
(Found by looking at the doxygen configuration files as used by Fossies).
|
|
|
|
|
|
|
|
|
|
|
|
| |
In case we set for instance `DOT_IMAGE_FORMAT = SVG` (and generate call graphs) we get messages from the `dot` tool like:
```
error: Problems running dot: exit code=1, command='dot', arguments='".../example/html/test_8cpp_a764ac60c654173eb1a0afd0906ad5a12_icgraph.dot" -TSVG -o ".../example/html/test_8cpp_a764ac60c654173eb1a0afd0906ad5a12_icgraph.SVG"'
```
on other places (e.g `HTML_FORMULA_FORMAT`) and an unknown or enum value with a wrong case, the default value is (silently) taken.
We now check:
- is the enum value of the correct case otherwise (silently) set it to the correct case.
- in case of an unknown enum value a warning is given and the default value is used
|
| |
|
|
|
|
| |
In #7419 the EXTENSION_MAPPING test was omitted
|
| |
|
|\
| |
| |
| | |
https://github.com/albert-github/doxygen into albert-github-feature/bug_terminate
|
| |
| |
| |
| |
| | |
In case a `WARN_LOGFILE` is used it in this file no clear whether doxygen finished correctly or exited beforehand in case a fatal error.
Now a 'Exiting...' is also given in the warning log file in case an error is a fatal error.
|
| | |
|
|\ \
| | |
| | |
| | | |
https://github.com/albert-github/doxygen into albert-github-feature/bug_config
|
| |/
| |
| |
| | |
In case of a configuration list item has an equal sign in it ('=') some extra restriction are imposed / made more understandable.
|
| | |
|
|/
|
|
|
|
| |
namespace
Fixing some memory issues caused by not correctly freed pointers
|
|
|
|
|
|
|
|
| |
In case we have an alias in the form: `mine_err(1)=\1` we get the warning:
```
error: Illegal alias format 'mine_err(1)=\1'. Use "name=value" or "name(n)=value", where n is the number of arguments
```
Note the round brackets in the explanation., these have to be curly brackets.
|
| |
|
|\
| |
| |
| | |
https://github.com/albert-github/doxygen into albert-github-feature/bug_consistency_state_2
|
| |
| |
| |
| |
| | |
Create a consistent way to display the state mnemonics of the different scanners (analogous to the fortranscanner.l)
Use an automatic procedure to generate the routine with the translation of the states to a string.
|
|/ |
|
|
|
|
| |
This is actually a regression on #7195. Due to the fact that the init() routine has moved up, the settings for `FILE_PATTERNS` have already been done and a call to init does not operate on the `FILE_PATTERNS` variable but on the structure from which the `FILE_PATTERNS` was derived. We have should here operate on the `FILE_PATTERNS` directly as well., bu getting the default values out of the underlying structure.
|
|\
| |
| | |
Consistency of configuration parameters
|
| |
| |
| |
| |
| |
| |
| |
| | |
Configuration parameters were used 2 forms in the code bare name like `PAPER_TYPE` and as string `"PAPER_TYPE"`.
In case of a non existing "bare name" (or misspelling) this is signaled by the compiler but in case of the string this was only shown at runtime.
In case of the usage of an obsolete configuration parameter the "bare name" was signaled as not existing (anymore).
The essential change here is the place where the ConfigValues are initialized.
|
|\ \
| | |
| | |
| | | |
https://github.com/albert-github/doxygen into albert-github-feature/bug_endblock_msg
|
| | |
| | |
| | |
| | | |
Consistency
|
|\ \ \
| |_|/
|/| |
| | | |
https://github.com/albert-github/doxygen into albert-github-feature/bug_gitversion
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The original version has as features:
- getting the git version number for usage in doxygen
- making the doxygen version number inclusion dependent on the VERSION file
The disadvantage of the chosen methodology was that an extra correction step was necessary, by defining getter methods to retrieve the values this correction can be hidden.
The information is coming from different sources:
- the VERSION file
- the git "repository and build system (when present)
Furthermore there are a couple of places where the version information is used (a.o. doxygen and doxywizard executable, though the doxygenwizard was only done "half hearted")
The handling of the VERSION file has been made in such a way that it is comparable with the generation of the git version changes.
For a better abstraction the version handling is all done in a separate directory.
|