Hi,

a few more coding style despite the ones I already pointed out:

> +  if(!tile)return FALSE;

Please write this as

  if (! tile)
    return FALSE;

> +      if(PENDING_WRITE(t))
> +        acc+=t->size;
> +      t=t->next;

Please insert an empty line like this:

  if (PENDING_WRITE (t))
    acc += t->size;

  t = t->next;


Other than these style issues, the patch looks good. In particular your
benchmarking data looks promising. Nice work!


Sven


_______________________________________________
Gimp-developer mailing list
Gimp-developer@lists.XCF.Berkeley.EDU
https://lists.XCF.Berkeley.EDU/mailman/listinfo/gimp-developer

Reply via email to