Why was replay format changed?

[TradeMark]

trademark: i didn't notice any bug reports from you before release, is it perhaps because you, like, didn't test it?

Or because the only way you can notice this is when you upload replay at replays.adune.nl

This is because only the host has to specify the script and Spring sends it over by itself. Reduces chance for sync errors due to lobby errors by a huge amount.

Fucking stupid eh, that we reduced chance for sync errors caused by lobby bugs?

My point is that you make such changes which requires me to use unitsync.dll to read the replay script.txt, when in old versions there was no need for such, and you could easily just cut the script.txt from the replay by using the header data. Now you need to process the stuff with unitsync.dll, when the simpliest thing is to just tell where is the script.txt and how long it is.

You guys love to make / explain stuff too complicated

[imbaczek]

it's not that hard to parse a binary stream, really. you could actually try a recent build (post 78.1.1) and see for yourself that some bugs (like the missing startscript) were fixed.

[MelTraX]

TradeMark, it's basically a bug. You shouldn't have to change anything to get to the startscript, it's just not there in the current version. It will be in the next. (I know it looks like it's there but it isn't where it's supposed to be.)

[Auswaschbar]

Information is wrong. Its not (yet) fixed.

[MelTraX]

I didn't say it's fixed.. I said it will be in the version..

[Auswaschbar]

Should now work. Uploaded a replay to http://replays.adune.nl/?648, no errors. (can be deleted, thought)

[MelTraX]

Yeah, that one seems to work fine. :)

[TradeMark]

Should now work. Uploaded a replay to http://replays.adune.nl/?648, no errors. (can be deleted, thought)

So there is two script.txt's now, first my own, then the host?

[MelTraX]

Should be the same. The second one is in the demostream. It's the one actually sent over the network (which is recorded in the demofile).

The first one is just a copy for convinience so you don't have to parse the demostream (or guess the position/do hacky stuff).

[Auswaschbar]

Should now work. Uploaded a replay to http://replays.adune.nl/?648, no errors. (can be deleted, thought)

So there is two script.txt's now, first my own, then the host?

No, they are both from the host (and such, equivalent copies).

Or do you need such a script too:

[GAME]
{
HostIP=91.66.184.52;
HostPort=8454;
IsHost=0;
ModHash=989582745;
MapHash=-1843968990;
Mapname=Mearth_v3.smf;
GameType=BA661.sd7;

MyPlayerName=Auswaschbar;
}

???

[TradeMark]

I need it as it was before

[Tobi]

That's not going to happen.

[MelTraX]

It's as close as it'll get. Just work with that replay Auswaschbar uploaded.

[TradeMark]

I want to know who played the replay, so everyone wont get autohost as player when they play, nor get their autohosts buggy scripts without flags or ranks etc.

Cant you send the data important for sync only, and include the player own script.txt. flags etc probably wont desync the game :p

[Hoi]

That's not going to happen.

[Tobi]

Actually that kind of stuff would be possible (including lobby generated script (ie. the one Auswaschbar quoted) and then TradeMark concvincing all lobby devs to still include this player information. (Since it won't be required for Spring anymore.)

[lurker]

Actually that kind of stuff would be possible (including lobby generated script (ie. the one Auswaschbar quoted) and then TradeMark concvincing all lobby devs to still include this player information. (Since it won't be required for Spring anymore.)

Well, lobby devs don't have to include that; if both scripts are include Trademark can parse the first for myplayername and get the rest of the info from the real script.

[Tobi]

Ya but he then still doesn't have flags etc. when the host didn't write those in the script.

[MelTraX]

And he would still have to parse the demostream to do it right.

[Tobi]

Nah, a new field could be added to the header, clientScriptSize

Client script could then follow (host) script, no need to parse stream.