Project

General

Profile

Bug #10213

Some inappropriate consts

Added by Greg Farnum over 9 years ago. Updated over 9 years ago.

Status:
Resolved
Priority:
High
Assignee:
Category:
-
Target version:
-
% Done:

80%

Source:
Development
Tags:
Backport:
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

https://github.com/ceph/ceph/pull/3011/files
https://github.com/ceph/ceph/pull/3037/files

These added some consts that shouldn't be there; I believe I commented on them all.

Associated revisions

Revision 0f1c9fd1 (diff)
Added by Loic Dachary over 9 years ago

msg: do not const methods that return a mutable pointer

Add remove misleading const on post_rx_buffer argument.

http://tracker.ceph.com/issues/10213 Fixes: #10213

Signed-off-by: Loic Dachary <>

History

#1 Updated by Loïc Dachary over 9 years ago

  • Status changed from New to Fix Under Review
  • % Done changed from 0 to 80

My apologies for not being careful enough on this review. https://github.com/ceph/ceph/pull/3050

#2 Updated by Greg Farnum over 9 years ago

  • Status changed from Fix Under Review to Resolved

Thanks!

Also available in: Atom PDF