From c552e248c1b2d29684ab7199654009acd7f821cd Mon Sep 17 00:00:00 2001
From: eidheim <eidheim@gmail.com>
Date: Fri, 18 May 2018 22:11:33 +0200
Subject: [PATCH] Language protocol: optimization of autocomplete insertion
 when language server answers with incomplete insertText

---
 src/source_language_protocol.cc | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/src/source_language_protocol.cc b/src/source_language_protocol.cc
index 7d7813b..fc10ec1 100644
--- a/src/source_language_protocol.cc
+++ b/src/source_language_protocol.cc
@@ -1225,10 +1225,13 @@ void Source::LanguageProtocolView::setup_autocomplete() {
             auto detail=it->second.get<std::string>("detail", "");
             auto documentation=it->second.get<std::string>("documentation", "");
             auto insert=it->second.get<std::string>("insertText", "");
-            if(!insert.empty()) { // In case ( is missing in insert but is present in label
-              auto pos=label.find('(');
-              if(pos!=std::string::npos && pos==insert.size() && pos+1<label.size() && label.back()==')' && insert.find('(')==std::string::npos)
-                insert+="(${1:"+label.substr(pos+1, label.size()-1-(pos+1))+"})";
+            if(!insert.empty()) {
+              // In case ( is missing in insert but is present in label
+              if(label.size()>insert.size() && label.back()==')' && insert.find('(')==std::string::npos) {
+                auto pos=label.find('(');
+                if(pos!=std::string::npos && pos==insert.size() && pos+1<label.size())
+                  insert+="(${1:"+label.substr(pos+1, label.size()-1-(pos+1))+"})";
+              }
             }
             else {
               insert=label;