Closed Bug 1423595 Opened 5 years ago Closed 5 years ago

[Static Analysis] Add "performance-for-range-copy" to clang based static-analysis

Categories

(Developer Infrastructure :: Source Code Analysis, enhancement)

enhancement
Not set
normal

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 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)
Stupid me, this was already part of:

>> - name: performance-*
>>   publish: !!bool yes
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → INVALID
Attachment #8934996 - Attachment is obsolete: true
Product: Core → Firefox Build System
Product: Firefox Build System → Developer Infrastructure
You need to log in before you can comment on or make changes to this bug.