#2451 closed defect (fixed)
Do not check model while saving
| Reported by: | Owned by: | Adeel Asghar | |
|---|---|---|---|
| Priority: | high | Milestone: | 1.9.4 |
| Component: | OMEdit | Version: | trunk |
| Keywords: | Cc: |
Description
Saving a model (or library) in OMEdit takes too long. It looks like OMEdit checks the full package before saving the files? This is unnecessary, we can use 'check model' to do that.
Saving a bundle of text files should take a split second, not 15s.
Change History (10)
comment:1 by , 11 years ago
| Component: | Backend → OMEdit |
|---|---|
| Owner: | changed from to |
| Type: | enhancement → defect |
comment:2 by , 11 years ago
comment:3 by , 11 years ago
Can you push those user changes only for the modified models? Now I see the translation of a bunch of models that is not affected by a minimalistic change in a single file.
comment:4 by , 10 years ago
| Milestone: | 1.9.1 → 1.9.2 |
|---|
This ticket was not closed for 1.9.1, which has now been released. It was batch modified for milestone 1.9.2 (but maybe an empty milestone was more appropriate; feel free to change it).
comment:5 by , 10 years ago
| Milestone: | 1.9.2 → 1.9.3 |
|---|
Milestone changed to 1.9.3 since 1.9.2 was released.
comment:7 by , 9 years ago
| Status: | new → accepted |
|---|
This issue has now been fixed in a development branch https://github.com/adeas31/OMEdit/tree/indentation for OpenModelica 1.9.4 final release.
comment:8 by , 9 years ago
| Resolution: | → fixed |
|---|---|
| Status: | accepted → closed |
The fix is now available via the nightly build.
No, we can't do that. We must parse the file before saving it. We need to push the user changes to the AST so that OMEdit can do all the graphics.