Skip to content

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

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

Champion: relax ordering constraints around ref and partial modifiers on type declarations #946

Closed
1 of 5 tasks
agocke opened this issue Sep 28, 2017 · 3 comments
Closed
1 of 5 tasks
Assignees
Labels
Needs Implementation The specification for this issue has been approved, it needs an implementation Proposal champion Proposal
Milestone

Comments

@agocke
Copy link
Member

agocke commented Sep 28, 2017

Currently, partial must appear directly before struct, class, or another type declaration keyword. If the type is a ref struct, ref must appear immediately before partial or struct. It seems likely that various other keywords could be used to disambiguate these contextual modifiers and allow us to relax the constraints on where partial and ref can appear in the modifier list.

  • Proposal added
  • Discussed in LDM
  • Decision in LDM
  • Finalized (done, rejected, inactive)
  • Spec'ed

See https://github.com/dotnet/csharplang/blob/master/meetings/2017/LDM-2017-12-04.md

@CyrusNajmabadi
Copy link
Member

Note: i've looked at https://github.com/dotnet/csharplang/blob/master/meetings/2017/LDM-2017-12-04.md, but i can't find where in it it refers to this issue. Can you link/copy the appropriate section?

Thanks!

@jcouv jcouv modified the milestones: 7.3, 8.0 candidate Apr 5, 2018
@jcouv jcouv assigned jcouv and unassigned mattwar Apr 6, 2018
@Joe4evr
Copy link
Contributor

Joe4evr commented Apr 6, 2018

Can restrictions around partial just be relaxed entirely? I'd like it if partial abstract class worked just the same as abstract partial class.

@333fred 333fred added the Needs Implementation The specification for this issue has been approved, it needs an implementation label Oct 15, 2020
@Joe4evr
Copy link
Contributor

Joe4evr commented Apr 29, 2022

See https://github.com/dotnet/csharplang/blob/master/meetings/2017/LDM-2017-12-04.md

So that link does not talk about partial anywhere (as @CyrusNajmabadi noted), so I did a quick search for the paper trail myself:

@RikkiGibson RikkiGibson self-assigned this May 17, 2024
@jcouv jcouv removed their assignment Nov 19, 2024
@dotnet dotnet locked and limited conversation to collaborators Nov 19, 2024
@RikkiGibson RikkiGibson converted this issue into discussion #8965 Jan 6, 2025

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

Labels
Needs Implementation The specification for this issue has been approved, it needs an implementation Proposal champion Proposal
Projects
None yet
Development

No branches or pull requests

9 participants