emo/busybox
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

unit-tests: remove unnecesary field from struct bbunit_listelem

Browse Source 
In the initial submission struct bbunit_listelem was manipulated by
custom list functions implemented in bbunit.c. Since the tests are now
added to the list by llist_add_to_end(), which allocates the llist_t
objects behind the scenes, there's no need for the *next field.

function                                             old     new   delta
unit_main                                            142     141      -1
bbunit_strrstr_elem                                   24      16      -8
bbunit_obscure_weak_pass_elem                         24      16      -8
bbunit_obscure_strong_pass_elem                       24      16      -8
------------------------------------------------------------------------------
(add/remove: 0/0 grow/shrink: 0/4 up/down: 0/-25)             Total: -25 bytes

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
This commit is contained in:
Bartosz Golaszewski 2015-08-13 15:57:21 +02:00 committed by Denys Vlasenko
parent fb52769f4a
commit d077565bb2
2 changed files with 0 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
include/libbb.h
View File

@ -1983,7 +1983,6 @@ static ALWAYS_INLINE unsigned char bb_ascii_tolower(unsigned char a)
typedef void (*bbunit_testfunc)(void); typedef void (*bbunit_testfunc)(void);
struct bbunit_listelem { struct bbunit_listelem {
struct bbunit_listelem* next;
const char* name; const char* name;
bbunit_testfunc testfunc; bbunit_testfunc testfunc;
}; };

1
libbb/bbunit.c
View File

@ -71,7 +71,6 @@ int unit_main(int argc UNUSED_PARAM, char **argv UNUSED_PARAM)
tests_failed++; tests_failed++;
} }
tests_run++; tests_run++;
el = el->next;
} }
#if WANT_TIMING #if WANT_TIMING