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

Invalid cyclic error for recursive readonly types #35686

Closed
rdhananjaya opened this issue Apr 6, 2022 · 3 comments
Closed

Invalid cyclic error for recursive readonly types #35686

rdhananjaya opened this issue Apr 6, 2022 · 3 comments
Assignees
@SasinduDilshara
Labels
Lang/Readonly Type `readonly` related issues Points/5 Equivalent to five day effort Priority/Blocker Team/CompilerFE All issues related to Language implementation and Compiler, this exclude run times. Type/Bug

Comments

@rdhananjaya
Copy link
Member

Description:

type T readonly & (A|B); // invalid cyclic type reference in '[B, T, B]'(BCE2036)
type A readonly & [T];
type B readonly & [T];

Steps to reproduce:

Affected Versions:
Ballerina 2201.0.2 (Swan Lake)

OS, DB, other environment details and versions:

Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):
@rdhananjaya rdhananjaya added Type/Bug Priority/Blocker Team/CompilerFE All issues related to Language implementation and Compiler, this exclude run times. Lang/Readonly Type `readonly` related issues labels Apr 6, 2022
@rdulmina
Copy link
Contributor

rdulmina commented Jun 1, 2022

Assigned to @prakanth97

@rdulmina rdulmina added the Points/5 Equivalent to five day effort label Jun 1, 2022
@rdulmina rdulmina assigned ushirask and unassigned ushirask Sep 21, 2022
@SasinduDilshara SasinduDilshara mentioned this issue Oct 18, 2022
Closed
13 tasks
SasinduDilshara added a commit to SasinduDilshara/ballerina-lang that referenced this issue Nov 15, 2022
@SasinduDilshara
Merge branch 'master' of https://github.com/ballerina-platform/baller…
2a37096 
…ina-lang into issue-ballerina-platform#35686
@SasinduDilshara
Copy link
Contributor

This is fixed with #40663

@github-project-automation github-project-automation bot moved this from PR Sent to Done in Ballerina Team Main Board Jan 12, 2024
@github-actions GitHub Actions
Copy link

This issue is NOT closed with a proper Reason/ label. Make sure to add proper reason label before closing. Please add or leave a comment with the proper reason label now.

      - Reason/EngineeringMistake - The issue occurred due to a mistake made in the past.
      - Reason/Regression - The issue has introduced a regression.
      - Reason/MultipleComponentInteraction - Issue occured due to interactions in multiple components.
      - Reason/Complex - Issue occurred due to complex scenario.
      - Reason/Invalid - Issue is invalid.
      - Reason/Other - None of the above cases.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@SasinduDilshara SasinduDilshara

Labels
Lang/Readonly Type `readonly` related issues Points/5 Equivalent to five day effort Priority/Blocker Team/CompilerFE All issues related to Language implementation and Compiler, this exclude run times. Type/Bug
Projects
Ballerina Team Main Board
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Master] Fix invalid cyclic error for recursive readonly types SasinduDilshara/ballerina-lang
4 participants
@rdhananjaya @rdulmina @ushirask @SasinduDilshara