From 1bb87638162cf437db3a8b0e481f69b9f848d376 Mon Sep 17 00:00:00 2001 From: Fedor Isakov Date: Fri, 14 May 2021 19:14:29 +0200 Subject: [PATCH] Refactor AspectLookup to depend on repository, cache results --- build.xml | 2 + .../models/jetbrains.mps.coderules.build.mps | 5 + .../jetbrains.mps.coderules.aspect@tests.mps | 19 + ...ains.mps.coderules.testhelper.testutil.mps | 25 +- ...rains.mps.coderules.typechecking@tests.mps | 23 +- ...s.mps.coderules.typechecking.ui.plugin.mps | 79 +- ...ins.mps.coderules.typechecking.service.mps | 30 +- .../jetbrains.mps.coderules/models/plugin.mps | 25 + .../models/plugin.mps | 42 +- .../jetbrains.mps.lang.coderules.template.msd | 1 + .../jetbrains.mps.lang.coderules.aspect.mps | 1009 +++++++++++++++-- .../jetbrains.mps.lang.controlflow.aspect.mps | 68 +- ...jetbrains.mps.lang.typechecking.aspect.mps | 70 +- 13 files changed, 1094 insertions(+), 304 deletions(-) diff --git a/build.xml b/build.xml index fdbce08c..d7669265 100644 --- a/build.xml +++ b/build.xml @@ -628,6 +628,7 @@ + @@ -1818,6 +1819,7 @@ + diff --git a/coderules/solutions/jetbrains.mps.coderules.build/models/jetbrains.mps.coderules.build.mps b/coderules/solutions/jetbrains.mps.coderules.build/models/jetbrains.mps.coderules.build.mps index 8704014d..2011020c 100644 --- a/coderules/solutions/jetbrains.mps.coderules.build/models/jetbrains.mps.coderules.build.mps +++ b/coderules/solutions/jetbrains.mps.coderules.build/models/jetbrains.mps.coderules.build.mps @@ -2655,6 +2655,11 @@ + + + + + diff --git a/coderules/solutions/jetbrains.mps.coderules.test/models/jetbrains.mps.coderules.aspect@tests.mps b/coderules/solutions/jetbrains.mps.coderules.test/models/jetbrains.mps.coderules.aspect@tests.mps index 8c46999e..27aed61d 100644 --- a/coderules/solutions/jetbrains.mps.coderules.test/models/jetbrains.mps.coderules.aspect@tests.mps +++ b/coderules/solutions/jetbrains.mps.coderules.test/models/jetbrains.mps.coderules.aspect@tests.mps @@ -2228,6 +2228,25 @@ + + + + + + + + + + + + + + + + + + + diff --git a/coderules/solutions/jetbrains.mps.coderules.testhelper/models/jetbrains.mps.coderules.testhelper.testutil.mps b/coderules/solutions/jetbrains.mps.coderules.testhelper/models/jetbrains.mps.coderules.testhelper.testutil.mps index 734d7f97..4457f827 100644 --- a/coderules/solutions/jetbrains.mps.coderules.testhelper/models/jetbrains.mps.coderules.testhelper.testutil.mps +++ b/coderules/solutions/jetbrains.mps.coderules.testhelper/models/jetbrains.mps.coderules.testhelper.testutil.mps @@ -1268,6 +1268,9 @@ + + + @@ -1279,9 +1282,6 @@ - - - @@ -1398,14 +1398,19 @@ - - - - - + + + + + + + - - + + + + + diff --git a/coderules/solutions/jetbrains.mps.coderules.typechecking.test/models/jetbrains.mps.coderules.typechecking@tests.mps b/coderules/solutions/jetbrains.mps.coderules.typechecking.test/models/jetbrains.mps.coderules.typechecking@tests.mps index e4b0d94c..274bf584 100644 --- a/coderules/solutions/jetbrains.mps.coderules.typechecking.test/models/jetbrains.mps.coderules.typechecking@tests.mps +++ b/coderules/solutions/jetbrains.mps.coderules.typechecking.test/models/jetbrains.mps.coderules.typechecking@tests.mps @@ -3083,17 +3083,22 @@ - - - - - - + + + + + + - - - + + + + + + + + diff --git a/coderules/solutions/jetbrains.mps.coderules.typechecking.ui/models/jetbrains.mps.coderules.typechecking.ui.plugin.mps b/coderules/solutions/jetbrains.mps.coderules.typechecking.ui/models/jetbrains.mps.coderules.typechecking.ui.plugin.mps index b37712a6..9b819250 100644 --- a/coderules/solutions/jetbrains.mps.coderules.typechecking.ui/models/jetbrains.mps.coderules.typechecking.ui.plugin.mps +++ b/coderules/solutions/jetbrains.mps.coderules.typechecking.ui/models/jetbrains.mps.coderules.typechecking.ui.plugin.mps @@ -1104,24 +1104,29 @@ - - - - - - - - - - - - - - + + + + + + + + + + + + + - - + + + + + + + + @@ -1899,21 +1904,26 @@ - - - - - - + + + + + + + + + + + - - - - - - - + + + + + + + @@ -3062,6 +3072,9 @@ + + + @@ -3077,14 +3090,6 @@ - - - - - - - - diff --git a/coderules/solutions/jetbrains.mps.coderules.typechecking/models/jetbrains.mps.coderules.typechecking.service.mps b/coderules/solutions/jetbrains.mps.coderules.typechecking/models/jetbrains.mps.coderules.typechecking.service.mps index ac91bde0..46744e23 100644 --- a/coderules/solutions/jetbrains.mps.coderules.typechecking/models/jetbrains.mps.coderules.typechecking.service.mps +++ b/coderules/solutions/jetbrains.mps.coderules.typechecking/models/jetbrains.mps.coderules.typechecking.service.mps @@ -11035,6 +11035,9 @@ + + + @@ -11044,9 +11047,6 @@ - - - @@ -11134,6 +11134,9 @@ + + + @@ -11145,9 +11148,6 @@ - - - @@ -11231,15 +11231,15 @@ + + + - - - @@ -11318,15 +11318,15 @@ + + + - - - @@ -11399,6 +11399,9 @@ + + + @@ -11410,9 +11413,6 @@ - - - diff --git a/coderules/solutions/jetbrains.mps.coderules/models/plugin.mps b/coderules/solutions/jetbrains.mps.coderules/models/plugin.mps index 5ef395cc..e9ed39aa 100644 --- a/coderules/solutions/jetbrains.mps.coderules/models/plugin.mps +++ b/coderules/solutions/jetbrains.mps.coderules/models/plugin.mps @@ -34,6 +34,7 @@ + @@ -8342,5 +8343,29 @@ + + + + + + + + + + + + + + + + + + + + + + + + diff --git a/coderules/solutions/jetbrains.mps.controlflow.ui/models/plugin.mps b/coderules/solutions/jetbrains.mps.controlflow.ui/models/plugin.mps index 65210a88..36f3e4ee 100644 --- a/coderules/solutions/jetbrains.mps.controlflow.ui/models/plugin.mps +++ b/coderules/solutions/jetbrains.mps.controlflow.ui/models/plugin.mps @@ -452,21 +452,26 @@ - - - - - - + + + + + + + + + + + - - - - - - - + + + + + + + @@ -885,6 +890,9 @@ + + + @@ -908,14 +916,6 @@ - - - - - - - - diff --git a/coderules/solutions/jetbrains.mps.lang.coderules.template/jetbrains.mps.lang.coderules.template.msd b/coderules/solutions/jetbrains.mps.lang.coderules.template/jetbrains.mps.lang.coderules.template.msd index b56ccd08..86800da6 100644 --- a/coderules/solutions/jetbrains.mps.lang.coderules.template/jetbrains.mps.lang.coderules.template.msd +++ b/coderules/solutions/jetbrains.mps.lang.coderules.template/jetbrains.mps.lang.coderules.template.msd @@ -19,6 +19,7 @@ bbf5c548-7111-4a53-a117-cdefc664cf34(jetbrains.mps.logic.reactor) 9a4afe51-f114-4595-b5df-048ce3c596be(jetbrains.mps.runtime) f113d56b-63e3-4120-a89c-2f5dcb8c6290(jetbrains.mps.coderules.runtime) + 3f233e7f-b8a6-46d2-a57f-795d56775243(Annotations) diff --git a/coderules/solutions/jetbrains.mps.lang.coderules.template/models/jetbrains.mps.lang.coderules.aspect.mps b/coderules/solutions/jetbrains.mps.lang.coderules.template/models/jetbrains.mps.lang.coderules.aspect.mps index 59c79508..f43369ee 100644 --- a/coderules/solutions/jetbrains.mps.lang.coderules.template/models/jetbrains.mps.lang.coderules.aspect.mps +++ b/coderules/solutions/jetbrains.mps.lang.coderules.template/models/jetbrains.mps.lang.coderules.aspect.mps @@ -20,6 +20,9 @@ + + + @@ -163,6 +166,7 @@ + @@ -318,12 +322,16 @@ + + + + @@ -341,6 +349,18 @@ + + + + + + + + + + + + @@ -448,6 +468,12 @@ + + + + + + @@ -468,16 +494,15 @@ - - - - - - - - - - + + + + + + + + + @@ -490,7 +515,501 @@ + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + @@ -504,7 +1023,7 @@ - + @@ -519,6 +1038,19 @@ + + + + + + + + + + + + + @@ -541,9 +1073,6 @@ - - - @@ -593,8 +1122,8 @@ - - + + @@ -708,12 +1237,6 @@ - - - - - - @@ -774,9 +1297,6 @@ - - - @@ -927,9 +1447,6 @@ - - - @@ -953,21 +1470,14 @@ - - - - - + + + + - + - - - - - - @@ -978,12 +1488,6 @@ - - - - - - @@ -1017,27 +1521,17 @@ - - - - - + + + + - + - + - - - - - - - - - @@ -1046,12 +1540,6 @@ - - - - - - @@ -1069,6 +1557,97 @@ + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + @@ -1076,13 +1655,79 @@ - - - - - - + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + @@ -1095,12 +1740,12 @@ - - + + - - + + @@ -1216,6 +1861,32 @@ + + + + + + + + + + + + + + + + + + + + + + + + + + @@ -1223,8 +1894,8 @@ - - + + @@ -1234,20 +1905,39 @@ - - + + - - - - - - + + + + + + + + + + + + + + + + + + + + + + + + + @@ -1277,8 +1967,8 @@ - - + + @@ -1349,20 +2039,123 @@ + + + + + + + + + + + + + + + + - - + + - + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + @@ -1382,6 +2175,14 @@ + + + + + + + + @@ -1428,6 +2229,28 @@ + + + + + + + + + + + + + + + + + + + + + + diff --git a/coderules/solutions/jetbrains.mps.lang.controlflow.runtime/models/jetbrains.mps.lang.controlflow.aspect.mps b/coderules/solutions/jetbrains.mps.lang.controlflow.runtime/models/jetbrains.mps.lang.controlflow.aspect.mps index 2f516909..73fb07b6 100644 --- a/coderules/solutions/jetbrains.mps.lang.controlflow.runtime/models/jetbrains.mps.lang.controlflow.aspect.mps +++ b/coderules/solutions/jetbrains.mps.lang.controlflow.runtime/models/jetbrains.mps.lang.controlflow.aspect.mps @@ -18,12 +18,7 @@ - - - - - @@ -45,9 +40,6 @@ - - - @@ -85,9 +77,6 @@ - - - @@ -96,12 +85,6 @@ - - - - - - @@ -109,6 +92,12 @@ + + + + + + @@ -121,6 +110,9 @@ + + + @@ -133,48 +125,6 @@ - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - diff --git a/coderules/solutions/jetbrains.mps.lang.typechecking.runtime/models/jetbrains.mps.lang.typechecking.aspect.mps b/coderules/solutions/jetbrains.mps.lang.typechecking.runtime/models/jetbrains.mps.lang.typechecking.aspect.mps index d19bfc60..0a4acf25 100644 --- a/coderules/solutions/jetbrains.mps.lang.typechecking.runtime/models/jetbrains.mps.lang.typechecking.aspect.mps +++ b/coderules/solutions/jetbrains.mps.lang.typechecking.runtime/models/jetbrains.mps.lang.typechecking.aspect.mps @@ -19,12 +19,7 @@ - - - - - @@ -46,9 +41,6 @@ - - - @@ -86,9 +78,6 @@ - - - @@ -97,12 +86,6 @@ - - - - - - @@ -110,6 +93,12 @@ + + + + + + @@ -122,6 +111,9 @@ + + + @@ -133,49 +125,7 @@ - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - +