Nexuiz startup vs network stability

Developer discussion of experimental fixes, changes, and improvements.

Moderators: Nexuiz Moderators, Moderators

Postby divVerent » Mon Mar 30, 2009 6:16 am

Against, as NetRadiant aims to be mostly code compatible with GtkRadiant, as long as there are other GtkRadiant forks out there that may have features NetRadiant could use.

If I did a batch change to change all these ifdefs, patches will be unlikely to still apply.

What I can do is add robustness by adding -DWIN32 to the compiler command line when building a Win32 build.

Actually, I just checked: the Makefile, which is an 1:1 port of the previous SCons based build system, always adds -DWIN32 when doing a Windows build. So in NetRadiant, this is just fine. Anyone adding a Visual Studio project has to read the Makefile anyway, as NetRadiant consists of lots of subprojects that need different compile settings (e.g. because there are multiple header files of the same name). I know it's ugly, but it matches what Id Software released as GtkRadiant and I'm not going to change that as long as there are other merge-worthy Radiant forks like DarkRadiant.

BTW, I just merged NetRadiant's q3map2 with VorteX's changes to the Urban Terror map compiler fork.
1. Open Notepad
2. Paste: ÿþMSMSMS
3. Save
4. Open the file in Notepad again

You can vary the number of "MS", so you can clearly see it's MS which is causing it.
divVerent
Site admin and keyboard killer
 
Posts: 3809
Joined: Thu Mar 02, 2006 4:46 pm
Location: BRLOGENSHFEGLE

Postby tundramagi » Mon Mar 30, 2009 6:44 am

divVerent wrote:BTW, I just merged NetRadiant's q3map2 with VorteX's changes to the Urban Terror map compiler fork.


What new goodies does that bring :D :D :D?
tundramagi
Forum addon
 
Posts: 974
Joined: Sun Jan 04, 2009 4:53 pm

Postby Alien » Mon Mar 30, 2009 11:46 am

Should have I expected more? :mrgreen:
Code: Select all
libs\math\vector.h(98) : fatal error C1001: An internal error has occurred in the compiler.
1>(compiler file 'msc1.cpp', line 1411)
1> To work around this problem, try simplifying or changing the program near the locations listed above.
1>Please choose the Technical Support command on the Visual C++
1> Help menu, or open the Technical Support help file for more information
Btw,
Code: Select all
#ifndef boolean
#ifdef __cplusplus
#define   boolean   bool
#else
#define boolean int
#endif
#endif
should be after
Code: Select all
#ifdef _WIN32
#include "windows.h"
#include "stdio.h"
#endif
in radiant_jpeglib.h otherwise it overrides internal windows header boolean definitions.
Alien
Forum addon
 
Posts: 1212
Joined: Tue Apr 22, 2008 7:12 am

Postby divVerent » Mon Mar 30, 2009 11:54 am

That file is unmodified from idsoftware's :P

And what does it override? How does windows.h define boolean?

If windows.h does define boolean, I rather suggest that this block should be skipped instead. So I moved the windows.h include more up. Please test.
1. Open Notepad
2. Paste: ÿþMSMSMS
3. Save
4. Open the file in Notepad again

You can vary the number of "MS", so you can clearly see it's MS which is causing it.
divVerent
Site admin and keyboard killer
 
Posts: 3809
Joined: Thu Mar 02, 2006 4:46 pm
Location: BRLOGENSHFEGLE

Postby Alien » Mon Mar 30, 2009 12:04 pm

It's rpcndr.h which:
Code: Select all
typedef unsigned char boolean;
Then we get this error: http://msdn.microsoft.com/en-us/library/ws579tt7(VS.80).aspx

IdSoftware used(-s) STLPort instead.
Alien
Forum addon
 
Posts: 1212
Joined: Tue Apr 22, 2008 7:12 am

Postby divVerent » Mon Mar 30, 2009 12:26 pm

STLPort instead of what?

Why can't you just use STLPort too?
1. Open Notepad
2. Paste: ÿþMSMSMS
3. Save
4. Open the file in Notepad again

You can vary the number of "MS", so you can clearly see it's MS which is causing it.
divVerent
Site admin and keyboard killer
 
Posts: 3809
Joined: Thu Mar 02, 2006 4:46 pm
Location: BRLOGENSHFEGLE

Postby Alien » Mon Mar 30, 2009 12:46 pm

They use stlport in zeroradiant (it was required last time i compiled it) instead of relying of correct microsoft stl implementation.
Alien
Forum addon
 
Posts: 1212
Joined: Tue Apr 22, 2008 7:12 am

Postby Alien » Mon Mar 30, 2009 10:33 pm

Tested your commit. jpeg6 now builds correctly.
Alien
Forum addon
 
Posts: 1212
Joined: Tue Apr 22, 2008 7:12 am

Previous

Return to Nexuiz - Development

Who is online

Users browsing this forum: No registered users and 1 guest

cron