From d85dc20742ae1d988bb7b6204031bf34931653fb Mon Sep 17 00:00:00 2001
From: eidheim <eidheim@gmail.com>
Date: Sat, 23 Sep 2017 15:49:09 +0200
Subject: [PATCH] Cleanup of CompletionString class

---
 src/CompletionString.cc | 30 +++++++++++++++---------------
 src/CompletionString.h  | 12 ++++++------
 2 files changed, 21 insertions(+), 21 deletions(-)

diff --git a/src/CompletionString.cc b/src/CompletionString.cc
index a154885..e343552 100644
--- a/src/CompletionString.cc
+++ b/src/CompletionString.cc
@@ -1,28 +1,28 @@
 #include "CompletionString.h"
 #include "Utility.h"
 
-clangmm::CompletionString::
-CompletionString(const CXCompletionString &cx_completion_sting) : cx_completion_sting(cx_completion_sting) {}
+clangmm::CompletionChunk::CompletionChunk(std::string text, CompletionChunkKind kind)
+    : text(std::move(text)), kind(kind) {}
 
-bool clangmm::CompletionString::available() {
+clangmm::CompletionString::CompletionString(const CXCompletionString &cx_completion_sting)
+    : cx_completion_sting(cx_completion_sting) {}
+
+bool clangmm::CompletionString::available() const {
   return clang_getCompletionAvailability(cx_completion_sting) == CXAvailability_Available;
 }
 
-unsigned clangmm::CompletionString::get_num_chunks() {
-    return clang_getNumCompletionChunks(cx_completion_sting);
+unsigned clangmm::CompletionString::get_num_chunks() const {
+  return clang_getNumCompletionChunks(cx_completion_sting);
 }
 
-std::vector<clangmm::CompletionChunk> clangmm::CompletionString::get_chunks() {
-  std::vector<CompletionChunk> res;
-  for (unsigned i = 0; i < get_num_chunks(); i++) {
-    res.emplace_back(to_string(clang_getCompletionChunkText(cx_completion_sting, i)), static_cast<CompletionChunkKind> (clang_getCompletionChunkKind(cx_completion_sting, i)));
-  }
-  return res;
+std::vector<clangmm::CompletionChunk> clangmm::CompletionString::get_chunks() const {
+  std::vector<CompletionChunk> chunks;
+  for(unsigned i = 0; i < get_num_chunks(); ++i)
+    chunks.emplace_back(to_string(clang_getCompletionChunkText(cx_completion_sting, i)),
+                        static_cast<CompletionChunkKind>(clang_getCompletionChunkKind(cx_completion_sting, i)));
+  return chunks;
 }
 
-std::string clangmm::CompletionString::get_brief_comments() {
+std::string clangmm::CompletionString::get_brief_comments() const {
   return to_string(clang_getCompletionBriefComment(cx_completion_sting));
 }
-
-clangmm::CompletionChunk::CompletionChunk(std::string chunk, CompletionChunkKind kind) :
-  chunk(chunk), kind(kind) { }
diff --git a/src/CompletionString.h b/src/CompletionString.h
index c03221b..e43501e 100644
--- a/src/CompletionString.h
+++ b/src/CompletionString.h
@@ -21,18 +21,18 @@ namespace clangmm {
 
   class CompletionChunk {
   public:
-    CompletionChunk(std::string chunk, CompletionChunkKind kind);
-    std::string chunk;
+    CompletionChunk(std::string text, CompletionChunkKind kind);
+    std::string text;
     CompletionChunkKind kind;
   };
 
   class CompletionString {
   public:
     explicit CompletionString(const CXCompletionString &cx_completion_sting);
-    bool available();
-    std::vector<CompletionChunk> get_chunks();
-    std::string get_brief_comments();
-    unsigned get_num_chunks();
+    bool available() const;
+    std::vector<CompletionChunk> get_chunks() const;
+    std::string get_brief_comments() const;
+    unsigned get_num_chunks() const;
     
     CXCompletionString cx_completion_sting;
   };