diff options
| author | Marek Polacek <polacek@redhat.com> | 2021-02-03 11:58:13 -0500 |
|---|---|---|
| committer | Marek Polacek <polacek@redhat.com> | 2021-02-03 13:24:40 -0500 |
| commit | b52a1dfe12a6303c7649f3ff5b8dac6c1001d49a (patch) | |
| tree | 36c6a58ad9d498be45eaff3b82ff0f724f87b70a | |
| parent | 5c62e4f255bfac65e18213fd93ee1c9908b4a750 (diff) | |
| download | gcc-b52a1dfe12a6303c7649f3ff5b8dac6c1001d49a.zip gcc-b52a1dfe12a6303c7649f3ff5b8dac6c1001d49a.tar.gz gcc-b52a1dfe12a6303c7649f3ff5b8dac6c1001d49a.tar.bz2 | |
c++: Mark member functions as const [PR98951]
These member functions look like they could be marked const, since
they don't modify any (non-mutable) class members.
PR c++/98951
* call.c (struct z_candidate): Mark rewritten and reversed as const.
(struct NonPublicField): Mark operator() as const.
(struct NonTrivialField): Likewise.
| -rw-r--r-- | gcc/cp/call.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/gcc/cp/call.c b/gcc/cp/call.c index 87a7af1..3068c0f 100644 --- a/gcc/cp/call.c +++ b/gcc/cp/call.c @@ -520,8 +520,8 @@ struct z_candidate { /* The flags active in add_candidate. */ int flags; - bool rewritten () { return (flags & LOOKUP_REWRITTEN); } - bool reversed () { return (flags & LOOKUP_REVERSED); } + bool rewritten () const { return (flags & LOOKUP_REWRITTEN); } + bool reversed () const { return (flags & LOOKUP_REVERSED); } }; /* Returns true iff T is a null pointer constant in the sense of @@ -9474,7 +9474,7 @@ first_non_static_field (tree type, Predicate pred) struct NonPublicField { - bool operator() (const_tree t) + bool operator() (const_tree t) const { return DECL_P (t) && (TREE_PRIVATE (t) || TREE_PROTECTED (t)); } @@ -9491,7 +9491,7 @@ first_non_public_field (tree type) struct NonTrivialField { - bool operator() (const_tree t) + bool operator() (const_tree t) const { return !trivial_type_p (DECL_P (t) ? TREE_TYPE (t) : t); } |