Re: [PATCH 2/2] optimize set_shared_perm()

2013-03-30 Thread Torsten Bögershausen
On 29.03.13 22:20, Junio C Hamano wrote: [snip] The last two points can become a separate preparation step. The result would be easier to read. Your updated adjust_shared_perm() does not begin with: if (!shared_repository) return 0; as the original, but it always

Re: [PATCH 2/2] optimize set_shared_perm()

2013-03-29 Thread Junio C Hamano
Torsten Bögershausen tbo...@web.de writes: optimize set_shared_perm() in path.c: - sometimes the chown() function is called even when not needed. (This can be provoced by running t1301, and adding some debug code) Save a chmod from 400 to 400, or from 600-600 on these files: