Closed
Bug 1423595
Opened 7 years ago
Closed 7 years ago
[Static Analysis] Add "performance-for-range-copy" to clang based static-analysis
Categories
(Developer Infrastructure :: Source Code Analysis, enhancement)
Developer Infrastructure
Source Code Analysis
Tracking
(firefox59 affected)
RESOLVED
INVALID
| Tracking | Status | |
|---|---|---|
| firefox59 | --- | affected |
People
(Reporter: andi, Assigned: andi)
References
(Blocks 1 open bug, )
Details
Attachments
(1 obsolete file)
In order to prevent the unnecessary copy of variable when looping over range based loops we should add this checker: "performance-for-range-copy".
For now there are no occurrences, but in the past there were a few, around 5 that were fixed lately.
| Comment hidden (mozreview-request) |
|
||
Comment 2•7 years ago
|
||
| mozreview-review | ||
Comment on attachment 8934996 [details]
Bug 1423595 - Add "performance-for-range-copy" to clang based static-analysis.
https://reviewboard.mozilla.org/r/205928/#review211470
I would prefer in alphabetic order
Attachment #8934996 -
Flags: review?(sledru)
|
Assignee | |
Comment 3•7 years ago
|
||
Stupid me, this was already part of:
>> - name: performance-*
>> publish: !!bool yes
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → INVALID
|
|
Assignee | |
Updated•7 years ago
|
Attachment #8934996 -
Attachment is obsolete: true
|
||
Updated•7 years ago
|
Product: Core → Firefox Build System
|
|
||
Updated•2 years ago
|
Product: Firefox Build System → Developer Infrastructure
You need to log in
before you can comment on or make changes to this bug.
Description
•