Re: small bugfixes for wily-0.13.41-9libs

  • From: Tommy Pettersson <ptp@xxxxxxxxxxxxxx>
  • To: wilyfans@xxxxxxxxxxxxx
  • Date: Mon, 31 Jan 2005 16:11:30 +0100

On Thu, Jan 27, 2005 at 06:59:51PM +0100, a pretentious moron named Tommy 
Pettersson wrote:
> Here's a rendering bug. 
[...]
> -             r.max.y += f->font->height;
> +             r.max.y = MAX(r.max.y+f->font->height, v->r.max.y);

If someone thinks that this doesn't work, they are absolutely
right.  My original fix was a bloated if(foo>bar) foo=bar,
but I edited the patch just before sending it to the list (not
a smart move), to use the more "elegant" MAX...  I now have
changed my mind and think that it would be much more elegant
with MIN.  (The cleared area should of course be restricted,
not unnecessarily and wrongly extended.)



-- 
Tommy Pettersson <ptp@xxxxxxxxxxxxxx>

Other related posts: