Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

remove painlessjson #219

Merged
merged 76 commits into from
Oct 27, 2022
Merged

remove painlessjson #219

merged 76 commits into from
Oct 27, 2022

Conversation

WebFreak001
Copy link
Member

@WebFreak001 WebFreak001 commented Mar 18, 2022 •
edited
Loading

fixes #216

plan:

  • workspace-d
  • protocol
  • serve-d

@WebFreak001 WebFreak001 changed the title remove painlessjson from :workspace-d remove painlessjson Mar 18, 2022
@WebFreak001
move served.lsp.protocol into own subpackage
bfbaf9b
@WebFreak001
migrate protocol from painlessjson to mir-ion
a5bf09f
@WebFreak001
add more tests
d114ec7
@WebFreak001
port lsp subpackage, add more tests
ce76baf
@WebFreak001
start porting serverbase
b9c7f6e
@WebFreak001
tests & documentation for TextDocumentManager.d
0872a5f
@WebFreak001
extend textdocumentmanager.d tests, fix bugs
e657c5d
@WebFreak001
ignore coverage files
8716f58
@WebFreak001
add :protocol subPackage
540e581
@WebFreak001
test filereader.readCodeWithBuffer
5525bc1 
+ fix max length issues
@WebFreak001
fix Variant!(NoneType,null) defaulting to null
f3ab352
@WebFreak001
start unittesting jsonrpc
38bc7b7
@WebFreak001
fix PosixStdinReader for remapped stdin
3932f83
@WebFreak001
fix PosixFileReader with non-stdin reading
cc24f46
@WebFreak001
fix jsonrpc not terminating correctly
4f0635f
@WebFreak001
increase RPCProcessor fiber size
4261b03 
needed for mir-ion, which uses about 16*4096 bytes of stack memory
@WebFreak001
test various RPC scenarios
4567fa9
@WebFreak001
improve null_server_test debugging
787a79b
@WebFreak001
make jsonops inout
dda84e7
@WebFreak001
fix ResponseError definition
eea19ff
@WebFreak001
fix LSP enums
594ab7a
@WebFreak001
remove painlessjson from serverbase
37314fb
@WebFreak001
fix PackageBuildSettings
3c5694a
@WebFreak001
add rpc.registerCapability method
daa1b6b
@WebFreak001
fix jsonrpc.log
0e66b18
@WebFreak001
fix jsonops.toJSON
9bf134c
@WebFreak001
add toStdJSONValue
829e418
@WebFreak001
fix deref issues with OptionalJSONValue
e1dee33
@WebFreak001
add debug logging
b7b5adf
@WebFreak001
add missing optionals
ad104e8
@WebFreak001
update mir-ion
b99c3a0
@WebFreak001
remove std.json dependencies
78bb4be
@WebFreak001
WIP mir finishing touches (needs mir-ion w/ #22)
a757a32
@WebFreak001
use mir void in Variant instead of custom NoneType
eecbc7f
@WebFreak001
update dependencies
32f602d
@WebFreak001
upgrade mir-ion
5845c69
@WebFreak001
add toString for Request/ResponseMessageRaw
42b15b9
@WebFreak001
test jsonrpc on windows as well
263ce9a
@WebFreak001
add timeout support + tracing to jsonrpc
77fcfd2
@WebFreak001
warn for common mir-ion struct misuse in protocol
bd0e46d
@WebFreak001
include serve-d version in LSP init response
8ae529a
@WebFreak001
fix tc_as_a_exe
6b3588a
@WebFreak001
only include serverInfo in regular builds
b95fd76
@WebFreak001
fix tc_dub_empty
2333893
@WebFreak001
improve CI logging
0957532
@WebFreak001
wait longer for serve-d to exit in test
153e471
@WebFreak001
add test step timeouts
f45c282
@WebFreak001
CI integration tests: build dmd in debug mode
91cffaf 
Optimizer breaks mir-ion code in release there
@WebFreak001
close FileReaders with core.sync.event
bb75cc5 
Renames Posix*Reader to BuggyPosix*Reader, because it's relying on
undefined behavior.

Also increase small delays inside jsonrpc unittests
@WebFreak001
exit with libc exit to not trigger GC
de39492 
Only on POSIX, works around GC segfault issue on exit on LDC
@WebFreak001
upgrade dependencies
0d11888
@WebFreak001
fix PosixStdinReader using slow polling
070cd8a
@WebFreak001
windows file reader attempt #1
97e86ac
@WebFreak001
windows file reader attempt #2
726831e
@WebFreak001
use -lowmem to fix CI runners
64b92f5 
On my Windows VM running `dub test` had LDC have a peak usage of 6.7 GB,
using `-lowmem` lowered peak usage by 2 GB.
@WebFreak001
Merge branch 'master' into remove-painlessjson
a8ddc76
@WebFreak001 WebFreak001 marked this pull request as ready for review October 27, 2022 23:13
@WebFreak001 WebFreak001 merged commit a8ddc76 into master Oct 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

remove painlessjson
1 participant
@WebFreak001