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

Parser fileAdd() and fileUpdate calls. #8

Open
chrisber opened this issue Mar 13, 2015 · 0 comments
Open

Parser fileAdd() and fileUpdate calls. #8

chrisber opened this issue Mar 13, 2015 · 0 comments

Comments

@chrisber
Copy link
Contributor

Observer behavior of the parser is that on every character change it
tries to add the current file AddFile() to the host environment also it then tries
to add all files in the project to the host env. AddFiles().
To gain more performance we should move the file add process elsewhere.
chrisber referenced this issue in chrisber/typescript-addin Mar 13, 2015
@chrisber
[FIX] [WIP] Update script content on text edit #5 #8
47f1ce6 
Currently the content of a script is not updated wenn editing this lead
to the situation that the native typescript service and monodevelop are
out of sync, monodevelop then search for dom regions that are not
available. Result index out of range exception.

Note:
Observer behavior of the parser is that on every character change it
tries to add the current file (AddFile()) to the host environment also it then tries
to add all files in the project to the host env. (AddFiles).
To Gain performance we should move the file add proccess elsewhere.
chrisber referenced this issue in chrisber/typescript-addin Mar 15, 2015
@chrisber
Undo impl of static v8engine and add ScriptSnapshot #5 #8
b363119 
This commit withdraw's changes that were introduced do to an false assumption
about TypeScriptContext.cs.
We removed V8TypescriptProvider.cs and placed the V8Engine back to the
TypeScriptContext.cs to restore the old behavior.

To not to introduce V8Engine into the Typescript-addin we added a new
interace (IScriptSnapshotExtension) which adds properties to
IScriptSnapshot. The properties are needed to compare the changes from
an old and current snapshot.
@chrisber chrisber mentioned this issue Mar 15, 2015
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@chrisber