Skip to content

Commit

Permalink
Merge pull request #339 from amiramw/patch-1
Browse files Browse the repository at this point in the history
typo fix
  • Loading branch information
dbaeumer authored Nov 22, 2017
2 parents e0c6d2d + 967ebd8 commit b21a6c7
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions protocol.md
Original file line number Diff line number Diff line change
Expand Up @@ -1513,7 +1513,7 @@ Servers are allowed to run there own file watching mechansim and not rely on cli

- to our experience getting file watching on disk right is challenging, especially if it needs to be supported across multiple OSes.
- file watching is not for free especially if the implementation uses some sort of polling and keeps a file tree in memory to compare timestamps (as for example some node modules do)
- a client usually starts more than one server. If every server runs its own file watching is can become a CPU or memory problem.
- a client usually starts more than one server. If every server runs its own file watching it can become a CPU or memory problem.
- in general there are more server than client implementations. So this problem is better solved on the client side.


Expand Down Expand Up @@ -2873,4 +2873,4 @@ _Response_:
* result: `null` | [`WorkspaceEdit`](#workspaceedit) describing the modification to the workspace.
* error: code and message set in case an exception happens during the rename request.

_Registration Options_: `TextDocumentRegistrationOptions`
_Registration Options_: `TextDocumentRegistrationOptions`

0 comments on commit b21a6c7

Please sign in to comment.