Project

General

Profile

Actions

Emulator Issues #628

closed

[Soap] r2362 does not build on windows

Added by BhaaL about 15 years ago.

Status:
Invalid
Priority:
Normal
Assignee:
% Done:

0%

Operating system:
N/A
Issue type:
Bug
Milestone:
Regression:
No
Relates to usability:
No
Relates to performance:
No
Easy:
No
Relates to maintainability:
No
Regression start:
Fixed in:

Description

Uh, guess you did a great job rewriting FileUtil for cross platform stuff
etc, but theres a few things that prevent it from compiling on Win32:

1) Log.h redefines ERROR_LOG (at least) even if DEBUG and DEBUGFAST are not
defined, this yields a lot of linker errors.
2) Dolphin.vcproj does not include WxUtils.h/cpp.
3) Win32 does not have an equivalent of strndup.
4) SanitizePath uses copy.size(), while copy is a plain char* ;)
5) Some files include stdafx.h, but they arent added to the project/do not
exist.

Actions #1

Updated by BhaaL about 15 years ago

Steps in order to make it build:

1) DolphinWX.vcproj needs Src/WxUtils.h and Src/WxUtils.cpp included.
2) Core.vcproj needs Src/LogManager.h, Src/LogManager.cpp and Src/Corerecording.cpp
removed.
3) Added #if defined(DEBUG) || defined(DEBUGFAST) around the LOGLEVEL checks in
Common/Src/Log.h to prevent the redefines in non-Debug builds.
4) Common/Src/FileUtil.cpp: fix line 137 with strlen() rather than copy.size(), add a
Win32-implementation for strndup (koders.com has one for instance).
5) Common/Src/CPUDetect.cpp complains about a bool* to PBOOL conversion on line 96
6) HLE, Src/PCHW/DSoundStream.h cant find stdafx in the current folder.
7) Common/Src/CommonFuncs.h has a forward declaration for GetLastErrorMsg, but it is
not declared anywhere.

Actions #2

Updated by nakeee about 15 years ago

  • Status changed from New to Accepted

Thanks for the really high quality feedback:)

Actions #3

Updated by Anonymous about 15 years ago

r2376 ?

Actions #4

Updated by BhaaL about 15 years ago

No problem, always glad to help.

And no, r2376 isnt completely clean yet. CPUDetect still complains about the bool* to
PBOOL conversion on the IsWOW64Process call.

Actions #5

Updated by nakeee about 15 years ago

  • Status changed from Accepted to Invalid

soap is going to be merged to trunk
no point in lingering:)

Actions

Also available in: Atom PDF