Looky1173 / launchpad-bugs-migration2

0 stars 0 forks source link

[BUG n°1274713] STFReader performance: ORTS skips exterior sounds after reading comment in Lights section #793

Closed Looky1173 closed 9 years ago

Looky1173 commented 10 years ago

Imported from https://bugs.launchpad.net/bugs/1274713

Property Value
Reported by Sid Penstone (penstone)
Date reported Thu, 30 Jan 2014 21:46:10 GMT
Tags tests

Version X1966 (and earlier). Also reproducible in default and other .eng files. This was first noticed in the Paul Charland set of 10 locomotives "cpsw10pk.zip" from Trainsim. There was no ORTS exterior sound in any of these locos; the cab sound worked fine. MSTS behaved normally. By trial, it was discovered that if the exterior sound line in the .eng file: Sound ( "SW1200eng.sms" ) was moved to a location above the Lights() section, the exterior sounds worked normally. It was then noticed that there was a comment line immediately preceding the line containing the closing right bracket of the Lights() section of the .eng file: e.g.

(++++++++++++++++++++++++ End of light section +++++++++++++++++++++++++++++)

)

Sound ( "SW1200eng.sms" ) ) (The last right bracket above is the closing of the Wagon() section.)

If this comment line immediately preceding the end of the Lights() section is removed, the exterior sound works normally, even when the sound line follows the Lights() section. I believe that this location is used for the exterior sound line in many .eng files.

As a further confirmation, the default GP38.eng file was modified by inserting a comment on the line ahead of the closing right bracket of the Lights () section. The sound line ' Sound ( "GP38Eng.sms" ) ' follows the Lights() section. The result was that the GP38 also lost all exterior sounds (i.e. 2-view and 3-view).

In all cases, MSTS works normally - i.e. it correctly produces exterior sounds, with or without the extra comment line. There was a discussion about allowable comment lines here: http://www.trainsim.com/vbts/showthread.php?278838-Why-are-some-comments-ok-and-others-not&highlight=comments

Looky1173 commented 10 years ago

Imported from https://bugs.launchpad.net/or/+bug/1274713/comments/1

Property Value
Posted by Sid Penstone (penstone)
Date posted Thu, 30 Jan 2014 21:49:10 GMT

P.S. The line "#(+++++++++++++++ End of light section +++++++ etc. )" ends on a single line in the file.

Looky1173 commented 10 years ago

Imported from https://bugs.launchpad.net/or/+bug/1274713/comments/2

Property Value
Posted by jeroenp (j-paasschens)
Date posted Sat, 14 Jun 2014 17:38:10 GMT

Dear all. I added a fix (2044). Can someone please confirm it is solved for them as well? Thanks

Looky1173 commented 10 years ago

Imported from https://bugs.launchpad.net/or/+bug/1274713/comments/3

Property Value
Posted by jeroenp (j-paasschens)
Date posted Sat, 14 Jun 2014 17:59:46 GMT

Sorry, I mentioned the wrong version. It should be fixed in 2316

Looky1173 commented 10 years ago

Imported from https://bugs.launchpad.net/or/+bug/1274713/comments/4

Property Value
Posted by Sid Penstone (penstone)
Date posted Mon, 16 Jun 2014 00:58:42 GMT

Hi- I just ran X2318, and my test cases work fine. Thanks Sid.