Sebastian Andersson

I wouldn't use sprintf in new code, snprintf does the same thing but also makes sure you don't write outside of the given length: snprintf(path, sizeof(path), ....

1: I picked it up from move_cone, where a FLAG_LIFESAVE check is used to avoid removing the spell effect if set, but I now see that happens before the call to remove_or_fade_out, so it should be removed. 2: Yes, good suggestion.

Containerfile

Fade out spell effects.

Ignore empty lines in accounts

Nice! It works fine with the EU metaserver. I didn't get any input/output bytes, but that is most likely a problem with my websocket implementation as the other integers are handled the same and they work fine.

Sorry, I developed it and tested on master, but then created the formatted git patch on my other server and missed I had already added those four lines there. The "%.s" format takes an int as the field width, not a size_t, hence the cast, but one can of course just cast it when it is used as a parameter instead. I've done that in the updated patch.

Use tags for matching objects