View Issue Details

IDProjectCategoryView StatusLast Update
0000289FSSCPcutscenespublic2004-12-30 16:26
ReporterAlpha0 Assigned Totaylor  
PrioritynormalSeverityminorReproducibilityalways
Status resolvedResolutionfixed 
PlatformDell Latitude D600 NotebookOSWindows 2000 ProfessionalOS Version5.00.2195 SP4
Product Version3.6 
Summary0000289: Cutscenes do not work with recently released fs2_open v3.6.5.
DescriptionCustscenes do not play at any point in the released 3.6.5 version. Upon startup the game goes straight to the pilot select screen. Also in Tech Room / Cutscenes, movies do not play.
Steps To ReproduceStart game, game goes directly to pilot selection.
Alternatively, go to Tech Room / Cutscenes and attempt to play any of the ones listed.
Additional InformationTest system: Dell Latitude D600 Notebook, Intel Centrino 1.6GHz, 512MB RAM, Windows 2000 Professional SP4, Mobility Radeon 9000 w/ 32MB for graphics, graphics drivers version 6.13.10.6279. DX version 8.1 (4.08.01.0881).
After skimming through the code, it appears that the bug is located at code\directx\dx8show.cpp, line 106. swprintf expects a wide character string, but is being passed a non-wide character one instead. Proposed fix:
Change line 106 of code\directx\dx8show.cpp from
swprintf(wFile, L"%s", szFile);
to
mbstowcs (wFile, szFile, strlen (szFile) + 1);
TagsNo tags attached.

Activities

Goober5000

2004-12-26 20:33

administrator   ~0001458

Um...

What cutscenes are you talking about? The MVE ones on the CD, or the AVI ones?

As for the wide-character thing... taylor, what do you make of that?

taylor

2004-12-26 21:12

administrator   ~0001462

The wide-character thing was something FryDay changed to remove the ATL dependency. Might want to ask him about that.

Goober5000

2004-12-26 21:50

administrator   ~0001464

Assigning to Fry_Day then.

Alpha0

2004-12-27 07:18

reporter   ~0001470

Just to clarify, I was refering to the AVI cutscenes. The DivX codec is installed and they used to work fine with previous versions of fs2_open.

taylor

2004-12-30 10:30

administrator   ~0001478

OK I did go ahead and look at this. mbstowcs() would work fine but the fix is really as simple as %s -> %S which would convert it properly. Haven't commited this yet but will soon.

taylor

2004-12-30 16:26

administrator   ~0001479

Fixered.

Issue History

Date Modified Username Field Change
2004-12-26 14:21 Alpha0 New Issue
2004-12-26 20:33 Goober5000 Note Added: 0001458
2004-12-26 20:33 Goober5000 Assigned To => taylor
2004-12-26 20:33 Goober5000 Status new => assigned
2004-12-26 21:12 taylor Note Added: 0001462
2004-12-26 21:50 Goober5000 Note Added: 0001464
2004-12-26 21:50 Goober5000 Assigned To taylor => Fry_Day
2004-12-27 07:18 Alpha0 Note Added: 0001470
2004-12-30 10:30 taylor Note Added: 0001478
2004-12-30 10:30 taylor Assigned To Fry_Day => taylor
2004-12-30 16:26 taylor Status assigned => resolved
2004-12-30 16:26 taylor Resolution open => fixed
2004-12-30 16:26 taylor Note Added: 0001479