> >> I can see use doing the size of calculation in three places:
> >>
> >> $ git grep sizeof | grep "\["
> >> src/pacman/pacman.c: for(i = 0; i < sizeof(signals) /
> >> sizeof(signals[0]); i++) {
> >> src/pacman/sync.c: for(j = 0; j < sizeof(glob_skips) /
> >> sizeof(glob_skips[0]); j++) {
> >> src/pacman/util.c: static const int unitcount = sizeof(labels) /
> >> sizeof(labels[0]);
> >
> > Wrong regex:
> >
> > $ git grep "sizeof *([^)]*) */"
> > lib/libalpm/pkghash.c: loopsize = sizeof(prime_list) /
> > sizeof(*prime_list);
> > src/pacman/pacman.c: for(i = 0; i < sizeof(signals) /
> > sizeof(signals[0]); i++) {
> > src/pacman/sync.c: for(j = 0; j <
> > sizeof(glob_skips) / sizeof(glob_skips[0]); j++) {
> > src/pacman/util.c: static const int unitcount = sizeof(labels) /
> > sizeof(labels[0]);
> >
> > So yes, alpm has *one* use for it! :D
> >
>
> OK - keep it in util-common then. Will you submit a patch with those
> changed to use ARRAYSIZE? Or should I just patchcount++ myself?
>
> A
I'll do it just now.
--
Pierre Neidhardt
