Narrow AdvancedCollectionView trimming warning so that it only triggers in unsafe usages

[sylveon]

Fixes

Fixes #723

PR Type

What kind of change does this PR introduce?

• Bugfix

What is the current behavior?

Trimming warning triggers on every usage of AdvancedCollectionView

What is the new behavior?

Trimming warning only triggers when you make use of reflection-based sorting.

PR Checklist

Please check if your PR fulfills the following requirements:

• Created a feature/dev branch in your fork (vs. submitting directly from a commit on main)
• Based off latest main branch of toolkit
• Tested code with current supported SDKs
• New component
  • Documentation has been added
  • Sample in sample app has been added
  • Analyzers are passing for documentation and samples
  • Icon has been created (if new sample) following the Thumbnail Style Guide and templates
• Tests for the changes have been added (if applicable)
• Header has been added to all new source files
• Contains NO breaking changes

Other information

[Arlodotexe]

Looks good, but I'd like explicit approval from @Sergio0694 before we merge

[Sergio0694]

This is not a good way to fix this. This whole type would need to be reworked to be properly trim-safe. For instance, there's nothing enforcing that the T being used to create a sort descriptor will actually match the source type being used. Additionally annotating a T and then suppressing is not correct, the annotations should rather flow. As a rule of thumb, suppressions are almost always incorrect and should be avoided.

[Sergio0694]

Better than before but it's not entirely safe 😅