PathFinder status

[LogicParrot]

Regarding the code:
I merged cPathFinder and cPath into one class and called it cPath. The changes have been pushed to Github, there's still some polishing to be done but the interface and the code already feel much cleaner. Comments are welcome, especially about the interface.

Regarding code style:

• Can I make CheckBasicStyle.lua only check a specific file? scanning the whole project is just too slow.
  
• The constructor line is pretty long, I wanted to spread the parameters over several lines, but I can't seem to satisfy CheckBasicStyle.lua. What is your code convention here? Do you just keep the long line?
  
• Should I indent after an #ifdef?
  
• I removed most of the debugging mode bloat, only the really needed parts remain. The flag is stilled called __PATHFIND_DEBUG, better name suggestions are welcome.
  
• I hope all the code convention issues are fixed, Comments are welcome.
  

[worktycho]

You can modify the Allfiles.lst file before running CheckBasicStyle to change which files it runs over. Then just delete Allfiles.lst afterwards so cmake regenerates it.

---

Also you should probably remove the two leading underscores from the PATHFIND_DEBUG macro. Names that start with two underscores are resserved for the c++ implementation.

[xoft]

Rename the macro to "COMPILING_<whatever your test program is called>" - that way the intention is much clearer, than a generic Debug. Debug what? How?

[LogicParrot]

You can modify the Allfiles.lst file before running CheckBasicStyle to change which files it runs over. Then just delete Allfiles.lst afterwards so cmake regenerates it.

Thanks. I cobbled up a script:

Code:

#!/bin/bash
#AllFiles_mine.lst is a list of files you want checked.
cd ......./MCServer/src/
mv AllFiles.lst temp.lst
mv ../../AllFiles_mine.lst AllFiles.lst
lua CheckBasicStyle.lua
mv AllFiles.lst ../../AllFiles_mine.lst
mv temp.lst AllFiles.lst

[xoft]

I was hoping to add some possibilities to the check script, so that it can be run on a single file, or on a list of files. That way, you could use it from an IDE to check the current file, and as a git pre-commit hook to check only the files being committed. Haven't got around to it yet.

[LogicParrot]

Can we consider the interface roughly complete?

[LogicParrot]

I am having trouble include World.h in cPath.h, it is causing some weird compilation errors all over the project. I am confused, have I missed something?

[worktycho]

Its probably Globals.h. World.h assumes it is only included from Globals.h. Its nasty in terms of modularity, but It speeds up MSVC builds.

[LogicParrot]

All blocks that I place disappear after logging out, this is also true for the master branch. What's going on?

[NiLSPACE]

Do you have the world's scheme set to Forgetful? Otherwise the chunk might be corrupted. In that case try removing the region folder.