Fixed Negative fpoints for RMRelations

[Levis]

I see this popping up in error logs:

RUNTIME ERROR - file: nations\nations.c; line: 868
Negative func 'sqrt' argument
RUNTIME ERROR - file: nations\nations.c; line: 512
Not object
RUNTIME ERROR - file: nations\nations.c; line: 1070
Incorrect copy from single to array
RUNTIME ERROR - file: nations\nations.c; line: 523
missed attribute: index
RUNTIME ERROR - file: nations\nations.c; line: 523
no rAP data
RUNTIME ERROR - file: nations\nations.c; line: 47
Incorrect copy from single to array
RUNTIME ERROR - file: nations\nations.c; line: 868
Negative func 'sqrt' argument
RUNTIME ERROR - file: nations\nations.c; line: 512
Not object
RUNTIME ERROR - file: nations\nations.c; line: 1070
Incorrect copy from single to array
RUNTIME ERROR - file: nations\nations.c; line: 523
missed attribute: index
RUNTIME ERROR - file: nations\nations.c; line: 523
no rAP data
RUNTIME ERROR - file: nations\nations.c; line: 47
Incorrect copy from single to array

The first line seems to refer to this:

if(rel > REL_WAR) ChangeRMRelation(&char, iNation, -(sqrt(fPoints) * NEUTRAL_MULT));

In UpdateRMRelations where fpoints is an input variable. could it be this is not set up right?

 

[Pieter Boelen]

What are you doing in the game when that happens?
You can put a TraceAndLog prior to show the values of the various variables.

 

[Levis]

Spotted it in the log file of @A.H so maybe he can elaborate

http://www.piratesahoy.net/threads/higher-caliber-gun-on-captured-ship-then-max.25394/

 

[Pieter Boelen]

The first line seems to refer to this:

if(rel > REL_WAR) ChangeRMRelation(&char, iNation, -(sqrt(fPoints) * NEUTRAL_MULT));

In UpdateRMRelations where fpoints is an input variable. could it be this is not set up right?

I have completely rewritten that function now and that line no longer exists.
So that means this bug can no longer exist either. Maybe we'll get some new issues now, but not this one anymore.