[http://sourceforge.net/p/ufoai/patches/403 Item 403] imported from sourceforge.net tracker on 2013-01-28 20:39:33

This is a patch, that will support savegames in xml-format (both compressed/uncompressed).

There are most certain some problems with other machines (like windows, mac, etc).
But i think they are easy to circumvent.


You need to download the mxml Library sources (
http://www.minixml.org/software.php) and put the files in
src/client/mxml/

Needed are only:
client/mxml/mxml-attr.c
client/mxml/mxml-entity.c
client/mxml/mxml-file.c
client/mxml/mxml-index.c
client/mxml/mxml-node.c
client/mxml/mxml-private.c
client/mxml/mxml-search.c
client/mxml/mxml-set.c
client/mxml/mxml-string.c

And you will need the attached patched, where some more files are included and a diff file.


When it is tested and working it could be committed to the trunk...
===== Comments Ported from Sourceforge =====

====== clarki (2009-02-06 13:46:50) ======

Forgot to mention s.th.:

The game will still be saved both ways, and the xml File will be tried first.

So, if it breaks on loading a savegame just delete the xml file and everythink should work... :)
====== clarki (2009-02-06 14:00:35) ======

SF seems to change the file...

http://www.sven-stalinski.de/ufoai/patch_xml.tgz

md5sum:
http://www.sven-stalinski.de/ufoai/patch_xml.tgz.md5

====== clarki (2009-02-06 14:10:03) ======

Oh, forgotten:

You need to copy the mxml.h file to the src/client/mxml/
also.

SRy, copied the information above from my buildscript ;)
====== clarki (2009-02-06 14:22:28) ======

And another .h file from the mxml archive is needed:
mxml-private.h


Fixed a small bug in mxml_ufoai.c

Actual files are on my webspace, the SF download is corrupted.
====== clarki (2009-02-07 22:05:37) ======

Patch for revision 22463
====== clarki (2009-02-07 22:05:38) ======

mxml files are now located in svn.

attached patch for revision 22463.
the ufo.cbp is patched, don't know, if this will work. can s.o. test it?

Still to do:
- Renaming some functions
- Ensuring coding guidelines ;)
- doing some other cleanups in the files...


File Added: patch_22463
====== tlh2000 (2009-02-08 10:54:45) ======

applied some coding guidelines - but still a lot to do before we can accept it - there is also a compile error included, please check it.
====== tlh2000 (2009-02-09 06:56:07) ======

please check my updated attachment. i've applied some coding guidelines - and introduced a compile error ;)

until the scope of some vars is not reduced and the coding guidelines are not met we can't accept that patch. this is also something someone else can do.... so .... who? volunteers, one step in front please.

i would really like to get this into trunk asap - as i'm planning to move some stuff around which would make this patch unuseable just a few revisions ago and a pain to get it working again.
====== clarki (2009-02-09 21:54:40) ======

patch with some more coding guidelines ;)
====== clarki (2009-02-09 21:54:41) ======

i think i've got it and applied the coding guidelines.

Scope is reduced as much as possible.
patch is for rev 22543
File Added: patch_22543
====== tlh2000 (2009-02-10 17:24:42) ======

Applied to trunk - thanks