Skip to content
This repository was archived by the owner on Dec 29, 2022. It is now read-only.

Commit 77c32e9

Browse files
committed
Properly infer full_docs
3efe567 ("Enhanced hover tooltips") added the `full_docs` configuration option as an `Inferrable`. However, it failed to actually merge the option during a config update, causing a panic if the client specified `full_docs` to be `null`. Example panic: thread '<unnamed>' panicked at 'internal error: entered unreachable code', src/config.rs:111:33 Fix this to prevent the crash.
1 parent 67bce0b commit 77c32e9

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

src/config.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -206,6 +206,7 @@ impl Config {
206206
new.target_dir = self.target_dir.combine_with_default(&new.target_dir, None);
207207
new.build_lib = self.build_lib.combine_with_default(&new.build_lib, false);
208208
new.build_bin = self.build_bin.combine_with_default(&new.build_bin, None);
209+
new.full_docs = self.full_docs.combine_with_default(&new.full_docs, false);
209210

210211
*self = new;
211212
}

0 commit comments

Comments
 (0)