On 12/08/12 07:35, Andrew Gregory wrote: > Signed-off-by: Andrew Gregory <[email protected]> > --- > src/pacman/query.c | 6 ++++-- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/src/pacman/query.c b/src/pacman/query.c > index 9afe680..d882e5a 100644 > --- a/src/pacman/query.c > +++ b/src/pacman/query.c > @@ -123,8 +123,10 @@ static int query_fileowner(alpm_list_t *targets) > } > > /* append trailing '/' removed by realpath */ > - path[rootlen++] = '/'; > - path[rootlen] = '\0'; > + if(strcmp(path, "/") != 0) {
Given this is the uncommon case, can we change this to the less expensive: if(rootlen == 1 && path[0] == '/') > + path[rootlen++] = '/'; > + path[rootlen] = '\0'; > + } > > db_local = alpm_get_localdb(config->handle); > >
