View Issue Details

IDProjectCategoryView StatusLast Update
0001564FSSCP---------public2008-11-21 07:00
ReporterFUBAR-BDHR Assigned Tophreak  
PrioritynormalSeverityminorReproducibilityalways
Status resolvedResolutionfixed 
Product Version3.6.9 
Summary0001564: add-sun-bitmap / remove-sun-bitmap doesn't work when used more than once
DescriptionAdd a sun using add-sun-bitmap. Remove it using remove-sun-bitmap. Works fine. Add a second sun and remove it. It does not remove.
Additional InformationReproduced in 3.6.9 and 3_6_10-20071028T. Debug build minimizes screen for a second like it's going to pop up an assert but it pops right back to game play. Nothing in log file but I'll attach it anyways. Also attaching test mission. Just fly straight for about 40 seconds.
TagsNo tags attached.

Activities

2008-01-06 08:59

 

fs2_open.log (14,717 bytes)

2008-01-06 09:00

 

suntest.fs2 (3,073 bytes)

2008-01-08 03:27

 

planet_test.fs2 (6,063 bytes)

FUBAR-BDHR

2008-01-08 03:27

developer   ~0008787

Just found out the same bug exists with add/remove-background-bitmap.

FUBAR-BDHR

2008-01-08 05:28

developer   ~0008790

Last edited: 2008-01-08 05:45

It appears the sexps actually work but the values getting stored to the variables must be wrong. Hard coding the values in the remove sexps works.

Attaching planet_test3 with the hard coded values.

If you get a chance while your looking at that part of the code I noticed 2 other quarks. One is you can't changed the variable back to a number in the remove sexps. The other is when you change the variable the add sexps is saveing to the bitmap you are adding goes to <invalid>.

2008-01-08 05:29

 

planet_test3.fs2 (6,027 bytes)

Tolwyn

2008-01-08 14:48

reporter   ~0008794

yes, I've run into this issue more than once while coding missions for Saga.

Using numbers instead of variables does the trick.

taylor

2008-07-17 16:31

administrator   ~0009464

Not going to work on this.

phreak

2008-10-26 02:53

developer   ~0010125

Last edited: 2008-10-26 02:57

I have a possible fix. suntest.fs2 does not assert anymore, and by my inspection of the SEXPs, it looks like it works properly. I was unable to run the planet_test missions so I don't know about that.

Try the next nightly build.

chief1983

2008-10-28 17:30

administrator   ~0010132

I should have nightlies for Windows back in a day or two. Had to move and haven't got my desktop set up anywhere yet.

FUBAR-BDHR

2008-11-06 03:51

developer   ~0010158

Well I thought this was working. My initial test was with both sun and background in the same mission. That did work. I then changed the background to sun also. So basically there are 2 sun events. First set of add/removes work but the second do not. No idea why as it gives the correct values of what it is trying to remove. Attaching 2 missions. Nova2 is the second part by itself which does work. Nova contains two tests. The first one works. The second which is identical to Nova2 does not.

2008-11-06 03:51

 

nova2.fs2 (3,601 bytes)

2008-11-06 03:52

 

nova.fs2 (5,045 bytes)

phreak

2008-11-18 03:29

developer   ~0010215

applied a patch that should work in this case.

FUBAR-BDHR

2008-11-21 05:45

developer   ~0010249

Looks like you nailed it this time. I tested with the 2 nova missions, one new one I had for Vid and an even more complex one. Worked like a charm.

Issue History

Date Modified Username Field Change
2008-01-06 08:59 FUBAR-BDHR New Issue
2008-01-06 08:59 FUBAR-BDHR File Added: fs2_open.log
2008-01-06 09:00 FUBAR-BDHR File Added: suntest.fs2
2008-01-08 03:27 FUBAR-BDHR File Added: planet_test.fs2
2008-01-08 03:27 FUBAR-BDHR Note Added: 0008787
2008-01-08 04:53 taylor Status new => assigned
2008-01-08 04:53 taylor Assigned To => taylor
2008-01-08 05:28 FUBAR-BDHR Note Added: 0008790
2008-01-08 05:29 FUBAR-BDHR File Added: planet_test3.fs2
2008-01-08 05:29 FUBAR-BDHR Note Edited: 0008790
2008-01-08 05:45 FUBAR-BDHR Note Edited: 0008790
2008-01-08 14:48 Tolwyn Note Added: 0008794
2008-07-17 16:31 taylor Note Added: 0009464
2008-07-17 16:31 taylor Assigned To taylor =>
2008-07-17 16:31 taylor Status assigned => new
2008-10-02 18:07 phreak Status new => assigned
2008-10-02 18:07 phreak Assigned To => phreak
2008-10-26 02:53 phreak Note Added: 0010125
2008-10-26 02:57 phreak Note Edited: 0010125
2008-10-28 17:30 chief1983 Note Added: 0010132
2008-11-06 03:51 FUBAR-BDHR Note Added: 0010158
2008-11-06 03:51 FUBAR-BDHR File Added: nova2.fs2
2008-11-06 03:52 FUBAR-BDHR File Added: nova.fs2
2008-11-18 03:29 phreak Note Added: 0010215
2008-11-21 05:45 FUBAR-BDHR Note Added: 0010249
2008-11-21 07:00 phreak Status assigned => resolved
2008-11-21 07:00 phreak Resolution open => fixed