Opened 12 years ago
Closed 11 years ago
#459 closed enhancement (fixed)
Various improvements in the doc
Reported by: | Peter Kovacs | Owned by: | Peter Kovacs |
---|---|---|---|
Priority: | major | Milestone: | LEMON 1.3 release |
Component: | documentation | Version: | hg main |
Keywords: | Cc: | ||
Revision id: |
Description
The attached patches contain various doc fixes and improvements.
Attachments (5)
Change History (15)
Changed 12 years ago by
Attachment: | 459-1-7bf489cf624e.patch added |
---|
Changed 12 years ago by
Attachment: | 459-2-d9d1cb759951.patch added |
---|
Changed 12 years ago by
Attachment: | 459-3-4f9a45a6d6f0.patch added |
---|
Changed 12 years ago by
Attachment: | 459-4-eb2f9d453070.patch added |
---|
comment:1 Changed 12 years ago by
comment:2 Changed 12 years ago by
Status: | new → assigned |
---|
Changed 12 years ago by
Attachment: | lemon-refs.png added |
---|
comment:3 follow-up: 4 Changed 12 years ago by
I pushed all the patches above. However, I'm not satisfied with how the references page is rendered:
The citation code of the references are huge. IMHO, they should be of the same size as the text itself.
Btw, shouldn't we consider #456?
comment:4 Changed 12 years ago by
Replying to alpar:
I pushed all the patches above. However, I'm not satisfied with how the references page is rendered:
The citation code of the references are huge. IMHO, they should be of the same size as the text itself.
I agree. This was much better with Doxygen 1.7.x, but 1.8.x uses a much larger font size for \section
. I tried to use \subsection
instead of \section
, but that did not work.
Btw, shouldn't we consider #456?
Yes, we should consider #456.
comment:5 follow-up: 6 Changed 12 years ago by
I keep this ticket open, because I noticed that there are tons of warnings in doxygen.log
. Many of them are probably valid issues.
comment:6 Changed 11 years ago by
Resolution: | → done |
---|---|
Status: | assigned → closed |
Replying to alpar:
I keep this ticket open, because I noticed that there are tons of warnings in
doxygen.log
. Many of them are probably valid issues.
I pushed two changeset to the main branch:
- [97d978243703] fixes the majority of the unresolved doc references. What remains is the references to those tools are not yet ported from 0.x. These should be removed in the release branch
- [218171dc022d] makes the location of the gcc reference manual configurable. It also changes the default one to 4.7.3 from the ever changing latest development version.
I think these resolve all the remaining issues.
comment:7 follow-up: 9 Changed 11 years ago by
Resolution: | done |
---|---|
Status: | closed → reopened |
I reopened this ticket as I got a compiler warning that is due to [97d978243703] (using GCC 4.5.x on lime.cs.elte.hu).
The warning is:
lemon/lp_base.h:1010:5: warning: multi-line comment
for this part of the file:
1010 ///\e \ 1011 class UnsupportedFormatError : public Exception
comment:8 Changed 11 years ago by
Moreover, it also causes compiler errors like:
lemon/lp_base.h:1012:5: error: expected unqualified-id before ‘{’ token lemon/lp_base.cc:30:1: error: expected ‘}’ at end of input
comment:9 Changed 11 years ago by
Replying to kpeter:
1010 ///\e \ 1011 class UnsupportedFormatError : public Exception
Of course that \
at the end of line 1010 causes the problem. However, if I omit it, then doxygen sends a warning saying
/home/alpar/projects/LEMON/hg/459/lemon/lp_base.h:1009: warning: expected whitespace after e command
This is very strange, for we use ///\e
to force documenting a class without adding a description at many places in the source with no problem.
comment:10 Changed 11 years ago by
Resolution: | → fixed |
---|---|
Status: | reopened → closed |
The issue above is fixed in [38c432e01489]. Still can't understand why ///\e
doesn't work.
Attached changesets: