* Re: generic/232 regression - bda250db ("xfs: rewrite xfs_dq_get_next_id using xfs_iext_lookup_extent") [not found] <20170705151158.GC51285@bfoster.bfoster> @ 2017-07-05 15:22 ` Christoph Hellwig 2017-07-05 15:27 ` Brian Foster 0 siblings, 1 reply; 2+ messages in thread From: Christoph Hellwig @ 2017-07-05 15:22 UTC (permalink / raw) To: Brian Foster; +Cc: linux-xfs, Christoph Hellwig, Darrick J. Wong Thanks Brian, your fix looks correct to me. I usually got for curly braces on both sides of the else if they are needed, but thay might be a bit too nitpicky.. ^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: generic/232 regression - bda250db ("xfs: rewrite xfs_dq_get_next_id using xfs_iext_lookup_extent") 2017-07-05 15:22 ` generic/232 regression - bda250db ("xfs: rewrite xfs_dq_get_next_id using xfs_iext_lookup_extent") Christoph Hellwig @ 2017-07-05 15:27 ` Brian Foster 0 siblings, 0 replies; 2+ messages in thread From: Brian Foster @ 2017-07-05 15:27 UTC (permalink / raw) To: Christoph Hellwig; +Cc: linux-xfs, Darrick J. Wong On Wed, Jul 05, 2017 at 05:22:59PM +0200, Christoph Hellwig wrote: > Thanks Brian, > > your fix looks correct to me. I usually got for curly braces on > both sides of the else if they are needed, but thay might be a bit > too nitpicky.. Ok, I can fix that up and maybe add a comment too. Otherwise I'll write up a commit log and send a proper patch. Thanks for the quick feedback.. Brian > -- > To unsubscribe from this list: send the line "unsubscribe linux-xfs" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2017-07-05 15:27 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20170705151158.GC51285@bfoster.bfoster>
2017-07-05 15:22 ` generic/232 regression - bda250db ("xfs: rewrite xfs_dq_get_next_id using xfs_iext_lookup_extent") Christoph Hellwig
2017-07-05 15:27 ` Brian Foster
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox