shaktazuki Sep 02, 2008 7: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 10: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 2:36 pm
Subject: Re: Patches

Quote by shaktazuki:
My hours on IRC don't appear to coincide with yours.
Naw.. at 8 o' clock in the morning we usually work (students excluded) Wink

shaktazuki Sep 02, 2008 2:48 pm
Subject: Re: Patches

Quote by Eiko:

- 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.
I thought of that too. It was easier at the time to just do a search and replace on ".WAV" to ".SPW".

Quote:

- 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.

I just followed the example of the language test, where both of those were done... sorry!
Embarrassing

What is to be done now?

Eiko Sep 02, 2008 3:53 pm
Subject: Re: Patches

Quote by beko:
Quote by shaktazuki:
My hours on IRC don't appear to coincide with yours.
Naw.. at 8 o' clock in the morning we usually work (students excluded) Wink

correct, I am in bed at that time Laughing

Quote by shaktazuki:
[
I just followed the example of the language test, where both of those were done... sorry!
Embarrassing

I didn't say, that I did no unnecessary checks Wink 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 12: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 5: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 7:16 am
Subject: Re: Patches

Or,

C) delete your file and reupdate!

Wink

Here's a new patch for the language pack. Two minor changes.
Attachments:
lang_eng.zip (Filesize: 1.33 KB)

Eiko Sep 03, 2008 10:33 am
Subject: Re: Patches

language file update is checked in. Thanks Smiling

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 3: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.