After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 640588 - QHP_SECT_FILTER_ATTRS is not split correctly
QHP_SECT_FILTER_ATTRS is not split correctly
Status: RESOLVED FIXED
Product: doxygen
Classification: Other
Component: general
1.7.3
Other Linux
: Normal normal
: ---
Assigned To: Dimitri van Heesch
Dimitri van Heesch
Depends on:
Blocks:
 
 
Reported: 2011-01-25 21:53 UTC by Haffmans
Modified: 2011-03-28 14:19 UTC
See Also:
GNOME target: ---
GNOME version: ---


Description Haffmans 2011-01-25 21:53:43 UTC 
The configuration string QHP_SECT_FILTER_ATTRS is not split on spaces correctly, which results in filter attributes of the QHP file like "attr1 attr2", instead of two attributes "attr1" and "attr2".

This seems to be a compiler or Qt bug, as the issue is resolved by changing line 88 from:

QStringList customFilterAttributes = QStringList::split(' ', Config_getString("QHP_CUST_FILTER_ATTRS"));

to:

QStringList customFilterAttributes = QStringList::split(QChar(' '), Config_getString("QHP_CUST_FILTER_ATTRS"));

And making a similar change on line 99.

I assume this means the wrong version of QStringList::split() is picked. If I replace the QChar with QRegExp or QCString it breaks, so it looks like this hint is necessary - at least it is in my case. Perhaps there are other locations in the source code where the same change is required?

This is on Arch x64, for this test I downloaded and used the source from the Doxygen website. The bug doesn't seem to occur in the (1.6.3) OpenSuse packaged builds.
Comment 1 Haffmans 2011-01-25 21:54:44 UTC 
Forgot to mention the source file name this occurs in: src/qhp.cpp (lines 88 and 99)
Comment 2 Dimitri van Heesch 2011-01-25 22:38:24 UTC 
Thanks. I'll include the changes in the next subversion update.
Comment 3 Dimitri van Heesch 2011-03-28 14:19:30 UTC 
This bug was previously marked ASSIGNED, which means it should be fixed in
doxygen version 1.7.4. Please verify if this is indeed the case. Reopen the
bug if you think it is not fixed and please include any additional information
that you think can be relevant.