public inbox for linux-xfs@vger.kernel.org
 help / color / mirror / Atom feed
* 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