[Johnny got his gun]

There is this plugin (supposedly written by Spacedude), and then there is "Ludwigs Sound Download Management", so that would be two different plugins, right?

However before approval of this plugin, fix this:

1. Only add sma, don't ever add compiled amx:es (no reason to zip things unless you have other content)
2. #include <amxmodx>, not <amxmod>!

[Isobold]

Problem 2 fixed, but don't understand why I should not add the amx-file?

Ok, will offer 2 files, sma and amx-file with an readme seperatly...

[Johnny got his gun]

Ok. You get a "possibly unintended assignment" because you do this:

      while(nextline = read_file("soundfiles.txt",nextline,soundfile,127,txtlen)){
         precache_sound(soundfile)
      }

(on while line)
It's perfectly fine to do so. You can suppress the warning by putting another set of () around it all:

      while((nextline = read_file("soundfiles.txt",nextline,soundfile,127,txtlen))){
         precache_sound(soundfile)
      }

[Isobold]

it wasn't me who wrote this, but what's the difference between one pair of () or two???

In some languages the second pair would even be false!

Is this just a workaround, or does the second pair is really needed?



Edit:
Is the webcompiler "dead"? The new compile of the fixed version did not worked. Another plugin I recompiled without modifiing it, did not worked, too. The new files are only a half as tall as the old ones and I always get this when running them:
23:17:45 L 05/05/2004 - 23:08:28: [AMXX] Run time error 22 on line 0 (plugin "join_leave.amx")

[same for sound_files_upload]



I reuploaded the old but working version...

[Johnny got his gun]

The on-the-fly compiler is kind of broken currently...

And about putting = in if/while/etc statements. I think it works like this:

It's not as common to write "if (value = 1)" as it is writing "if (value == 1)", but they are both valid, but do different things. The first one (with the single =) will always be true because value will be set to 1, and then value will be treated as true because it's value is non-0. However these statements supposedly want real "true" or "false" values, not 0, 1 or anything else. So you can go around this by putting parentheses around value = 1, and a value of true or false "will come out of" that operation which is also taken care of first.

Hmm I hope you get my nontechy explanation :-)

[Isobold]

ok, I compiled it again with the compiler I get with amxx. This (fixed) version works fine.

Thx for your explaination. Don't worry about being too technical, i'm studying informatic, only my english is not very good ;)


The sourcecode now is updated too ;)




If somebody is wondering about the number of downloads. This plugin was downloaded something about 50 times before I made the fix.

[Sharp]

Yo i just wanna play songs and shit on the server...
is this too big? size wise ?? maybe just 1 song..1 minute long?? its a hl mod sessrver that i'm playing it off off is this posible?

[QuakerOates]

Hi guys had this plugin working ages ago on .16 havent tried in 1.0 but might try today.
Is there a plugin like this one but only load one sound file when someone joins the server.

Wanted to put a mery xmas tune to my server =]

Thx for response and a merry xmas to all =]]

[Geesu]

Author PM me to have this plugin approved