cs/unused-collection false positive?
- Ngôn ngữ chính
- CodeQL
- Star
- 10.1k
- Fork
- 2.1k
- Merge trung bình
- 2 ngày 15 giờ
- Pull request đã merge (30 ngày)
- 141
Mô tả
Hope this is the right place to file analyzer issues as well, didn't really see any info on recommendations for reporting issues anywhere.
We saw a "Container contents are never accessed" error on our report `cs/unused-collection`: https://github.com/CommunityToolkit/Windows/security/code-scanning/9
Flagging this code in our file:
```cs
private List _overflow = new List();
```
But it was used here in our ArrangeOverride method:
```cs
///
protected override Size ArrangeOverride(Size finalSize)
{
// Have grid to the bulk of our heavy lifting.
var size = base.ArrangeOverride(finalSize);
// Make sure all overflown elements have no size.
foreach (var child in _overflow)
{
child.Arrange(default);
}
```
Is this due to the `foreach` usage or because this is in a framework/platform method that is uncalled by our codebase directly but will be called by the XAML/UWP/WindowsAppSDK platform?
Hướng dẫn đóng góp
Hướng nghiên cứu
Review the code-scanning report for cs/unused-collection and compare it with the shown C# ArrangeOverride method and foreach usage. Determine whether the finding is a false positive related to foreach or framework-invoked methods; done means documenting the cause and identifying the analyzer behavior that needs correction.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- csharp
- Lĩnh vực
- devtools, security
- Loại issue
- Lỗi
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Khá rõ ràng
- Mức phù hợp với người mới
- 30/100