Nitpick:

Coding style and such

balt, basher
- __enter__ and __exit__ are defined in wx.Dialog, no need to override
- last wx.TheClipboard pruned
- nitpick in mod links (report warned as possibly uninitialized, wrapping)

bosh: imports cleanup, coding style, tableUpdate is a subset of toDelete IIUC

Couple leftover TODOs in patchers
Nitpick bolt - review the DataDict item:

bolt:

- REVIEW: bolt.DataDict:why does it try to return data for game master file ?
- comments and renames (header -> firstPickle)
- TableColumn: used self as an iterator
- typos
- Changed Settings.changed,deleted lists to sets as they were used as sets
 and deleted an unused variable.
Deleted InstallerProject_SyncPack - took methods and settings along:

Under #173.

_InstallerLink: projectExists(), getProjectPath() were only called from
InstallerProject_SyncPack
Also:

    dirs['builds'] = dirs['app'].join(u'Builds')

was only used in getProjectPath:

        return bosh.dirs['builds'].join(archive.sroot)

Some never implemented feature I guess.
Fix[bosh]: code paths in InitSettings:

See #178.

Ex.: If backup settings exists and is corrupted and usebck == False
dictFile.load() will try to load the backup again - fixed:
introduced 'ignoreBackup' param
Introduced delBackup whereupon I delete the backup when I know it's
"corrupted"

Exceptions raised from the error message:
u'%s' % "cPickle.UnpicklingError: invalid load key, 'ÿ'."
Traceback (most recent call last):
  File "<input>", line 1, in <module>
UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 44: ordinal not in range(128)

So yeah - use repr: http://stackoverflow.com/q/27652804/281545

Maybe other occurrences of this pattern in the code...

Needs work
RIP: dialogs from basher:

ListBoxes
ColorDialog
ImportFaceDialog
Mod_BaloGroups_Edit - dialogs.modList = None # TODO: Balo - bin
CreateNewProject
Nitpick: bosh, installers help messages and some de-wx'ing:

Be aware that balt.tooltip wraps at 50 chars - not a problem for the strings
here.

bosh:
checkUncheckMod belongs to ModList not to List
swapOrder needs not be before init
moveIsSafe is static
Deleted ModBaseData, ModBasePanel:

Under #173.
Possible bug - iniList may be None:

Untested - IniList may not have been initialized - similar logic as in
InstallersList.OnEditLabel --> if iniList is not None: # line 2843
Notice that in IniPanel.SetBaseIni() there are such checks also - smelly -
a property ?
Merge branch 'utumno-163-wip' into dev:

Under #163.
Various nitpick in links files - de-wx'ing - couple TODOs
dialog subclasses ripped of basher
Fixups in bosh.InitSettings()
Review: c33859b67e68384d9ae474dc5965cacc4b86b91b