julia-vscode / SymbolServer.jl

Other
23 stars 31 forks source link

More fully describe datatypes #123

Closed ZacLN closed 4 years ago

ZacLN commented 4 years ago

This PR picks up a fair bit of information that is not currently stored:

On the down side:

@davidanthoff, this is more for discussion than immediate merge as changes would across the languagesever/statliclint setup

davidanthoff commented 4 years ago

Cool! This also generally means that we are using our own types to represent all of this, which might make it easier to switch the cache file format to say some JSON thing, right? If we do want to go in the cloud direction, then using a file format that is not serialization is probably something we'll have to do... Would also help with the idea to compile the whole LS.

I don't understand the return type part for methods :) How would we decide for which input types we compute return types?

ZacLN commented 4 years ago

This is ready now (+ SS/LS relevant PRs). On the return types - this is disabled for now (too time consuming). Use of return types would require a bit on inference our side (i.e. sin(1.0)::Float64 as a really simple example)

Note: this is breaking

ZacLN commented 4 years ago

Yep, sounds good