diff --git a/imgconverter/qubesimgconverter/__init__.py b/imgconverter/qubesimgconverter/__init__.py index 3710d63..15cd3b4 100644 --- a/imgconverter/qubesimgconverter/__init__.py +++ b/imgconverter/qubesimgconverter/__init__.py @@ -49,9 +49,9 @@ ICON_MAXSIZE = 512 # header consists of two decimal numbers, SPC and LF re_imghdr = re.compile(br'^\d+ \d+\n$') -imghdrlen = lambda w, h: int(math.ceil(math.log10(w)) \ - + math.ceil(math.log10(h)) \ - + 2) +def imghdrlen(w, h): + # width & height are inclusive max vals, and +2 for ' ' and '\n' + return len(str(w)) + len(str(h)) + 2 class Image(object): def __init__(self, rgba, size): diff --git a/imgconverter/qubesimgconverter/test.py b/imgconverter/qubesimgconverter/test.py index 4bcd0a2..3891fc9 100644 --- a/imgconverter/qubesimgconverter/test.py +++ b/imgconverter/qubesimgconverter/test.py @@ -54,6 +54,7 @@ class TestCaseImage(unittest.TestCase): class TestCaseFunctionsAndConstants(unittest.TestCase): def test_00_imghdrlen(self): self.assertEqual(qubesimgconverter.imghdrlen(8, 15), len('8 15\n')) + self.assertEqual(qubesimgconverter.imghdrlen(100, 100), len('100 100\n')) def test_01_re_imghdr(self): self.assertTrue(qubesimgconverter.re_imghdr.match('8 15\n'))