Thanks Mircea :)
To all: I've had a look and made some comments, but this patch is definitely worth
another look by at least another person.
On Sep 28, 2011, at 1:51 PM, Mircea Markus wrote:
On 28 Sep 2011, at 10:05, Galder Zamarreño wrote:
> Thanks Mircea. I've made some comments.
Thanks, I've replied and updated the pull request.
Also added a summary:
https://github.com/infinispan/infinispan/pull/551
>
> To all:
>
> If you're gonna submit a big pull req, please use the description to *summarise
the changes* in order to help reviewers. This helps focus the attention of the reviewer to
the pieces of code that are most relevant.
+1
_______________________________________________
infinispan-dev mailing list
infinispan-dev(a)lists.jboss.org
https://lists.jboss.org/mailman/listinfo/infinispan-dev
--
Galder Zamarreño
Sr. Software Engineer
Infinispan, JBoss Cache