On Thu, May 31, 2018 at 07:19:09PM +0200, Carlos Maiolino wrote: > On Thu, May 31, 2018 at 11:58:51AM -0500, Eric Sandeen wrote: > > On 3/7/18 3:05 AM, Carlos Maiolino wrote: > > > Hi, > > > > > > this is a second version of the patchset aimed to clean up some old macros from > > > xfsprogs, as the objective to bring xfs_buf handling code closer to kernel code. > > > > > > It fixes some pointer castings Dave mentioned on the V1 patch, also, it removes > > > a few unneeded castings I spotted while reviewing the ones Dave mentioned. > > > > > > Cheers > > > > > > Carlos Maiolino (4): > > > Get rid of XFS_BUF_PTR() macro > > > Get rid of XFS_BUF_TARGET() macro > > > get rid of XFS_BUF_COUNT() macro > > > Get rid of XFS_BUF_SET_COUNT() macro > > > > Sorry, I'd still like to merge these but they need a rebase now. > > And did the casting-wars between dchinner & hch ever get resolved? > > > > > I don't know actually, I don't remember seeing Dave replying agreeing or not > with hch What was there to argue? Christoph wants us to rely on undocumented, compiler specific behaviour(*), I want it the pointer arithmetic to be explicitly correct with a cast. Maintainer's choice, really. Cheers, Dave. (*) It's been repeatedly demonstrated that the gcc developers don't care if they break code that relies on undefined behaviour in the C standard. -- Dave Chinner david@xxxxxxxxxxxxx -- To unsubscribe from this list: send the line "unsubscribe linux-xfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
