rehevkor5
commented
4 years ago - Updated to Python 3.8. Updated libraries.
- Fixed issue loading 64 bit Windows registry values.
- When one of the parsers decides the DBR should be ignored because it is invalid, rather than returning a somewhat mysterious empty dict, it now throws an InvalidItemError with relevant information in it. This helps prevent those empty dicts from accidentally being added to inappropriate places, because the exception must be handled in order for the code to continue running. Checks have been added around calls to parse() which should maintain equivalent behavior to the preexisting code. However, informative debug log messages have been added where appropriate.
- Parsers now use InvalidItemError rather than StopIteration because the parser function is just a regular function call, it's not an iterator next() call.
- If no images are available for creating a sprite sheet, we log a useful warning message rather than allowing PIL function calls to raise a somewhat confusing error. Also, we now explicitly close all image objects once we no longer need them.
- DBR read now uses "with" to automatically close the file.
- DBR read using generator comprehension rather than list comprehension when we don't actually need to create a list and can instead process items one at a time.
- DBR read no longer calls variable.parse_value() twice for each variable.
- Fixed IDE warning about using mutable default value in arg to parse()
- Simplified equipment file iteration, and made the filtering of particular folders platform-agnostic which were previously Windows specific.
- In regex: pipe character twice in character class is redundant. Escaping forward slash is unnecessary.
- In the README, clarify that the "C:/tqdb" path is not actually required, it is just an example. Also, now referring to the project directory more as the "git clone" rather than as the "repository" which is a little confusing.
fonsleenaars