Am Freitag, 29. September 2006 22:10 schrieb Christian Ohm: > Looks mostly good, just a few comments, some not very relevant. > > On Friday, 29 September 2006 at 19:22, Dennis Schridde wrote: > > @@ -3125,7 +3121,7 @@ > > > > //create the message filename > > aFileName[fileExtension] = '\0'; > > - strcat(aFileName, "mFlagState.bjo"); > > + strcat(aFileName, "mflagsstate.bjo"); > > /*Write the data to the file*/ > > if (!writeFlagFile(aFileName)) > > { > > Typo (one 's' too much)? Yes, typo. Thanks.
> > @@ -588,7 +588,7 @@ > > > > else > > if(IsEqualGUID(&(NetPlay.protocols[id-CON_TYPESID_START].guid), > > &SPGUID_MPLAYER) ) // mplayer { > > - if(system("multiplay\\MplayNow\\mplaynow.exe") != -1) > > // launch > > gizmo, if present. If not, tough... > > + if(system("multiplay/MplayNow/mplaynow.exe") != -1) > > // launch > > gizmo, if present. If not, tough... { > > changeTitleMode(QUIT); > > // shut down warzone... > > } > > That looks a bit outdated and very Windows-centric. Actually I think this part is commented out... Maybe I changed some \\ to much. > > Index: src/frontend.c > > =================================================================== > > --- src/frontend.c (revision 412) > > +++ src/frontend.c (working copy) > > @@ -511,7 +511,7 @@ > > strcpy(pLevelName,DEFAULT_LEVEL); > > seq_ClearSeqList(); > > > > - > > seq_AddSeqToList("CAM1\\c001.rpl",NULL,"CAM1\\c001.txa",FALSE,0); > > + > > seq_AddSeqToList("CAM1/c001.rpl",NULL,"CAM1/c001.txa",FALSE,0); > > > > seq_StartNextFullScreenVideo(); > > break; > > Have you intentionally left the directories out (there were a few more)? You mean the uppercase? No I forgot that. Mostly looked at the \\ when fixing the code... > > Index: src/intelmap.c > > =================================================================== > > --- src/intelmap.c (revision 412) > > +++ src/intelmap.c (working copy) > > @@ -1160,7 +1160,7 @@ > > { > > ASSERT( strlen(((VIEW_RESEARCH *)((VIEWDATA > > *)psMessage->pViewData)-> pData)->pAudio)<244,"sequence path+name greater > > than max string" ); - > > strcpy(aAudioName,"sequenceAudio\\"); > > + strcpy(aAudioName,"sequenceAudio/"); > > strcat(aAudioName,((VIEW_RESEARCH *)((VIEWDATA > > *)psMessage-> > > pViewData)->pData)->pAudio); > > This and the following, for example. > > > Index: src/mission.c > > =================================================================== > > --- src/mission.c (revision 412) > > +++ src/mission.c (working copy) > > @@ -575,9 +575,9 @@ > > //play the cam 2 video > > seq_ClearSeqList(); > > #ifndef PSX > > - > > seq_AddSeqToList("CAM2\\c002.rpl",NULL,"CAM2\\c002.txa",FALSE); > > + > > seq_AddSeqToList("CAM2/c002.rpl",NULL,"CAM2/c002.txa",FALSE); > > #else > > - seq_AddSeqToList("CAM2\\C002.STR","1656f"); > > + seq_AddSeqToList("CAM2/C002.STR","1656f"); > > #endif > > seq_StartNextFullScreenVideo(); > > } > > @@ -586,10 +586,10 @@ > > //play the cam 3 video > > seq_ClearSeqList(); > > #ifndef PSX > > - > > seq_AddSeqToList("CAM2\\cam2out.rpl",NULL,NULL,FALSE); > > - > > seq_AddSeqToList("CAM3\\c003.rpl",NULL,"CAM3\\c003.txa",FALSE); > > + > > seq_AddSeqToList("CAM2/cam2out.rpl",NULL,NULL,FALSE); > > + > > seq_AddSeqToList("CAM3/c003.rpl",NULL,"CAM3/c003.txa",FALSE); > > #else > > - seq_AddSeqToList("CAM3\\C003.STR","1656f"); > > + seq_AddSeqToList("CAM3/C003.STR","1656f"); > > #endif > > seq_StartNextFullScreenVideo(); > > }*/ > > Why do we still have PSX stuff in there? Because no one removed it. :P Per did quite some cleanup, but some is still left... --Dennis
pgpyEniYbm3RJ.pgp
Description: PGP signature
_______________________________________________ Warzone-dev mailing list Warzone-dev@gna.org https://mail.gna.org/listinfo/warzone-dev