From kde-commits  Fri Apr 08 14:16:26 2005
From: Tobias Anton <tobias.anton () esc-electronics ! de>
Date: Fri, 08 Apr 2005 14:16:26 +0000
To: kde-commits
Subject: Re: kdelibs/khtml
Message-Id: <200504081616.26038.tobias.anton () esc-electronics ! de>
X-MARC-Message: https://marc.info/?l=kde-commits&m=111296981821717

Am Freitag, 8. April 2005 14:56 schrieb Lubos Lunak:

> > Comparing your backward goto to a loop, I don't see any advantage of the
> > backward goto. The main disadvantage of the backward goto: The exit
> > condition is not clear, whereas a loop would make it clear.
>
>  Maybe if the code was written with a loop in mind, but the JPEG loader is
> written as linear code that just needs to go back in one place. I even
> tried to make it a loop first, but after getting it wrong two times I
> simply gave up (I did the mistake of trying to use a flag to bail out
> instead of the other way around, and that was pretty non-trivial).

When looking at the source, I think that my patch is equivalent to yours. But 
since I have no test case and no testing environment ready, I sticked to 
commenting only. Sorry for criticizing your coding style in general - it 
should have addressed the backward goto only.

Shall I commit the patch attached to my previous mail?


Cheers
-- Tobias