shaktazuki | Sep 02, 2008 8:16 am |
---|---|
Subject: Patches Here are some patches I've created. If there's a different way you want me to submit, please tell me. My hours on IRC don't appear to coincide with yours. Thanks! |
|
Attachments: patches.zip (Filesize: 11.83 KB) |
Eiko | Sep 02, 2008 11:21 am |
---|---|
Subject: Re: Patches Thanks for the patches. I checked in the update for the lang file. why did interplay rename all the wav files?!? They did it with the music files, too. There they used .msc. anyway... A few remarks about the resinstaller patch: - Duplicating all the wav related functions is not the best way to go. Generally duplicating code should be avoided. you could use a string to store the wave file extension. Set the string to .wav or .spw, depending on the tests, and use this string to generate all the sound file names. - You can use the function openFile (from file.cpp) so you don't have to distinguish between upper and lower case file names. - You don't need to test with extra path delimiter as it is added to sMAXPath earlier if missing. |
beko | Sep 02, 2008 3:36 pm |
---|---|
Subject: Re: Patches Quote by shaktazuki: Naw.. at 8 o' clock in the morning we usually work (students excluded) My hours on IRC don't appear to coincide with yours. |
shaktazuki | Sep 02, 2008 3:48 pm |
---|---|
Subject: Re: Patches Quote by Eiko: I thought of that too. It was easier at the time to just do a search and replace on ".WAV" to ".SPW".
Quote:
I just followed the example of the language test, where both of those were done... sorry! What is to be done now? |
Eiko | Sep 02, 2008 4:53 pm |
---|---|
Subject: Re: Patches Quote by beko:
correct, I am in bed at that time Quote by shaktazuki: [ I didn't say, that I did no unnecessary checks I noticed that today morning. with a single 'openFile( sMAXPath + testFileName, "r");' you will know if there are .WAV files in the MAX folder or the ones with the other extension. But that doesn't greatly matter. More important is the duplicate code. Please provide a version without it, and I'll be glad to commit the patch. |
shaktazuki | Sep 03, 2008 1:52 am |
---|---|
Subject: Re: Patches 1st - you are aware that to do that I will have to entirely rework the sound installation functions? But ok! Also, SVN reports a problem with the updated english language pack - it has an exclamation point icon on it. What is that? |
beko | Sep 03, 2008 6:32 am |
---|---|
Subject: Re: Patches probably a conflict with your local version. you worked on your local file and svn detects that. now it gets a newer version from svn so it ends up showing you a conflict. you can a.) undo local changes to get a fresh file from svn b.) merge svn version with your version and get a new file that you should send to us again |
shaktazuki | Sep 03, 2008 8:16 am |
---|---|
Subject: Re: Patches Or, C) delete your file and reupdate! Here's a new patch for the language pack. Two minor changes. |
|
Attachments: lang_eng.zip (Filesize: 1.33 KB) |
Eiko | Sep 03, 2008 11:33 am |
---|---|
Subject: Re: Patches language file update is checked in. Thanks hm, what do you mean with "entirely rework the sound installation functions"? I though of just replacing all ".wav" by "+ waveExtension". |
shaktazuki | Sep 03, 2008 4:11 pm |
---|---|
Subject: Re: Patches Yes, that's changing every sound file name for each time it is passed as a parameter... well, we'll see how it goes. I have a plan. |