[webkit-reviews] Win32 compile fix: bool_object.cpp
Darin Adler
darin at apple.com
Sun Jun 12 01:59:43 PDT 2005
On Jun 10, 2005, at 3:43 PM, Justin Haygood wrote:
> bool_object.cpp
>
> Changes:
>
> On Win32, include <assert.h>. Otherwise, nothing. Might be a good
> idea to include <assert.h> on other platforms as well
> Index: bool_object.cpp
> ===================================================================
> RCS file: /cvs/root/JavaScriptCore/kjs/bool_object.cpp,v
> retrieving revision 1.10
> diff -u -r1.10 bool_object.cpp
> --- bool_object.cpp 2005/04/29 00:22:55 1.10
> +++ bool_object.cpp 2005/06/10 22:37:54
> @@ -20,6 +20,7 @@
> *
> */
>
> +
> #include "value.h"
> #include "object.h"
> #include "types.h"
> @@ -28,7 +29,9 @@
> #include "bool_object.h"
> #include "error_object.h"
>
> +#ifdef WIN32
> #include <assert.h>
> +#endif
>
> using namespace KJS;
I don't understand this patch. The old code always included
<assert.h>. Your patch makes it only include <assert.h> on WIN32.
Landing this would break compilation on Mac OS X, for example.
How does this change help porting to Win32?
-- Darin
More information about the webkit-reviews
mailing list