[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: lynx-dev [PATCH 2.8.4dev.18] GridText.c cleanup
|
From: |
Thomas Dickey |
|
Subject: |
Re: lynx-dev [PATCH 2.8.4dev.18] GridText.c cleanup |
|
Date: |
Sun, 25 Feb 2001 06:51:21 -0500 (EST) |
> This patch does not add any new functionality. All I did was I went
> through more than 1000 lines of extremely badly written obfuscated C code
> worthy of a first year CS student...
>
> I managed to remove more than 1/3 of this code, unobfuscate most of
> the algorithms I could understand, remove unused structure members,
> remove duplicate code chunks, remove duplicate logic, introduce
> properly named tmp variables, etc etc etc. But this is the only a
> small fraction of GridText.c, and this file is only a small fraction
> of lynx... Of all the useful software I have seen, lynx definitely
> has the worst source code...
>
> The patch takes around 66K, so I hesitate to include it here. It is
> on
>
>
> ftp://ftp.math.ohio-state.edu/pub/users/ilya/nonprivate/diff_lynx-2.8-4-18-gridtext-cleanup
I'll look into it after the problems with NEXT_DOC are resolved, and we've
gotten some feedback on TRST (I found a 4th bug yesterday -- overlapping
fields).
--
Thomas E. Dickey <address@hidden> <address@hidden>
http://dickey.his.com
ftp://dickey.his.com
; To UNSUBSCRIBE: Send "unsubscribe lynx-dev" to address@hidden