diff options
author | Marc Espie <espie@cvs.openbsd.org> | 2000-06-17 14:43:41 +0000 |
---|---|---|
committer | Marc Espie <espie@cvs.openbsd.org> | 2000-06-17 14:43:41 +0000 |
commit | d0f862960eef69fdd7801239999a637a2b2d5021 (patch) | |
tree | d044a096bd0bd61354904e3f2933b6e6e9260f0c /usr.bin/ftp | |
parent | ba25a00b79070aa39d76e92f0c729f0e2a3f39a4 (diff) |
This removes the few instances of Lst_New left.
- replaces Lst_Duplicate with Lst_Clone, which does not allocate storage
- split Lst_Concat into Lst_Concat/Lst_ConcatDestroy
Thus, all the LstValid checks are gone, since we always invoke list
functions with valid pointers.
Note that dynamic list allocation accounted for roughly 20% of all calls
to malloc. The extraneous calls to malloc left are now mostly in parse.c,
which makes some wasteful usage of temporary buffers.
With those few patches, the code is sturdier, and easier to maintain.
Reviewed by millert@
Diffstat (limited to 'usr.bin/ftp')
0 files changed, 0 insertions, 0 deletions