| Summary: | Assignment in condition checker fails when assignment expressions are preceded by comma operators | ||||||
|---|---|---|---|---|---|---|---|
| Product: | [Tools] CDT | Reporter: | Meisam <meisam.fathi> | ||||
| Component: | cdt-codan | Assignee: | Marco Stornelli <marco.stornelli> | ||||
| Status: | RESOLVED FIXED | QA Contact: | Elena Laskavaia <elaskavaia.cdt> | ||||
| Severity: | normal | ||||||
| Priority: | P3 | CC: | cdtdoug, marco.stornelli, zeratul976 | ||||
| Version: | 7.0 | ||||||
| Target Milestone: | 9.8.0 | ||||||
| Hardware: | PC | ||||||
| OS: | All | ||||||
| See Also: |
https://git.eclipse.org/r/141614 https://git.eclipse.org/c/cdt/org.eclipse.cdt.git/commit/?id=ec6f9d204dc2ba6359276467cc82bfba480e07f8 |
||||||
| Whiteboard: | |||||||
| Attachments: |
|
||||||
Created attachment 175127 [details]
path to fix the bug + testcases
(In reply to comment #1) > Created attachment 175127 [details] > path to fix the bug + testcases Thanks for writing a patch! Could you please submit it to Gerrit? See http://wiki.eclipse.org/CDT/git#Using_Gerrit_for_CDT New Gerrit change created: https://git.eclipse.org/r/141614 Gerrit change https://git.eclipse.org/r/141614 was merged to [master]. Commit: http://git.eclipse.org/c/cdt/org.eclipse.cdt.git/commit/?id=ec6f9d204dc2ba6359276467cc82bfba480e07f8 |
test case: while (error_code = read_from_file(), error_code = NO_ERROR) { // AssignmentInConditionChecker must warn here // ... }