Bug(?): functions get added to class type of merged namespace when they shouldn't be about ldoc HOT 10 CLOSED

Interesting! I cloned your project, and found the script that made the config.ld. And (of course) the thing is working OK, because these two functions are considered to be functions in the Blitzwizard module!

I would have preferred it to misbehave ;) There are some 'not found' references which I'll look at.. I do remember that the master module thing is sensitive to ordering of the file list.

...
found master module     d:\kool\blitwizard\src\luafuncs.c
found master module     d:\kool\blitwizard\src\luafuncs.c
usage:14: function or section not found: blitwizard.graphics.getDefaultCamera in blitwizard.graphics blitwizard.graphics.getDefaultCamera
usage:14: function or section not found: blitwizard.graphics.camera:gameUnitsPerPixel in blitwizard.graphics blitwizard.graphics.camera:gameUnitsPerPixel
d:\kool\blitwizard\src\luafuncs_objectphysics.c:508: object:enableMovableCollision: function not found: object:shape_info in this module object:shape_info
d:\kool\blitwizard\src\luafuncs_objectphysics.c:519: object:enableStaticCollision: function not found: object:shape_info in this module object:shape_info
d:\kool\blitwizard\src\luafuncs_media_object.c:438: positionedSound:new: function not found: position in this module position
output written to d:\kool\blitwizard\docs

from ldoc.

So it works for you? I can see if there is any update to pull, but I believe to be on the latest master...

The missing two references in objectphysics don't appear here (and shouldn't!). The other missing references are "intentional", I haven't implemented and described those yet.

from ldoc.

I suspect it has to do with ordering. The file which is intended to be the 'master module' should be seen first. Perhaps on Windows your script gave me another ordering? I'll try that tomorrow.

from ldoc.

Sorry for that script btw :) it takes the files out of src/Makefile.am since I didn't want to write the list of all source files twice into a text file and update it in both places.

So the file list not magically generated or anything, just copied from src/Makefile.am since I'm lazy! (in fact, the script SHOULD also always end up with the same order. Not that I would have paid attention to that when writing it..)

from ldoc.

Just as a small update, the issue is still present with the most recent ldoc version (pulled a few moments ago).

"loadResourceArchive" is planted onto the "object" class type, while it is just in the "blitwizard" module (see luafuncs.c and verify for yourself). Same for "setStep" in the same file.

from ldoc.

Use this zip to reproduce the bug: http://www.blitwizard.de/temp/reproduce.zip

Check if "loadResourceArchives" is planted onto the "object" class type in the "blitwizard" module (it shouldn't be, instead it should be a separate function in the "blitwizard" module, and not part of any class type).

If I can help with anything else, let me know.

from ldoc.

Yep, I can confirm that it does indeed plant it in the wrong place. This is cool, since what can reproduce is what I can fix ....

from ldoc.

OK, here's the quick fix, while I think of the best way to fix this properly; swop luafuncs.c and luafuncs_object.c around in config.ld. It's using luafuncs_object as the master module and merging stuff into the type object section.

The proper fix involves not re-using the section, let's see...

from ldoc.

Try out latest commit, see if your docs look exactly like you need them ;)

from ldoc.

Works perfectly now, thanks a lot :) :)

from ldoc.