[r32] Compiling... well, you know :>

RazTK · Post by **RazTK** » Sat Feb 23, 2008 21:02

src\fragglescript\t_load.cpp(292) : error C2065: 'SpawnedThings' : undeclared identifier
src\fragglescript\t_load.cpp(292) : error C2228: left of '.Size' must have class/struct/union type is ''unknown-type''
src\fragglescript\t_load.cpp(292) : error C2227: left of '->actor' must point to class/struct/union/generic type

Post by **Graf Zahl** » Sat Feb 23, 2008 21:50

Please don't try to make SVN builds until further notice. I was well aware of the prpblem shortly after I committed the code.

RazTK · Post by **RazTK** » Sat Feb 23, 2008 23:02

Perhaps I shouldn't make at all if you're going to keep commiting untested/unfinished changes to the SVN (which seems wrong IMO, especially to the trunk directory). Or maybe I should just let you encounter those errors by yourself instead of reporting them. Just my personal opinions.

Post by **Graf Zahl** » Sun Feb 24, 2008 0:09

I'm in the middle of some major restructuring of the FraggleScript code right now. Of course I try not to commit non-compiling code but it can happen that some revisions are broken as a result. In any case, the changes that caused these errors have been completely reverted because I overlooked something and the whole thing didn't work. Otherwise I would have committed a fix as soon as I noticed the problem. But what's the point in fixing a compiler error when the result still doesn't work?

Furthermore, there's really nothing gained by these revisions. The code that's being changed is only used when playing the handful of Legacy WADs that happen to work with GZDoom.

I'll notify you as soon as this is done and some real changes are being made. But this can take some time. All the work I'm doing right now is making adjustments for the upcoming ZDoom changes.

RazTK · Post by **RazTK** » Sun Feb 24, 2008 0:47

Sorry, I guess I got a little bit swept away with my response. It's reasonable to assume that the latest changes may be a work in progress and possibly untested. I really shouldn't be too surprised in situations like this. Anyway, thanks for the heads up about the whole status.

Post by **Graf Zahl** » Mon Feb 25, 2008 16:59

Now you may do SVN builds again.

I just finished my FS rewrite. That code leaked even more than I would have imagined...

Post by **Enjay** » Mon Feb 25, 2008 17:43

Graf Zahl wrote:Now you may do SVN builds again.

Or maybe not

r42

Code: Select all

t_vari.cpp
c1xx : fatal error C1083: Cannot open source file: '.\src\fragglescript\t_vari.cpp': No such file or directory
t_spec.cpp
t_script.cpp
.\src\fragglescript\t_script.cpp(629) : error C2664: 'atterm' : cannot convert parameter 1 from 'void (__cdecl *)(void)' to 'void (__fastcall *)(void)'
        None of the functions with this name in scope match the target type
t_saveg.cpp
c1xx : fatal error C1083: Cannot open source file: '.\src\fragglescript\t_saveg.cpp': No such file or directory

Post by **Graf Zahl** » Mon Feb 25, 2008 19:05

Grrr... I worked with my backup project and forgot to update the real one...

Post by **Enjay** » Mon Feb 25, 2008 19:23

I still get this with a release build for r43

Code: Select all

t_script.cpp
.\src\fragglescript\t_script.cpp(629) : error C2664: 'atterm' : cannot convert parameter 1 from 'void (__cdecl *)(void)' to 'void (__fastcall *)(void)'
        None of the functions with this name in scope match the target type

A debug compile succeeds though.

RazTK · Post by **RazTK** » Mon Feb 25, 2008 20:56

Same here.

Post by **Graf Zahl** » Mon Feb 25, 2008 21:08

Argh! That code works fine in debug mode but not in release...

Post by **Enjay** » Mon Feb 25, 2008 21:31

r44 release

Code: Select all

========== Build: 4 succeeded, 0 failed, 9 up-to-date, 0 skipped ==========

What's more, I just loaded up the decorate file that I've been keeping on my desktop to test the "some strange combinations of launcher and command line options disable dynamic lights" bug and the dynamic lights didn't get disabled. I'll keep an eye on what's happening with that one.