client.Export() not working?

ID:141798
 
Nov 7 2008, 9:13 pm
Code:
    proc/ExportSafeFile(savefile/s)
        var/safetext = RC5_Encrypt(s.ExportText("/"), "test")
        del(s)
        s = new("[ckey].sav")
        s["safetext"] << safetext
        src.Export(s)


Problem description:

So, I'm using this code from the rc5 encryption demo, it will create the savefile in the host's folder, but it won't create a new save in my key_info. Does anyone have an idea what's wrong?
Nov 7 2008, 9:25 pm
Don't put anything in the new() proc. It's probably failing because you've assigned file name. Client saves assign their own.
Nov 7 2008, 9:36 pm
In response to Xooxer
I only put something there to check whether the save was valid at all. Before that it didn't work either, but I can retry.

Nope, not working.
Nov 7 2008, 9:53 pm
In response to CIB
You're using del() on s, then proceeding to continue using it. That kinda strikes me as odd.

Have you considered assigning another variable after the del(s) to take care of the rest?
Nov 7 2008, 9:58 pm
In response to Tiberath
Good point. Tried, but failed. This seems to be rather a problem I'm having with BYOND than with the code =/
Nov 7 2008, 9:59 pm
In response to CIB
Well, I'm not sure. Here's Chatters' save, which also uses RC5 and client saves. Maybe you can see a difference:

        Save(mob/chatter/C)
            var/savefile/S = new()
            C.Write(S)
            sleep(5)
            var/filetext = S.ExportText("/")
            var/crypt = RC5_Encrypt(filetext, md5(world.hub_password))
            var/savefile/F = new()
            F["version"] << savefile_version
            F["CRYPTO"] << crypt
            C.client.Export(F)
Nov 7 2008, 10:10 pm
In response to Xooxer
I simply copypasted that into my game and removed all the undefined variables and it.. didn't work? o.0
Nov 7 2008, 10:12 pm
In response to CIB
Weird. Is it making a -1.sav instead?
Nov 7 2008, 10:15 pm
In response to Xooxer
The proc wasn't being called for some reason. Making a verb that does so makes it work correctly.

Under client:
    Del()
        Save()
        sleep(100)
        ..()


Anything wrong with that?
Nov 7 2008, 10:17 pm
In response to CIB
Yes, read this: [link].
Nov 7 2008, 10:22 pm
In response to Xooxer
Oh, I see, that makes sense, though I'd have expected DreamSeeker to call Logout()/Del() a few ticks before it actually disconnects. Saving at a certain rate seems rather messy to me, I think I will rather call Save() everytime I modify a variable that is to be saved, since there aren't many of those.

Thanks alot for the help by the way =)
Nov 7 2008, 10:23 pm
In response to CIB
That's what I do in Chatters, and it seems to work well.