From a1d96dfeeb2ca31f7c978dcdffe6e253514e6e7f Mon Sep 17 00:00:00 2001 From: Bartek Kryza Date: Mon, 27 Sep 2021 21:59:39 +0200 Subject: [PATCH] Refactored bool setters --- src/uml/class_diagram_model.cc | 20 ++++++------ src/uml/class_diagram_model.h | 20 ++++++------ src/uml/class_diagram_visitor.cc | 56 ++++++++++++++++---------------- 3 files changed, 48 insertions(+), 48 deletions(-) diff --git a/src/uml/class_diagram_model.cc b/src/uml/class_diagram_model.cc index a686c46e..917cb062 100644 --- a/src/uml/class_diagram_model.cc +++ b/src/uml/class_diagram_model.cc @@ -150,14 +150,14 @@ class_member::class_member( bool class_member::is_relationship() const { return is_relationship_; } -void class_member::set_is_relationship(bool is_relationship) +void class_member::is_relationship(bool is_relationship) { is_relationship_ = is_relationship; } bool class_member::is_static() const { return is_static_; } -void class_member::set_is_static(bool is_static) { is_static_ = is_static; } +void class_member::is_static(bool is_static) { is_static_ = is_static; } // // element @@ -231,29 +231,29 @@ class_method::class_method( bool class_method::is_pure_virtual() const { return is_pure_virtual_; } -void class_method::set_is_pure_virtual(bool is_pure_virtual) +void class_method::is_pure_virtual(bool is_pure_virtual) { is_pure_virtual_ = is_pure_virtual; } bool class_method::is_virtual() const { return is_virtual_; } -void class_method::set_is_virtual(bool is_virtual) { is_virtual_ = is_virtual; } +void class_method::is_virtual(bool is_virtual) { is_virtual_ = is_virtual; } bool class_method::is_const() const { return is_const_; } -void class_method::set_is_const(bool is_const) { is_const_ = is_const; } +void class_method::is_const(bool is_const) { is_const_ = is_const; } bool class_method::is_defaulted() const { return is_defaulted_; } -void class_method::set_is_defaulted(bool is_defaulted) +void class_method::is_defaulted(bool is_defaulted) { is_defaulted_ = is_defaulted; } bool class_method::is_static() const { return is_static_; } -void class_method::set_is_static(bool is_static) { is_static_ = is_static; } +void class_method::is_static(bool is_static) { is_static_ = is_static; } const std::vector &class_method::parameters() const { @@ -294,18 +294,18 @@ class_::class_(const std::vector &using_namespaces) bool class_::is_struct() const { return is_struct_; } -void class_::set_is_struct(bool is_struct) { is_struct_ = is_struct; } +void class_::is_struct(bool is_struct) { is_struct_ = is_struct; } bool class_::is_template() const { return is_template_; } -void class_::set_is_template(bool is_template) { is_template_ = is_template; } +void class_::is_template(bool is_template) { is_template_ = is_template; } bool class_::is_template_instantiation() const { return is_template_instantiation_; } -void class_::set_is_template_instantiation(bool is_template_instantiation) +void class_::is_template_instantiation(bool is_template_instantiation) { is_template_instantiation_ = is_template_instantiation; } diff --git a/src/uml/class_diagram_model.h b/src/uml/class_diagram_model.h index 7e478d78..05d5d2da 100644 --- a/src/uml/class_diagram_model.h +++ b/src/uml/class_diagram_model.h @@ -104,10 +104,10 @@ public: scope_t scope, const std::string &name, const std::string &type); bool is_relationship() const; - void set_is_relationship(bool is_relationship); + void is_relationship(bool is_relationship); bool is_static() const; - void set_is_static(bool is_static); + void is_static(bool is_static); private: bool is_relationship_{false}; @@ -129,19 +129,19 @@ public: scope_t scope, const std::string &name, const std::string &type); bool is_pure_virtual() const; - void set_is_pure_virtual(bool is_pure_virtual); + void is_pure_virtual(bool is_pure_virtual); bool is_virtual() const; - void set_is_virtual(bool is_virtual); + void is_virtual(bool is_virtual); bool is_const() const; - void set_is_const(bool is_const); + void is_const(bool is_const); bool is_defaulted() const; - void set_is_defaulted(bool is_defaulted); + void is_defaulted(bool is_defaulted); bool is_static() const; - void set_is_static(bool is_static); + void is_static(bool is_static); const std::vector ¶meters() const; void add_parameter(method_parameter &¶meter); @@ -231,13 +231,13 @@ public: class_(const std::vector &using_namespaces); bool is_struct() const; - void set_is_struct(bool is_struct); + void is_struct(bool is_struct); bool is_template() const; - void set_is_template(bool is_template); + void is_template(bool is_template); bool is_template_instantiation() const; - void set_is_template_instantiation(bool is_template_instantiation); + void is_template_instantiation(bool is_template_instantiation); void add_member(class_member &&member); void add_method(class_method &&method); diff --git a/src/uml/class_diagram_visitor.cc b/src/uml/class_diagram_visitor.cc index 0ed47cca..b0f16f68 100644 --- a/src/uml/class_diagram_visitor.cc +++ b/src/uml/class_diagram_visitor.cc @@ -324,7 +324,7 @@ void tu_visitor::process_class_declaration(const cppast::cpp_class &cls, type_safe::optional_ref tspec) { class_ c{ctx.config.using_namespace}; - c.set_is_struct(cls.class_kind() == cppast::cpp_class_kind::struct_t); + c.is_struct(cls.class_kind() == cppast::cpp_class_kind::struct_t); c.set_name(cx::util::full_name(ctx.namespace_, cls)); if (cls.comment().has_value()) @@ -757,7 +757,7 @@ void tu_visitor::process_static_field(const cppast::cpp_variable &mv, class_ &c, class_member m{detail::cpp_access_specifier_to_scope(as), mv.name(), cppast::to_string(mv.type())}; - m.set_is_static(true); + m.is_static(true); if (mv.comment().has_value()) m.add_decorators(decorators::parse(mv.comment().value())); @@ -773,11 +773,11 @@ void tu_visitor::process_method(const cppast::cpp_member_function &mf, { class_method m{detail::cpp_access_specifier_to_scope(as), util::trim(mf.name()), cppast::to_string(mf.return_type())}; - m.set_is_pure_virtual(cppast::is_pure(mf.virtual_info())); - m.set_is_virtual(cppast::is_virtual(mf.virtual_info())); - m.set_is_const(cppast::is_const(mf.cv_qualifier())); - m.set_is_defaulted(false); - m.set_is_static(false); + m.is_pure_virtual(cppast::is_pure(mf.virtual_info())); + m.is_virtual(cppast::is_virtual(mf.virtual_info())); + m.is_const(cppast::is_const(mf.cv_qualifier())); + m.is_defaulted(false); + m.is_static(false); if (mf.comment().has_value()) m.add_decorators(decorators::parse(mf.comment().value())); @@ -807,13 +807,13 @@ void tu_visitor::process_template_method( class_method m{ detail::cpp_access_specifier_to_scope(as), util::trim(mf.name()), type}; - m.set_is_pure_virtual(false); - m.set_is_virtual(false); - m.set_is_const(cppast::is_const( + m.is_pure_virtual(false); + m.is_virtual(false); + m.is_const(cppast::is_const( static_cast(mf.function()) .cv_qualifier())); - m.set_is_defaulted(false); - m.set_is_static(false); + m.is_defaulted(false); + m.is_static(false); if (mf.comment().has_value()) m.add_decorators(decorators::parse(mf.comment().value())); @@ -839,11 +839,11 @@ void tu_visitor::process_static_method(const cppast::cpp_function &mf, { class_method m{detail::cpp_access_specifier_to_scope(as), util::trim(mf.name()), cppast::to_string(mf.return_type())}; - m.set_is_pure_virtual(false); - m.set_is_virtual(false); - m.set_is_const(false); - m.set_is_defaulted(false); - m.set_is_static(true); + m.is_pure_virtual(false); + m.is_virtual(false); + m.is_const(false); + m.is_defaulted(false); + m.is_static(true); if (mf.comment().has_value()) m.add_decorators(decorators::parse(mf.comment().value())); @@ -864,11 +864,11 @@ void tu_visitor::process_constructor(const cppast::cpp_constructor &mf, { class_method m{detail::cpp_access_specifier_to_scope(as), util::trim(mf.name()), "void"}; - m.set_is_pure_virtual(false); - m.set_is_virtual(false); - m.set_is_const(false); - m.set_is_defaulted(false); - m.set_is_static(true); + m.is_pure_virtual(false); + m.is_virtual(false); + m.is_const(false); + m.is_defaulted(false); + m.is_static(true); if (mf.comment().has_value()) m.add_decorators(decorators::parse(mf.comment().value())); @@ -887,11 +887,11 @@ void tu_visitor::process_destructor(const cppast::cpp_destructor &mf, class_ &c, { class_method m{detail::cpp_access_specifier_to_scope(as), util::trim(mf.name()), "void"}; - m.set_is_pure_virtual(false); - m.set_is_virtual(cppast::is_virtual(mf.virtual_info())); - m.set_is_const(false); - m.set_is_defaulted(false); - m.set_is_static(true); + m.is_pure_virtual(false); + m.is_virtual(cppast::is_virtual(mf.virtual_info())); + m.is_const(false); + m.is_defaulted(false); + m.is_static(true); c.add_method(std::move(m)); } @@ -1399,7 +1399,7 @@ class_ tu_visitor::build_template_instantiation( tinst.set_name(ns + util::split(cppast::to_string(t), "<")[0]); - tinst.set_is_template_instantiation(true); + tinst.is_template_instantiation(true); if (tinst.full_name().substr(0, tinst.full_name().find('<')).find("::") == std::string::npos) {