-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
[tracking]: Remove SafeFileHandle for Unix #61112
Comments
I couldn't figure out the best area label to add to this issue. If you have write-permissions please help me learn by adding exactly one area label. |
The proposal here is to find a way to move SafeFileHandle and such classes out of Microsoft.Win32.SafeHandles into some namespace that doesn't have the historic ties with Windows. Unfortunately as far as I can see that requires a nonexistent feature that allows types to be forwarded between namespaces. I would expect we would need much more compelling reasons to do something like that. @jkotas has that ever come up? |
Tagging subscribers to this area: @dotnet/area-system-runtime Issue Details@deeprobin yeah it's a historical issue, and would need a hypothetical feature to fix: #26227 (comment) Originally posted by @danmoseley in #60507 (comment)
|
Namespace forwarding is a very complex feature to build. It comes up occasionally when the technology names or positioning changes, the old names are not good fit anymore and people would like to stay compatible with old names and have new names at the same time. In practice, the changes in technology names or positioning are often accompanied by other (breaking) changes, so namespace forwarding would not actually help to fully solve the problem in many cases. |
Thanks. @deeprobin I'm going to close this because we don't have sufficient compelling reason to make such a major investment - although I don't like the name either. thanks for opening this tracking issue though. |
@deeprobin yeah it's a historical issue, and would need a hypothetical feature to fix: #26227 (comment)
Originally posted by @danmoseley in #60507 (comment)
The text was updated successfully, but these errors were encountered: