search

rliou92 / python-umonitor

Manage monitor configuration automatically.
MIT License
39 stars 6 forks source link

Fix UnicodeDecodeError #10

Closed quinn-brittain closed 2 years ago

quinn-brittain commented 2 years ago

Decode being set to UTF-8 was causing errors. Changing to Latin-1 fixed the UnicodeDecodeError.

rliou92 commented 2 years ago

I'm no expert on text encoding, but I don't think changing it to Latin-1 is a good solution for all the users of this program. Seems too specific. I don't want to merge until I become convinced that this would be good for everyone.

I'm glad it works for you though!

quinn-brittain commented 2 years ago

Yeah, I had concerns too on how this would behave on other setups too. The push you made rebuilding the screen.c has fixed the issue for me at least so merging would not be necessary.

Thanks!