Skip to content
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.

Sign up for GitHub

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

Jump to bottom

A4-7-1: /= is incorrectly reported #368

Closed
lcartey opened this issue Oct 2, 2023 · 0 comments
Closed

A4-7-1: /= is incorrectly reported #368

lcartey opened this issue Oct 2, 2023 · 0 comments
Assignees
@rvermeulen
Labels
Difficulty-Low A false positive or false negative report which is expected to take <1 day effort to address false positive/false negative An issue related to observed false positives or false negatives. Impact-Medium user-report Issue reported by an end user of CodeQL Coding Standards

Comments

@lcartey
Copy link
Collaborator

lcartey commented Oct 2, 2023

Affected rules

  • A4-7-1

Description

/= is incorrectly reported as causing data loss regardless of the arguments involved.

Example

i /= 2;
@lcartey lcartey added false positive/false negative An issue related to observed false positives or false negatives. Difficulty-Low A false positive or false negative report which is expected to take <1 day effort to address Impact-Medium user-report Issue reported by an end user of CodeQL Coding Standards labels Oct 2, 2023
@rvermeulen rvermeulen self-assigned this Jan 10, 2024
@rvermeulen rvermeulen mentioned this issue Jan 31, 2024
Merged
30 tasks
github-merge-queue bot pushed a commit that referenced this issue Feb 13, 2024
@rvermeulen
Merge pull request #516 from rvermeulen/rvermeulen/fix-368
bd425e0 
Fix #368
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rvermeulen rvermeulen

Labels
Difficulty-Low A false positive or false negative report which is expected to take <1 day effort to address false positive/false negative An issue related to observed false positives or false negatives. Impact-Medium user-report Issue reported by an end user of CodeQL Coding Standards
Projects
Coding Standards Public Development Board
Status: Done
Milestone
No milestone
Development

No branches or pull requests
3 participants
@rvermeulen @lcartey @knewbury01