Black list of erroneous/ misleading entries in doxygen docs

Anything and everything that's related to OGRE or the wider graphics field that doesn't fit into the other forums.
Post Reply
User avatar
paul424
Gnome
Posts: 314
Joined: Thu May 24, 2012 7:16 pm
x 13

Black list of erroneous/ misleading entries in doxygen docs

Post by paul424 »

Here, I would put all the >>flowers<< from the doc's. Error's , mistypos , lacking information, etc...
For the purpose of feature improvement ....

1.
virtual Viewport* Ogre::RenderTarget::addViewport ...

Adds a viewport to the rendering target. ...

Parameters:
cam The camera from which the viewport contents will be rendered (mandatory)
It's not mandatory , it can be NULL . ( Ogre engine won't crash ).

2.
EdgeData * getEdgeList (void)
Define a default implementation of method from ShadowCaster which implements no shadows.
bool hasEdgeList (void)
Define a default implementation of method from ShadowCaster which implements no shadows.
Description does not match method declaration ( doxygen's script fault ? ) .

3.
From Viewport class : classOgre_1_1Viewport.html
void addListener (Listener *l)
Add a listener to this camera.

4. http://www.ogre3d.org/docs/api/html/cla ... f7a5579b9c

NO IDEA HOW TO DEDUCE DEFAULT CAMERA MEMBERS VALUE.
That is how it will be oriented or positioned with default ctor.
Neither the methods' nor the memebers' info give a clue.
User avatar
Zonder
Ogre Magi
Posts: 1168
Joined: Mon Aug 04, 2008 7:51 pm
Location: Manchester - England
x 73

Re: Black list of erroneous/ misleading entries in doxygen d

Post by Zonder »

Issue pull requests with the fixed documentation.
There are 10 types of people in the world: Those who understand binary, and those who don't...
User avatar
paul424
Gnome
Posts: 314
Joined: Thu May 24, 2012 7:16 pm
x 13

Re: Black list of erroneous/ misleading entries in doxygen d

Post by paul424 »

HOW TO DEDUCE DEFAULT MEMBERS VALUE OF OBJECTS NOT SET IN CTOR IN OGRE ?


What does it mean issue pull request ? , Zonder ?
User avatar
masterfalcon
OGRE Team Member
OGRE Team Member
Posts: 4270
Joined: Sun Feb 25, 2007 4:56 am
Location: Bloomington, MN
x 126
Contact:

Re: Black list of erroneous/ misleading entries in doxygen d

Post by masterfalcon »

I'll take care of these myself. But for future reference: http://www.ogre3d.org/developers/submit-patch
User avatar
paul424
Gnome
Posts: 314
Joined: Thu May 24, 2012 7:16 pm
x 13

Re: Black list of erroneous/ misleading entries in doxygen d

Post by paul424 »

From the ChangeLog of Ogre 1 9 0
Loads of documentation updates
I go to the online docs . By links I get to http://www.ogre3d.org/docs/api/html/Ogr ... ource.html
54 namespace Ogre {
55 // Define ogre version
56 #define OGRE_VERSION_MAJOR 1
57 #define OGRE_VERSION_MINOR 8
58 #define OGRE_VERSION_PATCH 0
59 #define OGRE_VERSION_SUFFIX ""
60 #define OGRE_VERSION_NAME "Byatis"
BUAHAHA ! ( laugh through tears )

Could you just sort the docs versions like there is a link for every recent Ogre3d version , and link to it ?
http://www.ogre3d.org/docs/api/html/index.html Here, no info of which Ogre's version it does describe ... what a failure >_>

On my distro - OpenSuse 12.3 I got 1 9 0 with auto-update. I said ok , time to adapt my source code to newer version . Installed ogre-docs-1.9.0-1.1.x86_64.rpm , but Bang it's empty ! Looks like the maintainers just made dummy package as you didn't prepered anything :

rpm -ql ogre-docs-1.9.0-1.1.x86_64 gives:
/usr/share/OGRE/docs
/usr/share/OGRE/docs/CMakeLists.txt
/usr/share/OGRE/docs/ChangeLog.html
/usr/share/OGRE/docs/CodingStandards.html
/usr/share/OGRE/docs/License.html
/usr/share/OGRE/docs/ReadMe.html
/usr/share/OGRE/docs/api
/usr/share/OGRE/docs/licenses
/usr/share/OGRE/docs/licenses/bsd.txt
/usr/share/OGRE/docs/licenses/freeimage.txt
/usr/share/OGRE/docs/licenses/freetype.txt
/usr/share/OGRE/docs/licenses/libjpeg.txt
/usr/share/OGRE/docs/licenses/libmng.txt
/usr/share/OGRE/docs/licenses/libpng.txt
/usr/share/OGRE/docs/licenses/libtiff.txt
/usr/share/OGRE/docs/licenses/mit.txt
/usr/share/OGRE/docs/licenses/mpl.txt
/usr/share/OGRE/docs/licenses/nedmalloc_boost.txt
/usr/share/OGRE/docs/licenses/uiuc.txt
/usr/share/OGRE/docs/licenses/zlib.txt
/usr/share/OGRE/docs/ogre-logo-wetfloor.gif
/usr/share/OGRE/docs/ogre-logo.gif
/usr/share/OGRE/docs/shadows
/usr/share/OGRE/docs/shadows/OgreShadows.pdf
/usr/share/OGRE/docs/style.css
User avatar
c6burns
Beholder
Posts: 1512
Joined: Fri Feb 22, 2013 4:44 am
Location: Deep behind enemy lines
x 138

Re: Black list of erroneous/ misleading entries in doxygen d

Post by c6burns »

paul424 wrote:BUAHAHA ! ( laugh through tears )
I had the same reaction to your post about media wiki and Jimmy Wales :lol:

The stable version only just recently was updated from 1-8 to 1-9 so to me it's not outlandish to see 1-8 up there. Why don't you build the doxygen docs yourself from source, then you will have the latest docs for the exact version you are using? There should be an OgreDoc build target after generating via cmake.

As far the contents of your distro packages, I think that's something to take up with the package maintainer ... or whatever the process for your distro is.
User avatar
spacegaier
OGRE Team Member
OGRE Team Member
Posts: 4304
Joined: Mon Feb 04, 2008 2:02 pm
Location: Germany
x 135
Contact:

Re: Black list of erroneous/ misleading entries in doxygen d

Post by spacegaier »

New up-to-date online API will become available in the next few days.
Ogre Admin [Admin, Dev, PR, Finance, Wiki, etc.] | BasicOgreFramework | AdvancedOgreFramework
Don't know what to do in your spare time? Help the Ogre wiki grow! Or squash a bug...
User avatar
spacegaier
OGRE Team Member
OGRE Team Member
Posts: 4304
Joined: Mon Feb 04, 2008 2:02 pm
Location: Germany
x 135
Contact:

Re: Black list of erroneous/ misleading entries in doxygen d

Post by spacegaier »

Ogre Admin [Admin, Dev, PR, Finance, Wiki, etc.] | BasicOgreFramework | AdvancedOgreFramework
Don't know what to do in your spare time? Help the Ogre wiki grow! Or squash a bug...
Post Reply