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: remove duplicate flush formatter #12437

Merged
merged 1 commit into from Apr 4, 2017

Conversation

Projects
None yet
3 participants
@idealguo

idealguo commented Dec 12, 2016

The method rgw_flush_formatter_and_reset has been invoked in end_header, maybe rgw_flush_formatter is duplicate.

Signed-off-by: Guo Zhandong guozhandong@cmss.chinamobile.com

guozhandong
rgw: remove duplicate flush formatter
Signed-off-by: Guo Zhandong <guozhandong@cmss.chinamobile.com>
@idealguo

This comment has been minimized.

idealguo commented Mar 24, 2017

@cbodley , could you please have a look? is it duplicated?

@cbodley

agreed that end_header() has already flushed at this point

@cbodley cbodley self-assigned this Mar 24, 2017

@cbodley

This comment has been minimized.

@cbodley cbodley merged commit 3c9df55 into ceph:master Apr 4, 2017

3 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
default Build finished.
Details

@idealguo idealguo deleted the idealguo:remove-duplicate-flush branch Apr 25, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment