From 8052c585b3df8d74619b8cbecfb6b40c7135a59d Mon Sep 17 00:00:00 2001 From: Pascal Massimino Date: Tue, 21 Nov 2017 23:57:57 -0800 Subject: [PATCH] remove some petty TODOs from vwebp. they're rather low-prio anyway. Change-Id: I76dd74fcfb1c974c6f8a074472455d3f0b202e01 --- examples/vwebp.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/examples/vwebp.c b/examples/vwebp.c index 8feeccea..6376e563 100644 --- a/examples/vwebp.c +++ b/examples/vwebp.c @@ -248,9 +248,9 @@ static void HandleKey(unsigned char key, int pos_x, int pos_y) { } } } else if (key == 'i') { + // Note: doesn't handle refresh of animation's last-frame (it's quite + // more involved to do, since you need to save the previous frame). kParams.print_info = 1 - kParams.print_info; - // TODO(skal): handle refresh of animation's last-frame too. It's quite - // more involved though (need to save the previous frame). if (!kParams.has_animation) ClearPreviousFrame(); glutPostRedisplay(); } else if (key == 'd') { @@ -260,8 +260,8 @@ static void HandleKey(unsigned char key, int pos_x, int pos_y) { } static void HandleReshape(int width, int height) { - // TODO(skal): should we preserve aspect ratio? - // Also: handle larger-than-screen pictures correctly. + // Note: reshape doesn't preserve aspect ratio, and might + // be handling larger-than-screen pictures incorrectly. glViewport(0, 0, width, height); glMatrixMode(GL_PROJECTION); glLoadIdentity();