Skip to content

Commit

Permalink
Warning about internal doxygen inconsistency for python packages
Browse files Browse the repository at this point in the history
In the Ansible package (through Fossies) we get warnings like:
```
lib/ansible/module_utils/facts/virtual/base.py:27: warning: Internal inconsistency: scope for class ansible::module_utils::facts::virtual ::base::Virtual not found!
```
Due to the naming of python the classes in doxygen it is possible that those names will contain reserved names as the path is used in the identification of the class (here `virtual`).
When a `:` is in front of the word it is a class name and no space should be appended.
  • Loading branch information
albert-github committed Sep 6, 2023
1 parent d062df6 commit d60107a
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions src/util.cpp
Expand Up @@ -589,17 +589,17 @@ QCString removeRedundantWhiteSpace(const QCString &s)
if (csp<6 && c==constScope[csp] && // character matches substring "const"
(csp>0 || // inside search string
i==0 || // if it is the first character
!isId(pc) // the previous may not be a digit
!(isId(pc) || pc==':') // the previous may not be a digit or a colon
)
)
csp++;
else // reset counter
csp=0;

if (vosp<6 && c==volatileScope[vosp] && // character matches substring "volatile"
(vosp>0 || // inside search string
i==0 || // if it is the first character
!isId(pc) // the previous may not be a digit
(vosp>0 || // inside search string
i==0 || // if it is the first character
!(isId(pc) || pc==':') // the previous may not be a digit or a colon
)
)
vosp++;
Expand All @@ -610,7 +610,7 @@ QCString removeRedundantWhiteSpace(const QCString &s)
if (vsp<8 && c==virtualScope[vsp] && // character matches substring "virtual"
(vsp>0 || // inside search string
i==0 || // if it is the first character
!isId(pc) // the previous may not be a digit
!(isId(pc) || pc==':') // the previous may not be a digit or a colon
)
)
vsp++;
Expand All @@ -621,7 +621,7 @@ QCString removeRedundantWhiteSpace(const QCString &s)
if (osp<11 && (osp>=8 || c==operatorScope[osp]) && // character matches substring "operator" followed by 3 arbitrary characters
(osp>0 || // inside search string
i==0 || // if it is the first character
!isId(pc) // the previous may not be a digit
!(isId(pc) || pc==':') // the previous may not be a digit or a colon
)
)
osp++;
Expand Down

0 comments on commit d60107a

Please sign in to comment.