indygreg added inline comments.

INLINE COMMENTS

> bdiff.c:44
>  
> -     *lr = l = (struct bdiff_line *)malloc(sizeof(struct bdiff_line) * i);
> +     *lr = l = (struct bdiff_line *)calloc(i, sizeof(struct bdiff_line));
>       if (!l)

Memory throughput is already a bottleneck in bdiff code. Is there an impact on 
`hg perfbdiff` with the change from `malloc()` to `calloc()`?

REPOSITORY
  rHG Mercurial

BRANCH
  alloc-overflow (bookmark) on default (branch)

REVISION DETAIL
  https://phab.mercurial-scm.org/D1877

To: alex_gaynor, #hg-reviewers, durin42
Cc: indygreg, durin42, mercurial-devel
_______________________________________________
Mercurial-devel mailing list
Mercurial-devel@mercurial-scm.org
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel

Reply via email to