diff mbox series

[5/6] Make set_drops_artifact_from_ir non-const.

Message ID 20200429175133.19814-6-gprocida@google.com
State Superseded
Headers
Series Clean-ups |

Commit Message

Giuliano Procida April 29, 2020, 5:51 p.m. UTC 
  Setters should be non-const but set_drops_artifact_from_ir isn't.
This patch fixes this.

This reason this didn't cause a compilation failure is that const
shared_ptr<X> is equivalent to X *const, not const X*.

Note that resolving the apparent const-safety issue will require
std::experimental::propagate_const or similar.

	* include/abg-suppression.h
	(suppression_base::set_drops_artifact_from_ir):
	Drop const qualifier.
	* src/abg-suppression.cc
	(suppression_base::set_drops_artifact_from_ir):
	Drop const qualifier.

Signed-off-by: Giuliano Procida <gprocida@google.com>
---
 include/abg-suppression.h | 2 +-
 src/abg-suppression.cc    | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)
diff mbox series

Patch

diff --git a/include/abg-suppression.h b/include/abg-suppression.h
index 4f1fb417..6383b932 100644
--- a/include/abg-suppression.h
+++ b/include/abg-suppression.h
@@ -71,7 +71,7 @@  public:
   get_drops_artifact_from_ir() const;
 
   void
-  set_drops_artifact_from_ir(bool) const;
+  set_drops_artifact_from_ir(bool);
 
   bool
   get_is_artificial() const;
diff --git a/src/abg-suppression.cc b/src/abg-suppression.cc
index ab67f40a..6d9a0f5f 100644
--- a/src/abg-suppression.cc
+++ b/src/abg-suppression.cc
@@ -96,7 +96,7 @@  suppression_base::get_drops_artifact_from_ir() const
 /// specification is to avoid adding the matched ABI artifact to the
 /// internal representation.
 void
-suppression_base::set_drops_artifact_from_ir(bool f) const
+suppression_base::set_drops_artifact_from_ir(bool f)
 {priv_->drops_artifact_ = f;}
 
 /// Test is the suppression specification is artificial.