sinjax/ftlmerge — pull request #1 by mathieui (2012-10-18 19:09:09+00:00)
Title: A few changes (mostly aesthetics)

main changes:

- Reworked indentation (s/tabs/4 spaces/), which explains that almost every
line has changed
- Most variables and functions are now PEP8-compliant (no CamlCase)
- Direct dict initialisation instead of creation + manual assignment
- List expanding with .append(element) instead of += [element]
- merge_*() functions do not need the merged object anymore, since they
operate on the subelements
- (more) error handling on open()
- correct a wrong offset in ftlmerge_tool.py, the length was wrong
(could have only two args and write the "merge" of the first one
by overwriting the second)
- Python3 compat

(eventual) problems:

- Can't get the same md5sums for the files created by python2 and
python3. I believe this is due to python 3.3 randomizing the hashes of
the dicts.

I tried to run the game with both files, and had it seemed the same to me.