From e386c634aa321d47f407cb405e7c85571ab2626d Mon Sep 17 00:00:00 2001 From: Max Brunsfeld Date: Thu, 3 Sep 2015 17:19:43 -0700 Subject: [PATCH] Remove unused Grammar methods --- include/tree_sitter/compiler.h | 7 ++-- src/compiler/grammar.cc | 60 ++++++++++++---------------------- 2 files changed, 22 insertions(+), 45 deletions(-) diff --git a/include/tree_sitter/compiler.h b/include/tree_sitter/compiler.h index 0c62dcc9..8128da49 100644 --- a/include/tree_sitter/compiler.h +++ b/include/tree_sitter/compiler.h @@ -43,14 +43,11 @@ class Grammar { public: explicit Grammar(const std::vector> &); - bool operator==(const Grammar &) const; - std::string start_rule_name() const; - const rules::rule_ptr rule(const std::string &) const; + Grammar &ubiquitous_tokens(const std::set &); + Grammar &expected_conflicts(const std::set> &); const std::vector> &rules() const; const std::set &ubiquitous_tokens() const; - Grammar &ubiquitous_tokens(const std::set &); const std::set> &expected_conflicts() const; - Grammar &expected_conflicts(const std::set> &); }; enum GrammarErrorType { diff --git a/src/compiler/grammar.cc b/src/compiler/grammar.cc index 9cd1a555..4fae3163 100644 --- a/src/compiler/grammar.cc +++ b/src/compiler/grammar.cc @@ -10,27 +10,29 @@ using std::string; using std::vector; using rules::rule_ptr; -Grammar::Grammar(const std::vector> &rules) +Grammar::Grammar(const vector> &rules) : rules_(rules), ubiquitous_tokens_({}) {} -bool Grammar::operator==(const Grammar &other) const { - if (other.rules_.size() != rules_.size()) - return false; - - for (size_t i = 0; i < rules_.size(); i++) { - auto &pair = rules_[i]; - auto &other_pair = other.rules_[i]; - if (other_pair.first != pair.first) - return false; - if (!other_pair.second->operator==(*pair.second)) - return false; - } - - return true; +const vector> &Grammar::rules() const { + return rules_; } -string Grammar::start_rule_name() const { - return rules_.front().first; +const set &Grammar::ubiquitous_tokens() const { + return ubiquitous_tokens_; +} + +const set> &Grammar::expected_conflicts() const { + return expected_conflicts_; +} + +Grammar &Grammar::ubiquitous_tokens(const set &ubiquitous_tokens) { + ubiquitous_tokens_ = ubiquitous_tokens; + return *this; +} + +Grammar &Grammar::expected_conflicts(const set> &expected_conflicts) { + expected_conflicts_ = expected_conflicts; + return *this; } ostream &operator<<(ostream &stream, const Grammar &grammar) { @@ -48,7 +50,7 @@ ostream &operator<<(ostream &stream, const Grammar &grammar) { return stream << string("}>"); } -GrammarError::GrammarError(GrammarErrorType type, std::string message) +GrammarError::GrammarError(GrammarErrorType type, string message) : type(type), message(message) {} bool GrammarError::operator==(const GrammarError &other) const { @@ -62,26 +64,4 @@ ostream &operator<<(ostream &stream, const GrammarError *error) { return stream << string("#"); } -const set &Grammar::ubiquitous_tokens() const { - return ubiquitous_tokens_; -} - -Grammar &Grammar::ubiquitous_tokens(const set &ubiquitous_tokens) { - ubiquitous_tokens_ = ubiquitous_tokens; - return *this; -} - -const set> &Grammar::expected_conflicts() const { - return expected_conflicts_; -} - -Grammar &Grammar::expected_conflicts(const set> &expected_conflicts) { - expected_conflicts_ = expected_conflicts; - return *this; -} - -const vector> &Grammar::rules() const { - return rules_; -} - } // namespace tree_sitter