Initialize variables correctly in utils.py get_cell_size

[danschwarz]

I am using term-image within toot

• queries disabled
• forcing iterm2 image rendering protocol
• using what I think is the hterm.js terminal emulator (which claims to support iterm2 image rendering protocol) - it's the terminal embedded in the Google Cloud Shell webpage. There's a possiblity it might be xterm.js, which does not support iterm2 protocol natively.

Without this PR applied, there are code paths through get_cell_size(...) that reference uninitialized variables.

[AnonymouX47]

Hello!

Thanks for spotting this and the fix. Interestingly, I also just spotted and fixed it some days ago but the containing PR is yet to be merged. The linked commit also fixes another critical bug [^*] within the same function.

Ultimately, these bugs occurred and survived due to the lack of tests for utilities, which is among what #103 is intended to take care of. Even the "over-hyped" static typing which I just introduced in #100 was of no good in this case, or maybe it's just mypy.

Anyways, I'll go ahead and pull the commit referenced earlier, along with a couple other fixes into a branch off v0.7.0 and make a patch release from there.

Thanks so much. :bowing_man:

[^*]: Turns out the other bug was introduced after v0.7.0.

[AnonymouX47]

Fixed and released in v0.7.1.

Kindly test with the new version and let me know the results.