RecastDemo VC9 project doesn't compile : Linker errors (fix by adding .cpp file to project)

[GoogleCodeExporter]

Reproduction
1. Checkout trunk and open RecastDemo\Build\VC9\Recast.sln
2. Try to compile
3. -> You get linker errors

Expected behaviour:
Should compile - lol.

Version:
Latest trunk checkout

Fix:
In the project map explorer add "PerfTimer.cpp" to Demo/Source and recompile

Original issue reported on code.google.com by lukasm2...@googlemail.com on 30 Aug 2010 at 3:32

[GoogleCodeExporter]

Yeah, it is a known issue, look at author's blog 
http://digestingduck.blogspot.com/2010/08/recast-api-breakage-redux.html

Original comment by Dmitriy....@gmail.com on 30 Aug 2010 at 6:11

[GoogleCodeExporter]

The sources are there, but the project is out of date. I dev mainly on OSX, and 
have rare access to a windows machine.

Original comment by memono...@gmail.com on 30 Aug 2010 at 7:33

[GoogleCodeExporter]

@ dmitriy: You can't expect me to read any page related to the project and see 
if someone made a comment about the issue there! Also this is the official and 
only bug tracker - so if it is not here, i do not see how it is - as you call 
it - "known".

@ memononen: I know. That's why I wrote the simple fix at the end of the report.

As you said you do not develop on OSX i made this patch file from my .vcproj 
file:

http://pastebin.com/VncA4LUd

It also includes a change for default libraries settings, as there was a 
warning saying there is a conflict between them. However, i only changed that 
for debug. 

I hope this is helpful. I'd be glad about feedback

Original comment by lukasm2...@googlemail.com on 30 Aug 2010 at 8:54

[GoogleCodeExporter]

I added few more files today (R213) which have not found their way to the VC 
project. I will try to have access to Windows machine later this week and 
update the project too. 

Original comment by memono...@gmail.com on 31 Aug 2010 at 12:38

• Changed state: Accepted

[GoogleCodeExporter]

Okay, great :)

Original comment by lukasm2...@googlemail.com on 31 Aug 2010 at 1:23

[GoogleCodeExporter]

[deleted comment]

[GoogleCodeExporter]

[deleted comment]

[GoogleCodeExporter]

[deleted comment]

[GoogleCodeExporter]

[deleted comment]

[GoogleCodeExporter]

To all who still have problems at the newest rev (I think r214?) with VC projs.

error C3861: 'snprintf': identifier not found
FIX : replace snprint with sprintf_s

Linker errors
FIX: Re add every file from the folders, to your VC project, because it missing 
some file.

DetourObstacleAvoidance.cpp(197) : error C2660: 'operator new' : function does 
not take 2 arguments
FIX: add to the top : #include <new>

Original comment by warlordgalaris@gmail.com on 8 Sep 2010 at 3:46

[GoogleCodeExporter]

Fixed in r215

Original comment by memono...@gmail.com on 10 Sep 2010 at 6:56

• Changed state: Fixed

[GoogleCodeExporter]

Actually I still have 
DetourObstacleAvoidance.cpp(197) : error C2660: 'operator new' : function does 
not take 2 arguments

but #include <new> solves the problem.

Original comment by warlordgalaris@gmail.com on 10 Sep 2010 at 10:32

[GoogleCodeExporter]

The top of my file looks like this:

//    misrepresented as being the original software.
// 3. This notice may not be removed or altered from any source distribution.
//

#include "DetourObstacleAvoidance.h"
#include "DetourCommon.h"
#include "DetourAlloc.h"
#include "DetourAssert.h"
#include <string.h>
#include <math.h>
#include <float.h>
#include <new>

static int sweepCircleCircle(const float* c0, const float r0, const float* v,
                             const float* c1, const float r1,
                             float& tmin, float& tmax)
{
...

Original comment by memono...@gmail.com on 10 Sep 2010 at 10:37

[GoogleCodeExporter]

Weird, mine doesnt includes the <new> in default :O
I didnt edited it.
Anyways, thanks for this project ! :)

Original comment by warlordgalaris@gmail.com on 10 Sep 2010 at 10:43

[GoogleCodeExporter]

you're welcome! :) Maybe you did svn update on some sub directory?

Original comment by memono...@gmail.com on 10 Sep 2010 at 11:59