Re: [PATCH v2 0/9] revision: get rid of the flag leak_pending

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



René Scharfe <l.s.r@xxxxxx> writes:

> The flag leak_pending is weird, let's get rid of it.
>
> Changes from v1: Everything.

;-)

> An independent optimization found while working on this series:
>
>   commit: avoid allocation in clear_commit_marks_many()
>
> Trivial unrelated conversions (included as bonus patches):
>
>   commit: use clear_commit_marks_many() in remove_redundant()
>   ref-filter: use clear_commit_marks_many() in do_merge_filter()
>
> A new function is introduced, will be used by checkout:
>
>   object: add clear_commit_marks_all()
>
> The users of leak_pending are are converted to use alternatives:
>
>   bisect: avoid using the rev_info flag leak_pending
>   bundle: avoid using the rev_info flag leak_pending
>   checkout: avoid using the rev_info flag leak_pending
>
> Cleanups:
>
>   revision: remove the unused flag leak_pending
>   commit: remove unused function clear_commit_marks_for_object_array()

Will take a look at.  Thanks.

>
>  bisect.c           | 30 +++++++++---------------------
>  builtin/checkout.c | 13 +------------
>  bundle.c           | 35 ++++++++++++++++-------------------
>  commit.c           | 19 ++-----------------
>  commit.h           |  1 -
>  object.c           | 11 +++++++++++
>  object.h           |  5 +++++
>  ref-filter.c       |  3 +--
>  revision.c         |  3 +--
>  revision.h         | 12 ------------
>  10 files changed, 46 insertions(+), 86 deletions(-)




[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux