Re: [PATCH] t/helper/test-lazy-name-hash: fix compilation

2017-12-22 Thread Junio C Hamano
Stefan Beller writes: >> Heh; I do not think there particularly is much difference between >> stricter flags and DEVELOPER flags, but I would rather not lose the >> removal of duplicated 'today' I did while I queued the previous one >> ;-) > > > I did not realize it was queued anywhere, please ig

Re: [PATCH] t/helper/test-lazy-name-hash: fix compilation

2017-12-22 Thread Stefan Beller
> Heh; I do not think there particularly is much difference between > stricter flags and DEVELOPER flags, but I would rather not lose the > removal of duplicated 'today' I did while I queued the previous one > ;-) I did not realize it was queued anywhere, please ignore then.

Re: [PATCH] t/helper/test-lazy-name-hash: fix compilation

2017-12-22 Thread Junio C Hamano
Stefan Beller writes: > I was compiling origin/master today with the DEVELOPER compiler flags > today and was greeted by > > t/helper/test-lazy-init-name-hash.c: In function ‘cmd_main’: > t/helper/test-lazy-init-name-hash.c:172:5: error: ‘nr_threads_used’ may be > used uninitialized in this func

[PATCH] t/helper/test-lazy-name-hash: fix compilation

2017-12-20 Thread Stefan Beller
I was compiling origin/master today with the DEVELOPER compiler flags today and was greeted by t/helper/test-lazy-init-name-hash.c: In function ‘cmd_main’: t/helper/test-lazy-init-name-hash.c:172:5: error: ‘nr_threads_used’ may be used uninitialized in this function [-Werror=maybe-uninitialized]

Re: [FYI PATCH] t/helper/test-lazy-name-hash: fix compilation

2017-12-19 Thread Jeff Hostetler
On 12/18/2017 4:49 PM, Stefan Beller wrote: I was compiling origin/master today with stricter compiler flags today and was greeted by t/helper/test-lazy-init-name-hash.c: In function ‘cmd_main’: t/helper/test-lazy-init-name-hash.c:172:5: error: ‘nr_threads_used’ may be used uninitialized in t

Re: [FYI PATCH] t/helper/test-lazy-name-hash: fix compilation

2017-12-18 Thread Jonathan Tan
On Mon, 18 Dec 2017 13:49:47 -0800 Stefan Beller wrote: > I was compiling origin/master today with stricter compiler flags today > and was greeted by > > t/helper/test-lazy-init-name-hash.c: In function ‘cmd_main’: > t/helper/test-lazy-init-name-hash.c:172:5: error: ‘nr_threads_used’ may be > u

[FYI PATCH] t/helper/test-lazy-name-hash: fix compilation

2017-12-18 Thread Stefan Beller
I was compiling origin/master today with stricter compiler flags today and was greeted by t/helper/test-lazy-init-name-hash.c: In function ‘cmd_main’: t/helper/test-lazy-init-name-hash.c:172:5: error: ‘nr_threads_used’ may be used uninitialized in this function [-Werror=maybe-uninitialized]