Details
-
Bug
-
Resolution: Done
-
P2: Important
-
Qt Creator 4.10.0-beta2
-
-
c65e480cbd9511b0641d9fdc46c86694767b3a87 (qt-creator/qt-creator/4.10)
Description
The redundant but mandated field rangeLength is absent from (some?) didChange messages. The current specification of didChange says "If range and rangeLength are omitted the new text is considered to be the full content of the document." (my emphasis), i.e. it is not allowed to specify just the range.
This may break some language servers that expect the mandated rangeLength member to be present.
See this ticket that discuss the redundancy of the rangeLength member and the fact that it is currently obligatory (unless the range member is also absent).
qtc.languageclient.messages: StdIOClient send data: qtc.languageclient.messages: Content-Length: 256 {"jsonrpc":"2.0","method":"textDocument/didChange","params":{"contentChanges":[{"range":{"end":{"character":19,"line":9},"start":{"character":19,"line":9}},"text":"\n "}],"textDocument":{"uri":"file:///Users/perm/MyQtProject/blipp.pl","version":2}}}
Attachments
For Gerrit Dashboard: QTCREATORBUG-22694 | ||||||
---|---|---|---|---|---|---|
# | Subject | Branch | Project | Status | CR | V |
268146,2 | LanguageClient: include the range length in document updates | 4.10 | qt-creator/qt-creator | Status: MERGED | +2 | 0 |