wpf icon indicating copy to clipboard operation
wpf copied to clipboard

Remove FriendAccessAllowedAttribute usage

Open ThomasGoulet73 opened this issue 2 years ago • 3 comments

Description

Remove FriendAccessAllowedAttribute usage since it is not enforced in .Net Core. FriendAccessAllowedAttribute is an undocumented and non-public attribute which is not supported anymore. To use FriendAccessAllowedAttribute, when supported, you need to set InternalsVisibleToAttribute.AllInternalsVisible to false, which is never set in this repo.

There are modifications to generated files in src/Microsoft.DotNet.Wpf/src/PresentationCore/System/Windows/Generated but these files are generated from Elements.cs which is not runnable in this repo right now (dotnet/wpf#6135) but it should be fine because once we can run MilCodeGen, there shouldn't be any diffs.

Customer Impact

None.

Regression

No.

Testing

Local build + CI.

Risk

Low.

Microsoft Reviewers: Open in CodeFlow

ThomasGoulet73 avatar May 09 '22 23:05 ThomasGoulet73

I rebased to fix the conflicts.

ThomasGoulet73 avatar Sep 30 '23 02:09 ThomasGoulet73

Thanks @ThomasGoulet73! We'll consider this PR for our upcoming CTPs.

pchaurasia14 avatar Sep 30 '23 06:09 pchaurasia14

I rebased to fix the conflicts.

ThomasGoulet73 avatar May 01 '24 01:05 ThomasGoulet73

I rebased to fix the conflicts.

Now, this branch has conflicts

lindexi avatar Sep 03 '24 00:09 lindexi

I rebased to fix the conflicts.

Thanks for the ping @lindexi.

ThomasGoulet73 avatar Sep 07 '24 04:09 ThomasGoulet73

https://github.com/dotnet/runtime/issues/10707 has some more details on this attribute.

JeremyKuhne avatar Oct 02 '24 19:10 JeremyKuhne

Awesome! Those replacements from Thomas will clear out so much clutter.

h3xds1nz avatar Oct 04 '24 08:10 h3xds1nz