generic/232 regression - bda250db ("xfs: rewrite xfs_dq_get_next_id using xfs_iext_lookup_extent")

DISCUSSION Christoph Hellwig 2 participants 2 messages Jul 5, 2017 1 message not in the archive

lore

Share & tools

lore

A message that is not in the archive (sent off-list or not yet received): [email protected]

Christoph Hellwig

author

2017-07-05 15:23 UTC #1 lore raw

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..

Brian Foster

2017-07-05 15:27 UTC #2 lore raw

Quoted 5 lines · Christoph Hellwig

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

Quoted 4 lines

--
To unsubscribe from this list: send the line "unsubscribe linux-xfs" in the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html