On 23/12/15 10:27, Terry Bolt wrote:
> Signed-off-by: Terry Bolt <[email protected]>
> ---
>  lib/libalpm/diskspace.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 

That does not really comply with the coding style given every other loop
is like the original.

A

> diff --git a/lib/libalpm/diskspace.c b/lib/libalpm/diskspace.c
> index 5bb2173..7ee1289 100644
> --- a/lib/libalpm/diskspace.c
> +++ b/lib/libalpm/diskspace.c
> @@ -62,7 +62,7 @@ static void mount_point_list_free(alpm_list_t *mount_points)
>  {
>       alpm_list_t *i;
>  
> -     for(i = mount_points; i; i = i->next) {
> +     for(i = mount_points; i != NULL; i = i->next) {
>               alpm_mountpoint_t *data = i->data;
>               FREE(data->mount_dir);
>       }
> 

Reply via email to