jyapayne/Nim
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Revert "nimsuggest: don't add CRLF to replies (#13545)" (#13597)

Browse source 
This reverts commit a974684b14.

The CRLF was to separate between each response, and removing them causes
the responses to not be splitted correctly.

I didn't notice that it was in a loop when I made the #13545.
This commit is contained in:
alaviss 2020-03-06 18:29:28 +00:00 • committed by GitHub
commit 4bd388ad58
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 4 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

8
nimsuggest/nimsuggest.nim
View file

@ -252,10 +252,10 @@ proc toSocket(stdoutSocket: Socket) {.gcsafe.} =
let res = results.recv() let res = results.recv()
case res.section case res.section
of ideNone: break of ideNone: break
of ideMsg: stdoutSocket.send(res.doc) of ideMsg: stdoutSocket.send(res.doc & "\c\L")
of ideKnown: stdoutSocket.send($(res.quality == 1)) of ideKnown: stdoutSocket.send($(res.quality == 1) & "\c\L")
of ideProject: stdoutSocket.send(res.filePath) of ideProject: stdoutSocket.send(res.filePath & "\c\L")
else: stdoutSocket.send($res) else: stdoutSocket.send($res & "\c\L")
proc toEpc(client: Socket; uid: BiggestInt) {.gcsafe.} = proc toEpc(client: Socket; uid: BiggestInt) {.gcsafe.} =
var list = newSList() var list = newSList()