search

juanitogan / mkwinfont

Update to Simon Tatham's font utilities for working with Windows FNT and FON font files.
http://www.chiark.greenend.org.uk/~sgtatham/fonts/
21 stars 5 forks source link

added if __name__ == "__main__" around main part of code so i can use this as a library #2

Closed dvorakroth closed 2 years ago

dvorakroth commented 2 years ago

extremely simple change all in all

it looks like a lot, but it's really just adding the if statement to each file, then indenting a whole bunch of lines after it, otherwise unchanged

juanitogan commented 2 years ago

Yeah, weird that the indents totally trashed git's delta algorithm like that. Clearly git hates tabs. 😆 Looks like you also changed the error handling a bit with the argument parsing. Seemed good to me.

Anyway, I'm too busy at the moment to test this, so I'm putting all my faith in you. Thanks for the help.

dvorakroth commented 2 years ago

no problem 😄

btw the "changes" to the error handling are also just the diff choking; there's some duplicated lines of code with differing indentations -- and then the diff matches them up all wrong because the whitespace changed

i know that, because i had to use a proprietary diff tool that lets me manually align the old/new versions, to make sure i REALLY didn't change the error handling 😅

juanitogan commented 2 years ago

Oh, man, I see that now. I could see the offset mess in the diff, but, yes, I was reading too much into the heavy highlighting. Thanks git!