Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

rgw: drop useless lines #19817

Merged
merged 1 commit into from Jan 10, 2018
Merged

rgw: drop useless lines #19817

merged 1 commit into from Jan 10, 2018

Conversation

qrGitHub
Copy link

@qrGitHub qrGitHub commented Jan 7, 2018

There is no 'break' statement in the for loop, so 'eiter' must be equal to 'ent_map.end()' in this place.

* There is no 'break' statement in the for loop, so 'eiter' must be equal to 'ent_map.end()' in this place.

Signed-off-by: Bingyin Zhang <zhangbingyin@cloudin.cn>
@qrGitHub
Copy link
Author

qrGitHub commented Jan 8, 2018

Hi @cbodley, would you help to review this? Thanks.


// Either the back-end telling us truncated, or we don't consume all
// items returned per the amount caller request
truncated = (truncated || eiter != ent_map.end());

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@yuriw
Copy link
Contributor

yuriw commented Jan 9, 2018

@yuriw yuriw merged commit 319474a into ceph:master Jan 10, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants