-
Notifications
You must be signed in to change notification settings - Fork 177
bug(#4559): auto phi formation correctly parsed with application precedences #4560
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.
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
Draft
h1alexbel
wants to merge
23
commits into
objectionary:master
Choose a base branch
from
h1alexbel:4559
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Changes from 5 commits
Commits
Show all changes
23 commits
Select commit
Hold shift + click to select a range
06c4813
test(eo-parser): add test case for auto phi formation with applicatio…
h1alexbel 9bdb52f
test: update test cases to reflect new auto phi formation behavior wi…
h1alexbel da64738
refactor(XeEoListener): rename APPLICATION_PRECEDENCE to A_PRECEDENCE…
h1alexbel 554385c
bug(#4559): add new transpile pack for auto phi formation with applic…
h1alexbel d2f2257
bug(#4559): add new transpile pack for auto phi formation with applic…
h1alexbel 2ebe627
bug(#4559): last application
h1alexbel 32c2eec
bug(#4559): add auto-phi-with-application-precendence.yaml test resou…
h1alexbel 18aeaaf
bug(#4559): refine XSL template to exclude positioned objects from au…
h1alexbel 96763d6
bug(#4559): typo
h1alexbel eb1c1eb
bug(#4559): test case
h1alexbel d36d73f
bug(#4559): handle fully qualified names in auto-phi formation by str…
h1alexbel e2a0396
chore(#4559): remove empty line from test resource file
h1alexbel ece0c48
empty commit to restart pipelines
h1alexbel 7bedfad
bug(#4559): add test case for auto phi formation from FQN precedence …
h1alexbel 82cd9ba
bug(#4559): simplify Q-prefixed application handling by replacing Q w…
h1alexbel d9bea33
empty commit to restart pipelines
h1alexbel 4ae096f
bug(#4559): auto-phi formation from application with \rho
h1alexbel ac2de15
bug(#4559): extend identifier pattern to support Unicode letters and …
h1alexbel 0971a90
bug(#4559): redundant loop
h1alexbel 82f1cb7
bug(#4559): typo
h1alexbel a67b94b
bug(#4559): add support for QQ prefix in EO applications to map to Φ.…
h1alexbel cfc1e13
bug(#4559): add test case for auto phi formation from nested application
h1alexbel f384915
bug(#4559): skip auto-phi formation test due to parsing issues with n…
h1alexbel File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
26 changes: 26 additions & 0 deletions
26
...lang/maven/transpile-packs/transpiles-auto-phi-formation-with-application-precedence.yaml
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,26 @@ | ||
| # SPDX-FileCopyrightText: Copyright (c) 2016-2025 Objectionary.com | ||
| # SPDX-License-Identifier: MIT | ||
| --- | ||
| sheets: | ||
| - /org/eolang/maven/transpile/set-locators.xsl | ||
| - /org/eolang/maven/transpile/set-original-names.xsl | ||
| - /org/eolang/maven/transpile/classes.xsl | ||
| - /org/eolang/maven/transpile/tests.xsl | ||
| - /org/eolang/maven/transpile/attrs.xsl | ||
| - /org/eolang/maven/transpile/data.xsl | ||
| - /org/eolang/maven/transpile/to-java.xsl | ||
| asserts: | ||
| - /object[not(errors)] | ||
| - //java[contains(text(), 'Φ.custom-map.ap🌵46.φ')] | ||
| - //java[contains(text(), 'Φ.custom-map.ap🌵46.φ.ρ.ρ.α0')] | ||
| - //java[contains(text(), 'Phi rrbbb = Phi.Φ.take("org").take("eolang").take("text");')] | ||
| - //java[contains(text(), 'Phi rrbb1 = Phi.Φ.take("org").take("eolang").take("b");')] | ||
| - //java[contains(text(), 'rrbbb1 = new PhMethod(rrbbb1, "orgn");')] | ||
| - //java[contains(text(), 'Phi rrbb = new PhMethod(rrbbb, "what");')] | ||
| - //java[contains(text(), 'Phi rrb = new PhMethod(rrbb, "ttt");')] | ||
| input: | | ||
| [cont-list] > custom-map | ||
| mapped. > @ | ||
| cont-list | ||
| ((text orgn).what b).ttt >> [y] | ||
| origin > orgn! | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
17 changes: 17 additions & 0 deletions
17
...resources/org/eolang/parser/eo-syntax/auto-phi-formation-with-application-precedence.yaml
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,17 @@ | ||
| # SPDX-FileCopyrightText: Copyright (c) 2016-2025 Objectionary.com | ||
| # SPDX-License-Identifier: MIT | ||
| --- | ||
| # yamllint disable rule:line-length | ||
| sheets: [] | ||
| asserts: | ||
| - /object[not(errors)] | ||
| - //o[not(@base) and @name='ap🌵34' and o[@name='φ' and @base='.what']] | ||
| - //o[not(@base) and @name='ap🌵66' and o[@name='φ' and @base='.contains']] | ||
| input: | | ||
| [cont-list] > custom-map | ||
| foo > boom | ||
| ((text orgn).what b).ttt >> [y] | ||
| mapped. > @ | ||
| cont-list | ||
| (text orgn).contains x >> [x] | ||
| origin > orgn! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.