On 03/29/2012 06:38 PM, Jan Kiszka wrote:
>       for (vma = mm->mmap; vma; vma = vma->vm_next) {
> -             if (!is_cow_mapping(vma->vm_flags)
> -                 || !(vma->vm_flags & VM_WRITE))
> -                     continue;

I prefer this style.

> (...)
> +             if (is_cow_mapping(vma->vm_flags) &&
> +                 (vma->vm_flags & VM_WRITE)) {
> +                     result = __ipipe_pin_vma(mm, vma);
> +                     if (result < 0)
>                               goto done_mm;

Than this one, which uselessly cause indentation and scopes nesting.

-- 
                                                                Gilles.

_______________________________________________
Adeos-main mailing list
[email protected]
https://mail.gna.org/listinfo/adeos-main

Reply via email to