Re: [PATCH 0/2] a few more redundant system include cleanups

2019-07-03 Thread Carlo Arenas
On Wed, Jul 3, 2019 at 11:49 AM Junio C Hamano wrote: > > Carlo, no need to rush, but has this happened already? I do not > recall seeing an update, and I am wondering if I missed one. Junio; had yet to send the update, so nothing was missed Carlo

Re: [PATCH 0/2] a few more redundant system include cleanups

2019-07-03 Thread Junio C Hamano
Jeff King writes: > On Wed, Jun 19, 2019 at 02:12:30AM -0700, Carlo Arenas wrote: > >> > I did a quick grep for similar cases, and didn't find any that I think >> > would be problematic. There were a few cleanups, below. >> >> would you mind if I add your 2 patches to a series and include that >

Re: [PATCH 0/2] a few more redundant system include cleanups

2019-06-19 Thread Jeff King
On Wed, Jun 19, 2019 at 02:12:30AM -0700, Carlo Arenas wrote: > > I did a quick grep for similar cases, and didn't find any that I think > > would be problematic. There were a few cleanups, below. > > would you mind if I add your 2 patches to a series and include that > missing one?, that way I'l

Re: [PATCH 0/2] a few more redundant system include cleanups

2019-06-19 Thread Carlo Arenas
On Tue, Jun 18, 2019 at 8:53 AM Jeff King wrote: > > Thanks, this looks good to me. I fixed the system headers in xinclude.h, > but didn't think to check for individual C files including them. also noticed that after the refactoring some files will include a couple of headers twice (not an issue,